diff options
author | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2010-12-01 14:59:50 -0500 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2010-12-11 01:12:32 -0500 |
commit | 667445008db3f45a760c235d771be0c9671e59e5 (patch) | |
tree | 6b6fea62bfcfeb78a6deeb851b70c8eda0700c17 /drivers/net/ixgb | |
parent | 19a0b67afd174c4db261d587b5c67704dcd53c17 (diff) |
Intel Wired LAN drivers: Use static const
Based on work by Joe Perches <joe@perches.com>
Using static const to decrease data and overall object size.
CC: Joe Perches <joe@perches.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Tested-by: Emil Tantilov <emil.s.tantilov@intel.com>
Diffstat (limited to 'drivers/net/ixgb')
-rw-r--r-- | drivers/net/ixgb/ixgb_param.c | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/drivers/net/ixgb/ixgb_param.c b/drivers/net/ixgb/ixgb_param.c index 88a08f056241..dd7fbeb1f7d1 100644 --- a/drivers/net/ixgb/ixgb_param.c +++ b/drivers/net/ixgb/ixgb_param.c | |||
@@ -191,9 +191,9 @@ struct ixgb_option { | |||
191 | } r; | 191 | } r; |
192 | struct { /* list_option info */ | 192 | struct { /* list_option info */ |
193 | int nr; | 193 | int nr; |
194 | struct ixgb_opt_list { | 194 | const struct ixgb_opt_list { |
195 | int i; | 195 | int i; |
196 | char *str; | 196 | const char *str; |
197 | } *p; | 197 | } *p; |
198 | } l; | 198 | } l; |
199 | } arg; | 199 | } arg; |
@@ -226,7 +226,7 @@ ixgb_validate_option(unsigned int *value, const struct ixgb_option *opt) | |||
226 | break; | 226 | break; |
227 | case list_option: { | 227 | case list_option: { |
228 | int i; | 228 | int i; |
229 | struct ixgb_opt_list *ent; | 229 | const struct ixgb_opt_list *ent; |
230 | 230 | ||
231 | for (i = 0; i < opt->arg.l.nr; i++) { | 231 | for (i = 0; i < opt->arg.l.nr; i++) { |
232 | ent = &opt->arg.l.p[i]; | 232 | ent = &opt->arg.l.p[i]; |
@@ -322,14 +322,15 @@ ixgb_check_options(struct ixgb_adapter *adapter) | |||
322 | } | 322 | } |
323 | { /* Flow Control */ | 323 | { /* Flow Control */ |
324 | 324 | ||
325 | struct ixgb_opt_list fc_list[] = | 325 | static const struct ixgb_opt_list fc_list[] = { |
326 | {{ ixgb_fc_none, "Flow Control Disabled" }, | 326 | { ixgb_fc_none, "Flow Control Disabled" }, |
327 | { ixgb_fc_rx_pause,"Flow Control Receive Only" }, | 327 | { ixgb_fc_rx_pause, "Flow Control Receive Only" }, |
328 | { ixgb_fc_tx_pause,"Flow Control Transmit Only" }, | 328 | { ixgb_fc_tx_pause, "Flow Control Transmit Only" }, |
329 | { ixgb_fc_full, "Flow Control Enabled" }, | 329 | { ixgb_fc_full, "Flow Control Enabled" }, |
330 | { ixgb_fc_default, "Flow Control Hardware Default" }}; | 330 | { ixgb_fc_default, "Flow Control Hardware Default" } |
331 | }; | ||
331 | 332 | ||
332 | const struct ixgb_option opt = { | 333 | static const struct ixgb_option opt = { |
333 | .type = list_option, | 334 | .type = list_option, |
334 | .name = "Flow Control", | 335 | .name = "Flow Control", |
335 | .err = "reading default settings from EEPROM", | 336 | .err = "reading default settings from EEPROM", |