[dpdk-dev] [PATCH v2 08/58] net/bnxt: add action SRAM translation
Thomas Monjalon
thomas at monjalon.net
Mon Jul 5 23:23:59 CEST 2021
13/06/2021 02:06, Ajit Khaparde:
> From: Farah Smith <farah.smith at broadcom.com>
>
> - Translate Truflow action types for Thor to HCAPI RM
> resource defined SRAM banks.
> - move module type enum definitions to tf_core API
> - Switch to subtype concept for RM.
> - alloc/free working for Thor SRAM table type for full AR.
>
> Signed-off-by: Farah Smith <farah.smith at broadcom.com>
> Signed-off-by: Randy Schacher <stuart.schacher at broadcom.com>
> Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
> Reviewed-by: Peter Spreadborough <peter.spreadborough at broadcom.com>
> Reviewed-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
[...]
I see 2 issues in the beginning of this patch:
> --- a/drivers/net/bnxt/hcapi/cfa/hcapi_cfa.h
> +++ b/drivers/net/bnxt/hcapi/cfa/hcapi_cfa.h
> @@ -1,281 +1,126 @@
> -/* SPDX-License-Identifier: BSD-3-Clause
> - * Copyright(c) 2019-2021 Broadcom
> - * All rights reserved.
> +/*
> + * Copyright(c) Broadcom Limited.
> + * All rights reserved.
> */
SPDX copyright is removed.
>
> +/*!
> + * \file
> + * \brief Exported functions for CFA HW programming
> + */
> #ifndef _HCAPI_CFA_H_
> #define _HCAPI_CFA_H_
>
> #include <stdio.h>
> +#include <stddef.h>
> #include <string.h>
> #include <stdbool.h>
> #include <stdint.h>
> #include <stddef.h>
> +#include <errno.h>
stddef.h is included twice.
Please fix in next-net-brcm, thanks.
More information about the dev
mailing list