Don't populate the const read-only arrays mac_addr_reg on the stack at
run time, instead make them static, this reduces the object code size.
Size before:
text data bss dec hex filename
65066 11352 0 76418 12a82 drivers/net/phy/motorcomm.o
Size after:
text data bss dec hex filename
64761 11512 0 76273 129f1 drivers/net/phy/motorcomm.o
Reducton of 145 bytes (gcc 14.2.0 x86-64)
Signed-off-by: Colin Ian King <redacted>
---
drivers/net/phy/motorcomm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/phy/motorcomm.c b/drivers/net/phy/motorcomm.c
index 0e91f5d1a4fd..aeae7ec897c6 100644
--- a/drivers/net/phy/motorcomm.c
+++ b/drivers/net/phy/motorcomm.c
@@ -536,7 +536,7 @@ static void ytphy_get_wol(struct phy_device *phydev,
static int ytphy_set_wol(struct phy_device *phydev, struct ethtool_wolinfo *wol)
{
struct net_device *p_attached_dev;
- const u16 mac_addr_reg[] = {
+ static const u16 mac_addr_reg[] = {
YTPHY_WOL_MACADDR2_REG,
YTPHY_WOL_MACADDR1_REG,
YTPHY_WOL_MACADDR0_REG,@@ -608,7 +608,7 @@ static int ytphy_set_wol(struct phy_device *phydev, struct ethtool_wolinfo *wol)
static int yt8531_set_wol(struct phy_device *phydev,
struct ethtool_wolinfo *wol)
{
- const u16 mac_addr_reg[] = {
+ static const u16 mac_addr_reg[] = {
YTPHY_WOL_MACADDR2_REG,
YTPHY_WOL_MACADDR1_REG,
YTPHY_WOL_MACADDR0_REG,--
2.50.1