of_mdio: merge branch tails in of_phy_register_fixed_link()
authorSergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Sat, 12 Aug 2017 21:03:06 +0000 (00:03 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 14 Aug 2017 03:07:26 +0000 (20:07 -0700)
Looks  like gcc isn't always able to figure  out that 3 *if* branches in
of_phy_register_fixed_link() calling fixed_phy_register() at their ends
are similar enough and thus can be merged. The "manual" merge saves 40
bytes of the object code (AArch64 gcc 4.8.5), and still saves 12 bytes
even  if gcc was able to merge the branch tails (ARM gcc 4.8.5)...

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/of/of_mdio.c

index 94ca3470e94337459080a50b1eb3c7dab1119063..b14a00034fb12a1b493e5f7afeceb7c094532e2b 100644 (file)
@@ -422,16 +422,13 @@ int of_phy_register_fixed_link(struct device_node *np)
        struct fixed_phy_status status = {};
        struct device_node *fixed_link_node;
        u32 fixed_link_prop[5];
-       struct phy_device *phy;
        const char *managed;
-       int link_gpio;
+       int link_gpio = -1;
 
-       if (of_property_read_string(np, "managed", &managed) == 0) {
-               if (strcmp(managed, "in-band-status") == 0) {
-                       /* status is zeroed, namely its .link member */
-                       phy = fixed_phy_register(PHY_POLL, &status, -1, np);
-                       return PTR_ERR_OR_ZERO(phy);
-               }
+       if (of_property_read_string(np, "managed", &managed) == 0 &&
+           strcmp(managed, "in-band-status") == 0) {
+               /* status is zeroed, namely its .link member */
+               goto register_phy;
        }
 
        /* New binding */
@@ -454,8 +451,7 @@ int of_phy_register_fixed_link(struct device_node *np)
                if (link_gpio == -EPROBE_DEFER)
                        return -EPROBE_DEFER;
 
-               phy = fixed_phy_register(PHY_POLL, &status, link_gpio, np);
-               return PTR_ERR_OR_ZERO(phy);
+               goto register_phy;
        }
 
        /* Old binding */
@@ -466,11 +462,14 @@ int of_phy_register_fixed_link(struct device_node *np)
                status.speed  = fixed_link_prop[2];
                status.pause  = fixed_link_prop[3];
                status.asym_pause = fixed_link_prop[4];
-               phy = fixed_phy_register(PHY_POLL, &status, -1, np);
-               return PTR_ERR_OR_ZERO(phy);
+               goto register_phy;
        }
 
        return -ENODEV;
+
+register_phy:
+       return PTR_ERR_OR_ZERO(fixed_phy_register(PHY_POLL, &status, link_gpio,
+                                                 np));
 }
 EXPORT_SYMBOL(of_phy_register_fixed_link);