net: dsa: rzn1_a5psw: Make the read-only array offsets static const
authorColin Ian King <colin.i.king@gmail.com>
Thu, 17 Apr 2025 16:13:52 +0000 (17:13 +0100)
committerJakub Kicinski <kuba@kernel.org>
Wed, 23 Apr 2025 02:05:04 +0000 (19:05 -0700)
Don't populate the read-only array offsets on the stack at run time,
instead make it static const.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Link: https://patch.msgid.link/20250417161353.490219-1-colin.i.king@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/dsa/rzn1_a5psw.c

index 31ea8130a495ff2e72247f1fda4cec5a85aeedae..df7466d4fe8f08dc16580a60768abd062cd40dfd 100644 (file)
@@ -337,8 +337,9 @@ static void a5psw_port_rx_block_set(struct a5psw *a5psw, int port, bool block)
 static void a5psw_flooding_set_resolution(struct a5psw *a5psw, int port,
                                          bool set)
 {
-       u8 offsets[] = {A5PSW_UCAST_DEF_MASK, A5PSW_BCAST_DEF_MASK,
-                       A5PSW_MCAST_DEF_MASK};
+       static const u8 offsets[] = {
+               A5PSW_UCAST_DEF_MASK, A5PSW_BCAST_DEF_MASK, A5PSW_MCAST_DEF_MASK
+       };
        int i;
 
        for (i = 0; i < ARRAY_SIZE(offsets); i++)