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 /drivers | |
| 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>
Diffstat (limited to 'drivers')
| -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", |