[Intel-wired-lan] [next-queue] fm10k: add missing field initializers to TLV attributes)

Keller, Jacob E jacob.e.keller at intel.com
Tue Feb 19 22:27:41 UTC 2019


> -----Original Message-----
> From: Keller, Jacob E
> Sent: Tuesday, February 19, 2019 9:17 AM
> To: Intel Wired LAN <intel-wired-lan at lists.osuosl.org>
> Cc: Kirsher, Jeffrey T <jeffrey.t.kirsher at intel.com>; Keller, Jacob E
> <jacob.e.keller at intel.com>; nosbuild <nosbuild at intel.com>
> Subject: [next-queue] fm10k: add missing field initializers to TLV attributes)
> 
> From: "nosbuild (Code Review)" <nosbuild at intel.com>
> 

Not quite sure how the authorship got screwed up here. I've sent a v2 with this corrected.

Thanks,
Jake

> Add the missing field initializers for a couple of the TLV attribute
> macros. This resolves the last few -Wmissing-field-initializers warnings
> for the fm10k Linux driver.
> 
> Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
> ---
>  drivers/net/ethernet/intel/fm10k/fm10k_tlv.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/ethernet/intel/fm10k/fm10k_tlv.h
> b/drivers/net/ethernet/intel/fm10k/fm10k_tlv.h
> index 160bc5b78f99..ceb9b791f799 100644
> --- a/drivers/net/ethernet/intel/fm10k/fm10k_tlv.h
> +++ b/drivers/net/ethernet/intel/fm10k/fm10k_tlv.h
> @@ -1,5 +1,5 @@
>  /* SPDX-License-Identifier: GPL-2.0 */
> -/* Copyright(c) 2013 - 2018 Intel Corporation. */
> +/* Copyright(c) 2013 - 2019 Intel Corporation. */
> 
>  #ifndef _FM10K_TLV_H_
>  #define _FM10K_TLV_H_
> @@ -76,8 +76,8 @@ struct fm10k_tlv_attr {
>  #define FM10K_TLV_ATTR_S32(id)		    { id, FM10K_TLV_SIGNED, 4 }
>  #define FM10K_TLV_ATTR_S64(id)		    { id, FM10K_TLV_SIGNED, 8 }
>  #define FM10K_TLV_ATTR_LE_STRUCT(id, len)   { id, FM10K_TLV_LE_STRUCT, len }
> -#define FM10K_TLV_ATTR_NESTED(id)	    { id, FM10K_TLV_NESTED }
> -#define FM10K_TLV_ATTR_LAST		    { FM10K_TLV_ERROR }
> +#define FM10K_TLV_ATTR_NESTED(id)	    { id, FM10K_TLV_NESTED, 0 }
> +#define FM10K_TLV_ATTR_LAST		    { FM10K_TLV_ERROR, 0, 0 }
> 
>  struct fm10k_msg_data {
>  	unsigned int		    id;
> --
> 2.18.0.219.gaf81d287a9da



More information about the Intel-wired-lan mailing list