Re: [PATCH] igb: mark igb_rxnfc_write_vlan_prio_filter() static
On Monday 26 September 2016, Jeff Kirsher wrote: > On Sun, 2016-09-25 at 14:05 +0800, Baoyou Xie wrote: > > We get 1 warning when building kernel with W=1: > > drivers/net/ethernet/intel/igb/igb_ethtool.c:2707:5: warning: no previous > > prototype for 'igb_rxnfc_write_vlan_prio_filter' [-Wmissing-prototypes] > > > > In fact, this function is only used in the file in which it is > > declared and don't need a declaration, but can be made static. > > so this patch marks this function with 'static'. > > > > Signed-off-by: Baoyou Xie> > --- > > drivers/net/ethernet/intel/igb/igb_ethtool.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > This has been fixed as well, and there is already a patch. Baoyou, I've seen you do this on other patches as well, please make sure it doesn't happen again, and keep track of which patches you have already sent out. It can sometimes happen that a patch gets dropped by accident, but none of the changes you sent so far are important enough to warrant resending for the same kernel release. If a patch got posted multiple weeks before the merge window and still has not made it into linux-next by the time the following -rc2 release is out, you can resend it, but then it should be marked in the subject line, e.g. [PATCH RESEND], and an explanation below the '---' line why it got resent. Arnd
Re: [PATCH] igb: mark igb_rxnfc_write_vlan_prio_filter() static
On Sun, 2016-09-25 at 14:05 +0800, Baoyou Xie wrote: > We get 1 warning when building kernel with W=1: > drivers/net/ethernet/intel/igb/igb_ethtool.c:2707:5: warning: no previous > prototype for 'igb_rxnfc_write_vlan_prio_filter' [-Wmissing-prototypes] > > In fact, this function is only used in the file in which it is > declared and don't need a declaration, but can be made static. > so this patch marks this function with 'static'. > > Signed-off-by: Baoyou Xie> --- > drivers/net/ethernet/intel/igb/igb_ethtool.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) This has been fixed as well, and there is already a patch. signature.asc Description: This is a digitally signed message part
[PATCH] igb: mark igb_rxnfc_write_vlan_prio_filter() static
We get 1 warning when building kernel with W=1: drivers/net/ethernet/intel/igb/igb_ethtool.c:2707:5: warning: no previous prototype for 'igb_rxnfc_write_vlan_prio_filter' [-Wmissing-prototypes] In fact, this function is only used in the file in which it is declared and don't need a declaration, but can be made static. so this patch marks this function with 'static'. Signed-off-by: Baoyou Xie--- drivers/net/ethernet/intel/igb/igb_ethtool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel/igb/igb_ethtool.c b/drivers/net/ethernet/intel/igb/igb_ethtool.c index 0c33eca..737b664 100644 --- a/drivers/net/ethernet/intel/igb/igb_ethtool.c +++ b/drivers/net/ethernet/intel/igb/igb_ethtool.c @@ -2704,8 +2704,8 @@ static int igb_rxnfc_write_etype_filter(struct igb_adapter *adapter, return 0; } -int igb_rxnfc_write_vlan_prio_filter(struct igb_adapter *adapter, -struct igb_nfc_filter *input) +static int igb_rxnfc_write_vlan_prio_filter(struct igb_adapter *adapter, + struct igb_nfc_filter *input) { struct e1000_hw *hw = >hw; u8 vlan_priority; -- 2.7.4
Re: [PATCH] igb: mark igb_rxnfc_write_vlan_prio_filter() static
On Sun, 2016-09-18 at 16:50 +0800, Baoyou Xie wrote: > We get 1 warning when building kernel with W=1: > drivers/net/ethernet/intel/igb/igb_ethtool.c:2707:5: warning: no previous > prototype for 'igb_rxnfc_write_vlan_prio_filter' [-Wmissing-prototypes] > > In fact, this function is only used in the file in which it is > declared and don't need a declaration, but can be made static. > so this patch marks this function with 'static'. > > Signed-off-by: Baoyou Xie> --- > drivers/net/ethernet/intel/igb/igb_ethtool.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Already fixed, see http://patchwork.ozlabs.org/patch/661904/ signature.asc Description: This is a digitally signed message part
[PATCH] igb: mark igb_rxnfc_write_vlan_prio_filter() static
We get 1 warning when building kernel with W=1: drivers/net/ethernet/intel/igb/igb_ethtool.c:2707:5: warning: no previous prototype for 'igb_rxnfc_write_vlan_prio_filter' [-Wmissing-prototypes] In fact, this function is only used in the file in which it is declared and don't need a declaration, but can be made static. so this patch marks this function with 'static'. Signed-off-by: Baoyou Xie--- drivers/net/ethernet/intel/igb/igb_ethtool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel/igb/igb_ethtool.c b/drivers/net/ethernet/intel/igb/igb_ethtool.c index 0c33eca..737b664 100644 --- a/drivers/net/ethernet/intel/igb/igb_ethtool.c +++ b/drivers/net/ethernet/intel/igb/igb_ethtool.c @@ -2704,8 +2704,8 @@ static int igb_rxnfc_write_etype_filter(struct igb_adapter *adapter, return 0; } -int igb_rxnfc_write_vlan_prio_filter(struct igb_adapter *adapter, -struct igb_nfc_filter *input) +static int igb_rxnfc_write_vlan_prio_filter(struct igb_adapter *adapter, + struct igb_nfc_filter *input) { struct e1000_hw *hw = >hw; u8 vlan_priority; -- 2.7.4