Skip to content

Commit cf93c94

Browse files
Uwe Kleine-Königdavem330
Uwe Kleine-König
authored and
davem330
committed
net/phy: fix many "defined but unused" warnings
MODULE_DEVICE_TABLE only expands to something if it's compiled for a module. So when building-in support for the phys, the mdio_device_id tables are unused. Marking them with __maybe_unused fixes the following warnings: drivers/net/phy/bcm63xx.c:134: warning: 'bcm63xx_tbl' defined but not used drivers/net/phy/broadcom.c:933: warning: 'broadcom_tbl' defined but not used drivers/net/phy/cicada.c:162: warning: 'cicada_tbl' defined but not used drivers/net/phy/davicom.c:222: warning: 'davicom_tbl' defined but not used drivers/net/phy/et1011c.c:114: warning: 'et1011c_tbl' defined but not used drivers/net/phy/icplus.c:137: warning: 'icplus_tbl' defined but not used drivers/net/phy/lxt.c:226: warning: 'lxt_tbl' defined but not used drivers/net/phy/marvell.c:724: warning: 'marvell_tbl' defined but not used drivers/net/phy/micrel.c:234: warning: 'micrel_tbl' defined but not used drivers/net/phy/national.c:154: warning: 'ns_tbl' defined but not used drivers/net/phy/qsemi.c:141: warning: 'qs6612_tbl' defined but not used drivers/net/phy/realtek.c:82: warning: 'realtek_tbl' defined but not used drivers/net/phy/smsc.c:257: warning: 'smsc_tbl' defined but not used drivers/net/phy/ste10Xp.c:135: warning: 'ste10Xp_tbl' defined but not used drivers/net/phy/vitesse.c:195: warning: 'vitesse_tbl' defined but not used Signed-off-by: Uwe Kleine-König <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 29fa060 commit cf93c94

15 files changed

+15
-15
lines changed

‎drivers/net/phy/bcm63xx.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ static void __exit bcm63xx_phy_exit(void)
131131
module_init(bcm63xx_phy_init);
132132
module_exit(bcm63xx_phy_exit);
133133

134-
static struct mdio_device_id bcm63xx_tbl[] = {
134+
static struct mdio_device_id __maybe_unused bcm63xx_tbl[] = {
135135
{ 0x00406000, 0xfffffc00 },
136136
{ 0x002bdc00, 0xfffffc00 },
137137
{ }

‎drivers/net/phy/broadcom.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -930,7 +930,7 @@ static void __exit broadcom_exit(void)
930930
module_init(broadcom_init);
931931
module_exit(broadcom_exit);
932932

933-
static struct mdio_device_id broadcom_tbl[] = {
933+
static struct mdio_device_id __maybe_unused broadcom_tbl[] = {
934934
{ PHY_ID_BCM5411, 0xfffffff0 },
935935
{ PHY_ID_BCM5421, 0xfffffff0 },
936936
{ PHY_ID_BCM5461, 0xfffffff0 },

‎drivers/net/phy/cicada.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ static void __exit cicada_exit(void)
159159
module_init(cicada_init);
160160
module_exit(cicada_exit);
161161

162-
static struct mdio_device_id cicada_tbl[] = {
162+
static struct mdio_device_id __maybe_unused cicada_tbl[] = {
163163
{ 0x000fc410, 0x000ffff0 },
164164
{ 0x000fc440, 0x000fffc0 },
165165
{ }

‎drivers/net/phy/davicom.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ static void __exit davicom_exit(void)
219219
module_init(davicom_init);
220220
module_exit(davicom_exit);
221221

222-
static struct mdio_device_id davicom_tbl[] = {
222+
static struct mdio_device_id __maybe_unused davicom_tbl[] = {
223223
{ 0x0181b880, 0x0ffffff0 },
224224
{ 0x0181b8a0, 0x0ffffff0 },
225225
{ 0x00181b80, 0x0ffffff0 },

‎drivers/net/phy/et1011c.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ static void __exit et1011c_exit(void)
111111
module_init(et1011c_init);
112112
module_exit(et1011c_exit);
113113

114-
static struct mdio_device_id et1011c_tbl[] = {
114+
static struct mdio_device_id __maybe_unused et1011c_tbl[] = {
115115
{ 0x0282f014, 0xfffffff0 },
116116
{ }
117117
};

‎drivers/net/phy/icplus.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ static void __exit ip175c_exit(void)
134134
module_init(ip175c_init);
135135
module_exit(ip175c_exit);
136136

137-
static struct mdio_device_id icplus_tbl[] = {
137+
static struct mdio_device_id __maybe_unused icplus_tbl[] = {
138138
{ 0x02430d80, 0x0ffffff0 },
139139
{ }
140140
};

‎drivers/net/phy/lxt.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,7 @@ static void __exit lxt_exit(void)
223223
module_init(lxt_init);
224224
module_exit(lxt_exit);
225225

226-
static struct mdio_device_id lxt_tbl[] = {
226+
static struct mdio_device_id __maybe_unused lxt_tbl[] = {
227227
{ 0x78100000, 0xfffffff0 },
228228
{ 0x001378e0, 0xfffffff0 },
229229
{ 0x00137a10, 0xfffffff0 },

‎drivers/net/phy/marvell.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -721,7 +721,7 @@ static void __exit marvell_exit(void)
721721
module_init(marvell_init);
722722
module_exit(marvell_exit);
723723

724-
static struct mdio_device_id marvell_tbl[] = {
724+
static struct mdio_device_id __maybe_unused marvell_tbl[] = {
725725
{ 0x01410c60, 0xfffffff0 },
726726
{ 0x01410c90, 0xfffffff0 },
727727
{ 0x01410cc0, 0xfffffff0 },

‎drivers/net/phy/micrel.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ MODULE_DESCRIPTION("Micrel PHY driver");
231231
MODULE_AUTHOR("David J. Choi");
232232
MODULE_LICENSE("GPL");
233233

234-
static struct mdio_device_id micrel_tbl[] = {
234+
static struct mdio_device_id __maybe_unused micrel_tbl[] = {
235235
{ PHY_ID_KSZ9021, 0x000fff10 },
236236
{ PHY_ID_KS8001, 0x00fffff0 },
237237
{ PHY_ID_KS8737, 0x00fffff0 },

‎drivers/net/phy/national.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ MODULE_LICENSE("GPL");
151151
module_init(ns_init);
152152
module_exit(ns_exit);
153153

154-
static struct mdio_device_id ns_tbl[] = {
154+
static struct mdio_device_id __maybe_unused ns_tbl[] = {
155155
{ DP83865_PHY_ID, 0xfffffff0 },
156156
{ }
157157
};

‎drivers/net/phy/qsemi.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ static void __exit qs6612_exit(void)
138138
module_init(qs6612_init);
139139
module_exit(qs6612_exit);
140140

141-
static struct mdio_device_id qs6612_tbl[] = {
141+
static struct mdio_device_id __maybe_unused qs6612_tbl[] = {
142142
{ 0x00181440, 0xfffffff0 },
143143
{ }
144144
};

‎drivers/net/phy/realtek.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ static void __exit realtek_exit(void)
7979
module_init(realtek_init);
8080
module_exit(realtek_exit);
8181

82-
static struct mdio_device_id realtek_tbl[] = {
82+
static struct mdio_device_id __maybe_unused realtek_tbl[] = {
8383
{ 0x001cc912, 0x001fffff },
8484
{ }
8585
};

‎drivers/net/phy/smsc.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,7 @@ MODULE_LICENSE("GPL");
254254
module_init(smsc_init);
255255
module_exit(smsc_exit);
256256

257-
static struct mdio_device_id smsc_tbl[] = {
257+
static struct mdio_device_id __maybe_unused smsc_tbl[] = {
258258
{ 0x0007c0a0, 0xfffffff0 },
259259
{ 0x0007c0b0, 0xfffffff0 },
260260
{ 0x0007c0c0, 0xfffffff0 },

‎drivers/net/phy/ste10Xp.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ static void __exit ste10Xp_exit(void)
132132
module_init(ste10Xp_init);
133133
module_exit(ste10Xp_exit);
134134

135-
static struct mdio_device_id ste10Xp_tbl[] = {
135+
static struct mdio_device_id __maybe_unused ste10Xp_tbl[] = {
136136
{ STE101P_PHY_ID, 0xfffffff0 },
137137
{ STE100P_PHY_ID, 0xffffffff },
138138
{ }

‎drivers/net/phy/vitesse.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ static void __exit vsc82xx_exit(void)
192192
module_init(vsc82xx_init);
193193
module_exit(vsc82xx_exit);
194194

195-
static struct mdio_device_id vitesse_tbl[] = {
195+
static struct mdio_device_id __maybe_unused vitesse_tbl[] = {
196196
{ PHY_ID_VSC8244, 0x000fffc0 },
197197
{ PHY_ID_VSC8221, 0x000ffff0 },
198198
{ }

0 commit comments

Comments
 (0)