[PATCH] net: dsa: mv88e6xxx: Move forward declaration to where it is needed

Subsystems: marvell 88e6xxx ethernet switch fabric driver, networking drivers, networking [dsa], the rest

STALE3497d

3 messages, 3 authors, 2017-02-08 · open the first message on its own page

[PATCH] net: dsa: mv88e6xxx: Move forward declaration to where it is needed

From: Andrew Lunn <andrew@lunn.ch>
Date: 2017-02-08 01:10:38

Move it out from the middle for the #defines to just before it is
needed.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
 drivers/net/dsa/mv88e6xxx/mv88e6xxx.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/dsa/mv88e6xxx/mv88e6xxx.h b/drivers/net/dsa/mv88e6xxx/mv88e6xxx.h
index c284e7f1f294..6713776ad26b 100644
--- a/drivers/net/dsa/mv88e6xxx/mv88e6xxx.h
+++ b/drivers/net/dsa/mv88e6xxx/mv88e6xxx.h
@@ -660,8 +660,6 @@ enum mv88e6xxx_cap {
 	 MV88E6XXX_FLAGS_MULTI_CHIP |	\
 	 MV88E6XXX_FLAGS_PVT)
 
-struct mv88e6xxx_ops;
-
 #define MV88E6XXX_FLAGS_FAMILY_6390	\
 	(MV88E6XXX_FLAG_EEE |		\
 	 MV88E6XXX_FLAG_GLOBAL2 |	\
@@ -672,6 +670,8 @@ struct mv88e6xxx_ops;
 	 MV88E6XXX_FLAGS_MULTI_CHIP |	\
 	 MV88E6XXX_FLAGS_PVT)
 
+struct mv88e6xxx_ops;
+
 struct mv88e6xxx_info {
 	enum mv88e6xxx_family family;
 	u16 prod_num;
-- 
2.11.0

Re: [PATCH] net: dsa: mv88e6xxx: Move forward declaration to where it is needed

From: Vivien Didelot <hidden>
Date: 2017-02-08 15:57:05

Hi Andrew,

Andrew Lunn [off-list ref] writes:
Move it out from the middle for the #defines to just before it is
needed.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Vivien Didelot <redacted>

Thanks,

        Vivien

Re: [PATCH] net: dsa: mv88e6xxx: Move forward declaration to where it is needed

From: David Miller <davem@davemloft.net>
Date: 2017-02-08 19:47:52

From: Andrew Lunn <andrew@lunn.ch>
Date: Wed,  8 Feb 2017 01:40:01 +0100
Move it out from the middle for the #defines to just before it is
needed.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
This looks like net-next material to me, but this patch doesn't apply there.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help