diff options
author | Mitch Williams <mitch.a.williams@intel.com> | 2014-05-10 00:49:06 -0400 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2014-06-08 05:01:34 -0400 |
commit | 80e7289356cdba946dbf1563f7d73f46d5115248 (patch) | |
tree | 57344dd85a00c5a50aab414b38ec0e4b0ec4bdab /drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c | |
parent | 635ae1e3f55f215648b086855a83ab42361af4cd (diff) |
i40evf: clean up log message formatting
Clean up inconsistent log messages, mostly related to punctuation. Based
on the dogma that "kernel messages are not sentences", remove all
trailing periods. Reword a few of the messages to make them less
sentence-like.
Change-ID: Ibd849aa7623a77549b0709988c66ab05d1311472
Signed-off-by: Mitch Williams <mitch.a.williams@intel.com>
Tested-by: Sibai Li <sibai.li@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c')
-rw-r--r-- | drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c b/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c index 13faacdfabba..cd414dac7553 100644 --- a/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c +++ b/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c | |||
@@ -388,7 +388,7 @@ void i40evf_add_ether_addrs(struct i40evf_adapter *adapter) | |||
388 | len = sizeof(struct i40e_virtchnl_ether_addr_list) + | 388 | len = sizeof(struct i40e_virtchnl_ether_addr_list) + |
389 | (count * sizeof(struct i40e_virtchnl_ether_addr)); | 389 | (count * sizeof(struct i40e_virtchnl_ether_addr)); |
390 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { | 390 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { |
391 | dev_warn(&adapter->pdev->dev, "%s: Too many MAC address changes in one request.\n", | 391 | dev_warn(&adapter->pdev->dev, "%s: Too many MAC address changes in one request\n", |
392 | __func__); | 392 | __func__); |
393 | count = (I40EVF_MAX_AQ_BUF_SIZE - | 393 | count = (I40EVF_MAX_AQ_BUF_SIZE - |
394 | sizeof(struct i40e_virtchnl_ether_addr_list)) / | 394 | sizeof(struct i40e_virtchnl_ether_addr_list)) / |
@@ -450,7 +450,7 @@ void i40evf_del_ether_addrs(struct i40evf_adapter *adapter) | |||
450 | len = sizeof(struct i40e_virtchnl_ether_addr_list) + | 450 | len = sizeof(struct i40e_virtchnl_ether_addr_list) + |
451 | (count * sizeof(struct i40e_virtchnl_ether_addr)); | 451 | (count * sizeof(struct i40e_virtchnl_ether_addr)); |
452 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { | 452 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { |
453 | dev_warn(&adapter->pdev->dev, "%s: Too many MAC address changes in one request.\n", | 453 | dev_warn(&adapter->pdev->dev, "%s: Too many MAC address changes in one request\n", |
454 | __func__); | 454 | __func__); |
455 | count = (I40EVF_MAX_AQ_BUF_SIZE - | 455 | count = (I40EVF_MAX_AQ_BUF_SIZE - |
456 | sizeof(struct i40e_virtchnl_ether_addr_list)) / | 456 | sizeof(struct i40e_virtchnl_ether_addr_list)) / |
@@ -512,7 +512,7 @@ void i40evf_add_vlans(struct i40evf_adapter *adapter) | |||
512 | len = sizeof(struct i40e_virtchnl_vlan_filter_list) + | 512 | len = sizeof(struct i40e_virtchnl_vlan_filter_list) + |
513 | (count * sizeof(u16)); | 513 | (count * sizeof(u16)); |
514 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { | 514 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { |
515 | dev_warn(&adapter->pdev->dev, "%s: Too many VLAN changes in one request.\n", | 515 | dev_warn(&adapter->pdev->dev, "%s: Too many VLAN changes in one request\n", |
516 | __func__); | 516 | __func__); |
517 | count = (I40EVF_MAX_AQ_BUF_SIZE - | 517 | count = (I40EVF_MAX_AQ_BUF_SIZE - |
518 | sizeof(struct i40e_virtchnl_vlan_filter_list)) / | 518 | sizeof(struct i40e_virtchnl_vlan_filter_list)) / |
@@ -572,7 +572,7 @@ void i40evf_del_vlans(struct i40evf_adapter *adapter) | |||
572 | len = sizeof(struct i40e_virtchnl_vlan_filter_list) + | 572 | len = sizeof(struct i40e_virtchnl_vlan_filter_list) + |
573 | (count * sizeof(u16)); | 573 | (count * sizeof(u16)); |
574 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { | 574 | if (len > I40EVF_MAX_AQ_BUF_SIZE) { |
575 | dev_warn(&adapter->pdev->dev, "%s: Too many VLAN changes in one request.\n", | 575 | dev_warn(&adapter->pdev->dev, "%s: Too many VLAN changes in one request\n", |
576 | __func__); | 576 | __func__); |
577 | count = (I40EVF_MAX_AQ_BUF_SIZE - | 577 | count = (I40EVF_MAX_AQ_BUF_SIZE - |
578 | sizeof(struct i40e_virtchnl_vlan_filter_list)) / | 578 | sizeof(struct i40e_virtchnl_vlan_filter_list)) / |
@@ -711,7 +711,7 @@ void i40evf_virtchnl_completion(struct i40evf_adapter *adapter, | |||
711 | return; | 711 | return; |
712 | } | 712 | } |
713 | if (v_opcode != adapter->current_op) { | 713 | if (v_opcode != adapter->current_op) { |
714 | dev_err(&adapter->pdev->dev, "%s: Pending op is %d, received %d.\n", | 714 | dev_err(&adapter->pdev->dev, "%s: Pending op is %d, received %d\n", |
715 | __func__, adapter->current_op, v_opcode); | 715 | __func__, adapter->current_op, v_opcode); |
716 | /* We're probably completely screwed at this point, but clear | 716 | /* We're probably completely screwed at this point, but clear |
717 | * the current op and try to carry on.... | 717 | * the current op and try to carry on.... |
@@ -720,7 +720,7 @@ void i40evf_virtchnl_completion(struct i40evf_adapter *adapter, | |||
720 | return; | 720 | return; |
721 | } | 721 | } |
722 | if (v_retval) { | 722 | if (v_retval) { |
723 | dev_err(&adapter->pdev->dev, "%s: PF returned error %d to our request %d!\n", | 723 | dev_err(&adapter->pdev->dev, "%s: PF returned error %d to our request %d\n", |
724 | __func__, v_retval, v_opcode); | 724 | __func__, v_retval, v_opcode); |
725 | } | 725 | } |
726 | switch (v_opcode) { | 726 | switch (v_opcode) { |
@@ -770,7 +770,7 @@ void i40evf_virtchnl_completion(struct i40evf_adapter *adapter, | |||
770 | adapter->aq_pending &= ~(I40EVF_FLAG_AQ_MAP_VECTORS); | 770 | adapter->aq_pending &= ~(I40EVF_FLAG_AQ_MAP_VECTORS); |
771 | break; | 771 | break; |
772 | default: | 772 | default: |
773 | dev_warn(&adapter->pdev->dev, "%s: Received unexpected message %d from PF.\n", | 773 | dev_warn(&adapter->pdev->dev, "%s: Received unexpected message %d from PF\n", |
774 | __func__, v_opcode); | 774 | __func__, v_opcode); |
775 | break; | 775 | break; |
776 | } /* switch v_opcode */ | 776 | } /* switch v_opcode */ |