net: dsa: mv88e6xxx: make const read-only array lanes static

ColinIanKing · davem330 · commit ccd663caffc7 · 2023-09-20T11:51:14.000+01:00
Don't populate the const read-only array lanes on the stack, instead make
it static.

Signed-off-by: Colin Ian King &lt;colin.i.king@gmail.com&gt;
Reviewed-by: Andrew Lunn &lt;andrew@lunn.ch&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git a/drivers/net/dsa/mv88e6xxx/pcs-639x.c b/drivers/net/dsa/mv88e6xxx/pcs-639x.c
@@ -208,7 +208,7 @@ static void mv88e639x_sgmii_pcs_pre_config(struct phylink_pcs *pcs,
 
 static int mv88e6390_erratum_3_14(struct mv88e639x_pcs *mpcs)
 {
-	const int lanes[] = { MV88E6390_PORT9_LANE0, MV88E6390_PORT9_LANE1,
+	static const int lanes[] = { MV88E6390_PORT9_LANE0, MV88E6390_PORT9_LANE1,
 		MV88E6390_PORT9_LANE2, MV88E6390_PORT9_LANE3,
 		MV88E6390_PORT10_LANE0, MV88E6390_PORT10_LANE1,
 		MV88E6390_PORT10_LANE2, MV88E6390_PORT10_LANE3 };

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,7 @@ static void mv88e639x_sgmii_pcs_pre_config(struct phylink_pcs *pcs,`
`208`	`208`
`209`	`209`	`static int mv88e6390_erratum_3_14(struct mv88e639x_pcs *mpcs)`
`210`	`210`	`{`
`211`		`- const int lanes[] = { MV88E6390_PORT9_LANE0, MV88E6390_PORT9_LANE1,`
	`211`	`+ static const int lanes[] = { MV88E6390_PORT9_LANE0, MV88E6390_PORT9_LANE1,`
`212`	`212`	`MV88E6390_PORT9_LANE2, MV88E6390_PORT9_LANE3,`
`213`	`213`	`MV88E6390_PORT10_LANE0, MV88E6390_PORT10_LANE1,`
`214`	`214`	`MV88E6390_PORT10_LANE2, MV88E6390_PORT10_LANE3 };`