> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zh...@intel.com>
> Sent: Monday, May 10, 2021 11:23
> To: Yang, Qiming <qiming.y...@intel.com>
> Cc: dev@dpdk.org; Zhang, Qi Z <qi.z.zh...@intel.com>
> Subject: [PATCH] net/ice/base: remove unnecessary code
>
> Coverty issue: 370613
> Execution cannot reach this statement: "break;".
> Remove the unnecessary if branch.
>
> Fixes: 2913bc4155d2 ("net/ice/base: sign external device package
> programming")
>
> Signed-off-by: Qi Zhang <qi.z.zh...@intel.com>
> ---
> drivers/net/ice/base/ice_common.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/net/ice/base/ice_common.c
> b/drivers/net/ice/base/ice_common.c
> index ac412a1aa7..cf0a7d4e7f 100644
> --- a/drivers/net/ice/base/ice_common.c
> +++ b/drivers/net/ice/base/ice_common.c
> @@ -2009,9 +2009,6 @@ ice_parse_common_caps(struct ice_hw *hw, struct
> ice_hw_common_caps *caps,
> {
> u8 index = cap - ICE_AQC_CAPS_EXT_TOPO_DEV_IMG0;
>
> - if (index >= ICE_EXT_TOPO_DEV_IMG_COUNT)
> - break;
> -
> caps->ext_topo_dev_img_ver_high[index] = number;
> caps->ext_topo_dev_img_ver_low[index] = logical_id;
> caps->ext_topo_dev_img_part_num[index] =
> --
> 2.26.2
Acked-by: Qiming Yang <qiming.y...@intel.com>
