diff options
author | Jesse Brandeburg <jesse.brandeburg@intel.com> | 2010-04-09 06:51:09 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-04-13 05:58:25 -0400 |
commit | 6f59d6604d9361ef53a7721fb8aa1687ad6f8448 (patch) | |
tree | c22cef8d1dece5b39ef074af0998943596895865 | |
parent | cd58950a5345f006a318f178705b9250aa54425c (diff) |
e1000e: use static params to save stack space
used a modified checkstack to get the 56 number
(normally checkstack wouldn't show this low a value)
checkstack before:
0x0000012f e1000e_check_options [e1000e]: 272
after:
0x0000012f e1000e_check_options [e1000e]: 56
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/e1000e/param.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/net/e1000e/param.c b/drivers/net/e1000e/param.c index 2e399778cae5..f775a481063d 100644 --- a/drivers/net/e1000e/param.c +++ b/drivers/net/e1000e/param.c | |||
@@ -248,7 +248,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
248 | } | 248 | } |
249 | 249 | ||
250 | { /* Transmit Interrupt Delay */ | 250 | { /* Transmit Interrupt Delay */ |
251 | const struct e1000_option opt = { | 251 | static const struct e1000_option opt = { |
252 | .type = range_option, | 252 | .type = range_option, |
253 | .name = "Transmit Interrupt Delay", | 253 | .name = "Transmit Interrupt Delay", |
254 | .err = "using default of " | 254 | .err = "using default of " |
@@ -267,7 +267,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
267 | } | 267 | } |
268 | } | 268 | } |
269 | { /* Transmit Absolute Interrupt Delay */ | 269 | { /* Transmit Absolute Interrupt Delay */ |
270 | const struct e1000_option opt = { | 270 | static const struct e1000_option opt = { |
271 | .type = range_option, | 271 | .type = range_option, |
272 | .name = "Transmit Absolute Interrupt Delay", | 272 | .name = "Transmit Absolute Interrupt Delay", |
273 | .err = "using default of " | 273 | .err = "using default of " |
@@ -305,7 +305,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
305 | } | 305 | } |
306 | } | 306 | } |
307 | { /* Receive Absolute Interrupt Delay */ | 307 | { /* Receive Absolute Interrupt Delay */ |
308 | const struct e1000_option opt = { | 308 | static const struct e1000_option opt = { |
309 | .type = range_option, | 309 | .type = range_option, |
310 | .name = "Receive Absolute Interrupt Delay", | 310 | .name = "Receive Absolute Interrupt Delay", |
311 | .err = "using default of " | 311 | .err = "using default of " |
@@ -324,7 +324,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
324 | } | 324 | } |
325 | } | 325 | } |
326 | { /* Interrupt Throttling Rate */ | 326 | { /* Interrupt Throttling Rate */ |
327 | const struct e1000_option opt = { | 327 | static const struct e1000_option opt = { |
328 | .type = range_option, | 328 | .type = range_option, |
329 | .name = "Interrupt Throttling Rate (ints/sec)", | 329 | .name = "Interrupt Throttling Rate (ints/sec)", |
330 | .err = "using default of " | 330 | .err = "using default of " |
@@ -399,7 +399,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
399 | } | 399 | } |
400 | } | 400 | } |
401 | { /* Smart Power Down */ | 401 | { /* Smart Power Down */ |
402 | const struct e1000_option opt = { | 402 | static const struct e1000_option opt = { |
403 | .type = enable_option, | 403 | .type = enable_option, |
404 | .name = "PHY Smart Power Down", | 404 | .name = "PHY Smart Power Down", |
405 | .err = "defaulting to Disabled", | 405 | .err = "defaulting to Disabled", |
@@ -415,7 +415,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
415 | } | 415 | } |
416 | } | 416 | } |
417 | { /* CRC Stripping */ | 417 | { /* CRC Stripping */ |
418 | const struct e1000_option opt = { | 418 | static const struct e1000_option opt = { |
419 | .type = enable_option, | 419 | .type = enable_option, |
420 | .name = "CRC Stripping", | 420 | .name = "CRC Stripping", |
421 | .err = "defaulting to enabled", | 421 | .err = "defaulting to enabled", |
@@ -432,7 +432,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
432 | } | 432 | } |
433 | } | 433 | } |
434 | { /* Kumeran Lock Loss Workaround */ | 434 | { /* Kumeran Lock Loss Workaround */ |
435 | const struct e1000_option opt = { | 435 | static const struct e1000_option opt = { |
436 | .type = enable_option, | 436 | .type = enable_option, |
437 | .name = "Kumeran Lock Loss Workaround", | 437 | .name = "Kumeran Lock Loss Workaround", |
438 | .err = "defaulting to Enabled", | 438 | .err = "defaulting to Enabled", |
@@ -452,7 +452,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) | |||
452 | } | 452 | } |
453 | } | 453 | } |
454 | { /* Write-protect NVM */ | 454 | { /* Write-protect NVM */ |
455 | const struct e1000_option opt = { | 455 | static const struct e1000_option opt = { |
456 | .type = enable_option, | 456 | .type = enable_option, |
457 | .name = "Write-protect NVM", | 457 | .name = "Write-protect NVM", |
458 | .err = "defaulting to Enabled", | 458 | .err = "defaulting to Enabled", |