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 04ab62251fe3..cd91070b5467 100644
--- a/net/dsa/dsa2.c
+++ b/net/dsa/dsa2.c
@@ -587,8 +587,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 i, err;
 
@@ -596,6 +596,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.9.3

Reply via email to