[Intel-wired-lan] [next-queue] fm10k: initialize xps at driver load

[Keller, Jacob E]

On Mon, 2015-10-26 at 14:35 -0700, ND Linux CI Server wrote:
> Greetings,
> 
> This email is automatically generated by ND's Linux Patch Testing
> framework
> based on aiaiai. I have performed some automatic testing of a patch
> (series)
> you submitted to intel-wired-lan at lists.osuosl.org
> 
> The following contains output of any tests which failed to pass, and
> might be
> the result of developer error. The tests performed include but may
> not be
> limited to checkpatch.pl, bisection testing, compilation on a default
> kernel
> config, coccinelle scripts, cppcheck, and smatch.
> 
> If you have received this email in error, or believe that aiaiai has
> detected a
> false positive, please email Jacob Keller <jacob.e.keller at intel.com>.
> 
> ---
> 
> I have tested your changes
> 
> [Intel-wired-lan] [next-queue] fm10k: initialize xps at driver load
> 
> Project: next (net-next development queue)
> 
> Configurations: intel_defconfig,x86
> 
> Tested the patch(es) on top of the following commits:
> 16eb854 bridge: Notify port device of encap header length
> 310ab40 vlan: Notify real device of encap header length
> bdb32a4 e1000e: Add ndo_enc_hdr_len
> 0596af0 net: Add ndo_enc_hdr_len to notify extra header room for
> encapsulated frames
> 
> ---------------------------------------------------------------------
> -----------
> Failed to build the base commit: 16eb854 bridge: Notify port device
> of encap header length
> Configuration: "intel_defconfig, architecture x86".
> 
> drivers/net/ethernet/intel/ixgb/ixgb_hw.c:126:6: warning: variable
> ‘icr_reg’ set but not used [-Wunused-but-set-variable]
>   u32 icr_reg;
>       ^
> drivers/net/ethernet/intel/ixgb/ixgb_hw.c: In function
> ‘ixgb_init_hw’:
> drivers/net/ethernet/intel/ixgb/ixgb_hw.c:302:6: warning: variable
> ‘ctrl_reg’ set but not used [-Wunused-but-set-variable]
>   u32 ctrl_reg;
>       ^
> drivers/net/ethernet/intel/ixgb/ixgb_hw.c: In function
> ‘ixgb_clear_hw_cntrs’:
> drivers/net/ethernet/intel/ixgb/ixgb_hw.c:977:15: warning: variable
> ‘temp_reg’ set but not used [-Wunused-but-set-variable]
>   volatile u32 temp_reg;
>                ^
> drivers/net/ethernet/intel/ixgb/ixgb_hw.c: In function
> ‘ixgb_optics_reset’:
> drivers/net/ethernet/intel/ixgb/ixgb_hw.c:1189:7: warning: variable
> ‘mdio_reg’ set but not used [-Wunused-but-set-variable]
>    u16 mdio_reg;
>        ^
> drivers/net/ethernet/intel/fm10k/fm10k_pci.c: In function
> ‘fm10k_sw_init’:
> drivers/net/ethernet/intel/fm10k/fm10k_pci.c:1755:2: error: too many
> arguments to function ‘fm10k_dcbnl_set_ops’
>   fm10k_dcbnl_set_ops(netdev);
>   ^
> In file included from
> drivers/net/ethernet/intel/fm10k/fm10k_pci.c:24:0:
> drivers/net/ethernet/intel/fm10k/fm10k.h:563:20: note: declared here
>  static inline void fm10k_dcbnl_set_ops(void) {}
>                     ^
> make[6]: *** [drivers/net/ethernet/intel/fm10k/fm10k_pci.o] Error 1
> ---------------------------------------------------------------------
> -----------
> Failed to build : d639bf5 fm10k: initialize xps at driver load
> Configuration: "intel_defconfig, architecture x86".
> 
> drivers/net/ethernet/intel/e1000/e1000_hw.c:2398:6: warning: variable
> ‘rxcw’ set but not used [-Wunused-but-set-variable]
>   u32 rxcw = 0;
>       ^
> drivers/net/ethernet/intel/e1000/e1000_hw.c: In function
> ‘e1000_clear_hw_cntrs’:
> drivers/net/ethernet/intel/e1000/e1000_hw.c:4701:15: warning:
> variable ‘temp’ set but not used [-Wunused-but-set-variable]
>   volatile u32 temp;
>                ^
> drivers/net/ethernet/intel/e1000/e1000_main.c: In function
> ‘e1000_watchdog’:
> drivers/net/ethernet/intel/e1000/e1000_main.c:2446:9: warning:
> variable ‘txb2b’ set but not used [-Wunused-but-set-variable]
>     bool txb2b = true;
>          ^
> drivers/net/ethernet/intel/igb/igb_main.c: In function
> ‘igb_request_msix’:
> drivers/net/ethernet/intel/igb/igb_main.c:949:19: warning: unused
> variable ‘hw’ [-Wunused-variable]
>   struct e1000_hw *hw = &adapter->hw;
>                    ^
> drivers/net/ethernet/intel/fm10k/fm10k_pci.c: In function
> ‘fm10k_sw_init’:
> drivers/net/ethernet/intel/fm10k/fm10k_pci.c:1769:2: error: too many
> arguments to function ‘fm10k_dcbnl_set_ops’
>   fm10k_dcbnl_set_ops(netdev);
>   ^
> In file included from
> drivers/net/ethernet/intel/fm10k/fm10k_pci.c:24:0:
> drivers/net/ethernet/intel/fm10k/fm10k.h:566:20: note: declared here
>  static inline void fm10k_dcbnl_set_ops(void) {}
>                     ^
> make[6]: *** [drivers/net/ethernet/intel/fm10k/fm10k_pci.o] Error 1
> 
> ---------------------------------------------------------------------
> -----------


Hmm, looks like something got messed up here.

Regards,
Jake