Move the assignment of ports in _dsa_register_switch() closer to where
it is checked, no functional change. Re-order declarations to be
preserve the inverted christmas tree style.

Signed-off-by: Florian Fainelli <f.faine...@gmail.com>
---
 net/dsa/dsa2.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/net/dsa/dsa2.c b/net/dsa/dsa2.c
index bdde5d217326..a565bd919aa3 100644
--- a/net/dsa/dsa2.c
+++ b/net/dsa/dsa2.c
@@ -598,8 +598,8 @@ static struct device_node *dsa_get_ports(struct dsa_switch 
*ds,
 static int _dsa_register_switch(struct dsa_switch *ds, struct device *dev)
 {
        struct device_node *np = dev->of_node;
-       struct device_node *ports = dsa_get_ports(ds, np);
        struct dsa_switch_tree *dst;
+       struct device_node *ports;
        u32 tree, index;
        int err;
 
@@ -607,6 +607,7 @@ static int _dsa_register_switch(struct dsa_switch *ds, 
struct device *dev)
        if (err)
                return err;
 
+       ports = dsa_get_ports(ds, np);
        if (IS_ERR(ports))
                return PTR_ERR(ports);
 
-- 
2.7.4

Reply via email to