Commit 6f59d660 authored by Jesse Brandeburg's avatar Jesse Brandeburg Committed by David S. Miller

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: default avatarJesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent cd58950a
...@@ -248,7 +248,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -248,7 +248,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
{ /* Transmit Interrupt Delay */ { /* Transmit Interrupt Delay */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = range_option, .type = range_option,
.name = "Transmit Interrupt Delay", .name = "Transmit Interrupt Delay",
.err = "using default of " .err = "using default of "
...@@ -267,7 +267,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -267,7 +267,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
} }
{ /* Transmit Absolute Interrupt Delay */ { /* Transmit Absolute Interrupt Delay */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = range_option, .type = range_option,
.name = "Transmit Absolute Interrupt Delay", .name = "Transmit Absolute Interrupt Delay",
.err = "using default of " .err = "using default of "
...@@ -305,7 +305,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -305,7 +305,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
} }
{ /* Receive Absolute Interrupt Delay */ { /* Receive Absolute Interrupt Delay */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = range_option, .type = range_option,
.name = "Receive Absolute Interrupt Delay", .name = "Receive Absolute Interrupt Delay",
.err = "using default of " .err = "using default of "
...@@ -324,7 +324,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -324,7 +324,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
} }
{ /* Interrupt Throttling Rate */ { /* Interrupt Throttling Rate */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = range_option, .type = range_option,
.name = "Interrupt Throttling Rate (ints/sec)", .name = "Interrupt Throttling Rate (ints/sec)",
.err = "using default of " .err = "using default of "
...@@ -399,7 +399,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -399,7 +399,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
} }
{ /* Smart Power Down */ { /* Smart Power Down */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = enable_option, .type = enable_option,
.name = "PHY Smart Power Down", .name = "PHY Smart Power Down",
.err = "defaulting to Disabled", .err = "defaulting to Disabled",
...@@ -415,7 +415,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -415,7 +415,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
} }
{ /* CRC Stripping */ { /* CRC Stripping */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = enable_option, .type = enable_option,
.name = "CRC Stripping", .name = "CRC Stripping",
.err = "defaulting to enabled", .err = "defaulting to enabled",
...@@ -432,7 +432,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -432,7 +432,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
} }
{ /* Kumeran Lock Loss Workaround */ { /* Kumeran Lock Loss Workaround */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = enable_option, .type = enable_option,
.name = "Kumeran Lock Loss Workaround", .name = "Kumeran Lock Loss Workaround",
.err = "defaulting to Enabled", .err = "defaulting to Enabled",
...@@ -452,7 +452,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter) ...@@ -452,7 +452,7 @@ void __devinit e1000e_check_options(struct e1000_adapter *adapter)
} }
} }
{ /* Write-protect NVM */ { /* Write-protect NVM */
const struct e1000_option opt = { static const struct e1000_option opt = {
.type = enable_option, .type = enable_option,
.name = "Write-protect NVM", .name = "Write-protect NVM",
.err = "defaulting to Enabled", .err = "defaulting to Enabled",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment