net: fec: Rename "phy-reset-active-low" property
authorBernhard Walle <bernhard@bwalle.de>
Thu, 3 Mar 2016 09:15:55 +0000 (10:15 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Mar 2016 03:47:16 +0000 (22:47 -0500)
is actually "active high". Thanks for Troy Kisky for pointing
that out.

Since the patch is in linux-next, this patch is incremental and doesn't
replace the original patch.

Signed-off-by: Bernhard Walle <bernhard@bwalle.de>
Acked-by: Fugang Duan <fugang.duan@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Documentation/devicetree/bindings/net/fsl-fec.txt
drivers/net/ethernet/freescale/fec_main.c

index a4799fff0d1f0cf4a45af3a2fd265e885b59c32f..b037a9d78d931312f9f950db4e358c8e86a5c86b 100644 (file)
@@ -12,7 +12,7 @@ Optional properties:
   only if property "phy-reset-gpios" is available.  Missing the property
   will have the duration be 1 millisecond.  Numbers greater than 1000 are
   invalid and 1 millisecond will be used instead.
-- phy-reset-active-low : If present then the reset sequence using the GPIO
+- phy-reset-active-high : If present then the reset sequence using the GPIO
   specified in the "phy-reset-gpios" property is reversed (H=reset state,
   L=operation state).
 - phy-supply : regulator that powers the Ethernet PHY.
index bad0ba29a94a19478a48965beefdae558557772a..37c08158308433845f583671dc23d6db411178df 100644 (file)
@@ -3191,7 +3191,7 @@ static int fec_enet_init(struct net_device *ndev)
 static void fec_reset_phy(struct platform_device *pdev)
 {
        int err, phy_reset;
-       bool active_low = false;
+       bool active_high = false;
        int msec = 1;
        struct device_node *np = pdev->dev.of_node;
 
@@ -3207,17 +3207,17 @@ static void fec_reset_phy(struct platform_device *pdev)
        if (!gpio_is_valid(phy_reset))
                return;
 
-       active_low = of_property_read_bool(np, "phy-reset-active-low");
+       active_high = of_property_read_bool(np, "phy-reset-active-high");
 
        err = devm_gpio_request_one(&pdev->dev, phy_reset,
-                       active_low ? GPIOF_OUT_INIT_HIGH : GPIOF_OUT_INIT_LOW,
+                       active_high ? GPIOF_OUT_INIT_HIGH : GPIOF_OUT_INIT_LOW,
                        "phy-reset");
        if (err) {
                dev_err(&pdev->dev, "failed to get phy-reset-gpios: %d\n", err);
                return;
        }
        msleep(msec);
-       gpio_set_value_cansleep(phy_reset, !active_low);
+       gpio_set_value_cansleep(phy_reset, !active_high);
 }
 #else /* CONFIG_OF */
 static void fec_reset_phy(struct platform_device *pdev)