diff options
Diffstat (limited to 'drivers/net/ixgb/ixgb_param.c')
-rw-r--r-- | drivers/net/ixgb/ixgb_param.c | 31 |
1 files changed, 13 insertions, 18 deletions
diff --git a/drivers/net/ixgb/ixgb_param.c b/drivers/net/ixgb/ixgb_param.c index af35e1ddadd6..88a08f056241 100644 --- a/drivers/net/ixgb/ixgb_param.c +++ b/drivers/net/ixgb/ixgb_param.c | |||
@@ -26,6 +26,8 @@ | |||
26 | 26 | ||
27 | *******************************************************************************/ | 27 | *******************************************************************************/ |
28 | 28 | ||
29 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | ||
30 | |||
29 | #include "ixgb.h" | 31 | #include "ixgb.h" |
30 | 32 | ||
31 | /* This is the only thing that needs to be changed to adjust the | 33 | /* This is the only thing that needs to be changed to adjust the |
@@ -209,16 +211,16 @@ ixgb_validate_option(unsigned int *value, const struct ixgb_option *opt) | |||
209 | case enable_option: | 211 | case enable_option: |
210 | switch (*value) { | 212 | switch (*value) { |
211 | case OPTION_ENABLED: | 213 | case OPTION_ENABLED: |
212 | printk(KERN_INFO "%s Enabled\n", opt->name); | 214 | pr_info("%s Enabled\n", opt->name); |
213 | return 0; | 215 | return 0; |
214 | case OPTION_DISABLED: | 216 | case OPTION_DISABLED: |
215 | printk(KERN_INFO "%s Disabled\n", opt->name); | 217 | pr_info("%s Disabled\n", opt->name); |
216 | return 0; | 218 | return 0; |
217 | } | 219 | } |
218 | break; | 220 | break; |
219 | case range_option: | 221 | case range_option: |
220 | if (*value >= opt->arg.r.min && *value <= opt->arg.r.max) { | 222 | if (*value >= opt->arg.r.min && *value <= opt->arg.r.max) { |
221 | printk(KERN_INFO "%s set to %i\n", opt->name, *value); | 223 | pr_info("%s set to %i\n", opt->name, *value); |
222 | return 0; | 224 | return 0; |
223 | } | 225 | } |
224 | break; | 226 | break; |
@@ -230,7 +232,7 @@ ixgb_validate_option(unsigned int *value, const struct ixgb_option *opt) | |||
230 | ent = &opt->arg.l.p[i]; | 232 | ent = &opt->arg.l.p[i]; |
231 | if (*value == ent->i) { | 233 | if (*value == ent->i) { |
232 | if (ent->str[0] != '\0') | 234 | if (ent->str[0] != '\0') |
233 | printk(KERN_INFO "%s\n", ent->str); | 235 | pr_info("%s\n", ent->str); |
234 | return 0; | 236 | return 0; |
235 | } | 237 | } |
236 | } | 238 | } |
@@ -240,8 +242,7 @@ ixgb_validate_option(unsigned int *value, const struct ixgb_option *opt) | |||
240 | BUG(); | 242 | BUG(); |
241 | } | 243 | } |
242 | 244 | ||
243 | printk(KERN_INFO "Invalid %s specified (%i) %s\n", | 245 | pr_info("Invalid %s specified (%i) %s\n", opt->name, *value, opt->err); |
244 | opt->name, *value, opt->err); | ||
245 | *value = opt->def; | 246 | *value = opt->def; |
246 | return -1; | 247 | return -1; |
247 | } | 248 | } |
@@ -261,9 +262,8 @@ ixgb_check_options(struct ixgb_adapter *adapter) | |||
261 | { | 262 | { |
262 | int bd = adapter->bd_number; | 263 | int bd = adapter->bd_number; |
263 | if (bd >= IXGB_MAX_NIC) { | 264 | if (bd >= IXGB_MAX_NIC) { |
264 | printk(KERN_NOTICE | 265 | pr_notice("Warning: no configuration for board #%i\n", bd); |
265 | "Warning: no configuration for board #%i\n", bd); | 266 | pr_notice("Using defaults for all values\n"); |
266 | printk(KERN_NOTICE "Using defaults for all values\n"); | ||
267 | } | 267 | } |
268 | 268 | ||
269 | { /* Transmit Descriptor Count */ | 269 | { /* Transmit Descriptor Count */ |
@@ -363,8 +363,7 @@ ixgb_check_options(struct ixgb_adapter *adapter) | |||
363 | adapter->hw.fc.high_water = opt.def; | 363 | adapter->hw.fc.high_water = opt.def; |
364 | } | 364 | } |
365 | if (!(adapter->hw.fc.type & ixgb_fc_tx_pause) ) | 365 | if (!(adapter->hw.fc.type & ixgb_fc_tx_pause) ) |
366 | printk(KERN_INFO | 366 | pr_info("Ignoring RxFCHighThresh when no RxFC\n"); |
367 | "Ignoring RxFCHighThresh when no RxFC\n"); | ||
368 | } | 367 | } |
369 | { /* Receive Flow Control Low Threshold */ | 368 | { /* Receive Flow Control Low Threshold */ |
370 | const struct ixgb_option opt = { | 369 | const struct ixgb_option opt = { |
@@ -383,8 +382,7 @@ ixgb_check_options(struct ixgb_adapter *adapter) | |||
383 | adapter->hw.fc.low_water = opt.def; | 382 | adapter->hw.fc.low_water = opt.def; |
384 | } | 383 | } |
385 | if (!(adapter->hw.fc.type & ixgb_fc_tx_pause) ) | 384 | if (!(adapter->hw.fc.type & ixgb_fc_tx_pause) ) |
386 | printk(KERN_INFO | 385 | pr_info("Ignoring RxFCLowThresh when no RxFC\n"); |
387 | "Ignoring RxFCLowThresh when no RxFC\n"); | ||
388 | } | 386 | } |
389 | { /* Flow Control Pause Time Request*/ | 387 | { /* Flow Control Pause Time Request*/ |
390 | const struct ixgb_option opt = { | 388 | const struct ixgb_option opt = { |
@@ -404,17 +402,14 @@ ixgb_check_options(struct ixgb_adapter *adapter) | |||
404 | adapter->hw.fc.pause_time = opt.def; | 402 | adapter->hw.fc.pause_time = opt.def; |
405 | } | 403 | } |
406 | if (!(adapter->hw.fc.type & ixgb_fc_tx_pause) ) | 404 | if (!(adapter->hw.fc.type & ixgb_fc_tx_pause) ) |
407 | printk(KERN_INFO | 405 | pr_info("Ignoring FCReqTimeout when no RxFC\n"); |
408 | "Ignoring FCReqTimeout when no RxFC\n"); | ||
409 | } | 406 | } |
410 | /* high low and spacing check for rx flow control thresholds */ | 407 | /* high low and spacing check for rx flow control thresholds */ |
411 | if (adapter->hw.fc.type & ixgb_fc_tx_pause) { | 408 | if (adapter->hw.fc.type & ixgb_fc_tx_pause) { |
412 | /* high must be greater than low */ | 409 | /* high must be greater than low */ |
413 | if (adapter->hw.fc.high_water < (adapter->hw.fc.low_water + 8)) { | 410 | if (adapter->hw.fc.high_water < (adapter->hw.fc.low_water + 8)) { |
414 | /* set defaults */ | 411 | /* set defaults */ |
415 | printk(KERN_INFO | 412 | pr_info("RxFCHighThresh must be >= (RxFCLowThresh + 8), Using Defaults\n"); |
416 | "RxFCHighThresh must be >= (RxFCLowThresh + 8), " | ||
417 | "Using Defaults\n"); | ||
418 | adapter->hw.fc.high_water = DEFAULT_FCRTH; | 413 | adapter->hw.fc.high_water = DEFAULT_FCRTH; |
419 | adapter->hw.fc.low_water = DEFAULT_FCRTL; | 414 | adapter->hw.fc.low_water = DEFAULT_FCRTL; |
420 | } | 415 | } |