gpio: stp-xway: Use the of_property_read_u32 helper
authorMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Tue, 26 May 2015 21:12:00 +0000 (23:12 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 2 Jun 2015 09:22:02 +0000 (11:22 +0200)
This removes some redundant code but does not have any functional impact.

Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-stp-xway.c

index 6d4148f53b51a1567a08139713a264c066ad01c3..81bdbe7ba2a4bce1b8e103d3b06d789b13e92e70 100644 (file)
@@ -200,7 +200,7 @@ static int xway_stp_hw_init(struct xway_stp *chip)
 static int xway_stp_probe(struct platform_device *pdev)
 {
        struct resource *res;
-       const __be32 *shadow, *groups, *dsl, *phy;
+       u32 shadow, groups, dsl, phy;
        struct xway_stp *chip;
        struct clk *clk;
        int ret = 0;
@@ -223,33 +223,28 @@ static int xway_stp_probe(struct platform_device *pdev)
        chip->gc.owner = THIS_MODULE;
 
        /* store the shadow value if one was passed by the devicetree */
-       shadow = of_get_property(pdev->dev.of_node, "lantiq,shadow", NULL);
-       if (shadow)
-               chip->shadow = be32_to_cpu(*shadow);
+       if (!of_property_read_u32(pdev->dev.of_node, "lantiq,shadow", &shadow))
+               chip->shadow = shadow;
 
        /* find out which gpio groups should be enabled */
-       groups = of_get_property(pdev->dev.of_node, "lantiq,groups", NULL);
-       if (groups)
-               chip->groups = be32_to_cpu(*groups) & XWAY_STP_GROUP_MASK;
+       if (!of_property_read_u32(pdev->dev.of_node, "lantiq,groups", &groups))
+               chip->groups = groups & XWAY_STP_GROUP_MASK;
        else
                chip->groups = XWAY_STP_GROUP0;
        chip->gc.ngpio = fls(chip->groups) * 8;
 
        /* find out which gpios are controlled by the dsl core */
-       dsl = of_get_property(pdev->dev.of_node, "lantiq,dsl", NULL);
-       if (dsl)
-               chip->dsl = be32_to_cpu(*dsl) & XWAY_STP_ADSL_MASK;
+       if (!of_property_read_u32(pdev->dev.of_node, "lantiq,dsl", &dsl))
+               chip->dsl = dsl & XWAY_STP_ADSL_MASK;
 
        /* find out which gpios are controlled by the phys */
        if (of_machine_is_compatible("lantiq,ar9") ||
                        of_machine_is_compatible("lantiq,gr9") ||
                        of_machine_is_compatible("lantiq,vr9")) {
-               phy = of_get_property(pdev->dev.of_node, "lantiq,phy1", NULL);
-               if (phy)
-                       chip->phy1 = be32_to_cpu(*phy) & XWAY_STP_PHY_MASK;
-               phy = of_get_property(pdev->dev.of_node, "lantiq,phy2", NULL);
-               if (phy)
-                       chip->phy2 = be32_to_cpu(*phy) & XWAY_STP_PHY_MASK;
+               if (!of_property_read_u32(pdev->dev.of_node, "lantiq,phy1", &phy))
+                       chip->phy1 = phy & XWAY_STP_PHY_MASK;
+               if (!of_property_read_u32(pdev->dev.of_node, "lantiq,phy2", &phy))
+                       chip->phy2 = phy & XWAY_STP_PHY_MASK;
        }
 
        /* check which edge trigger we should use, default to a falling edge */