[dpdk-dev] [PATCH v5 16/51] net/bnxt: add core changes for EM and EEM lookups

Ferruh Yigit ferruh.yigit at intel.com
Tue Jul 7 10:08:16 CEST 2020


On 7/3/2020 10:01 PM, Ajit Khaparde wrote:
> From: Randy Schacher <stuart.schacher at broadcom.com>
> 
> - Move External Exact and Exact Match to device module using HCAPI
>   to add and delete entries
> - Make EM active through the device interface.
> 
> Signed-off-by: Randy Schacher <stuart.schacher at broadcom.com>
> Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
> Reviewed-by: Shahaji Bhosle <shahaji.bhosle at broadcom.com>
> Reviewed-by: Ajit Khaparde <ajit.khaparde at broadcom.com>

<...>

> diff --git a/drivers/net/bnxt/tf_core/Makefile b/drivers/net/bnxt/tf_core/Makefile
> index 2c02e29e7..5ed32f12a 100644
> --- a/drivers/net/bnxt/tf_core/Makefile
> +++ b/drivers/net/bnxt/tf_core/Makefile
> @@ -24,3 +24,11 @@ SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += tf_core/tf_tbl_type.c
>  SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += tf_core/tf_tcam.c
>  SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += tf_core/tf_util.c
>  SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += tf_core/tf_rm_new.c
> +
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += tf_core/tf_core.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += tf_core/tf_project.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += tf_core/tf_device.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += tf_core/tf_identifier.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += tf_core/tf_tbl.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += tf_core/stack.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += tf_core/tf_tcam.h

Same comment here, these header files should keep PMD local and shouldn't installed.
Installing a header file as generic as "stack.h" is breaking some sample apps
but anyway I assume these are added by mistake and I will drop them.



More information about the dev mailing list