Re: [dpdk-dev] [EXT] Re: [PATCH] config: update Marvell ARMADA

[Liron Himi <lironh@marvell.com>]

Regards,
Liron

-----Original Message-----
From: Jerin Jacob <jerinjacobk@gmail.com> 
Sent: Friday, 29 November 2019 16:05
To: Liron Himi <lironh@marvell.com>
Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; dpdk-dev <dev@dpdk.org>; dpdk stable <stable@dpdk.org>
Subject: [EXT] Re: [dpdk-dev] [PATCH] config: update Marvell ARMADA

External Email

----------------------------------------------------------------------
On Fri, Nov 29, 2019 at 3:55 PM <lironh@marvell.com> wrote:
>
> From: Liron Himi <lironh@marvell.com>
>
> disable more NXP modules that conflict with MUSDK

# Please share more details on the conflict.
[L.H.] both components calls of_<x> APIs so when MUSDK is compiled statically it conflicts with NXP's code.
Note that the original armada config already had some NXP flags disabled, but in recent version NXP moved the of_<x> code to be depends on 'CONFIG_RTE_LIBRTE_COMMON_DPAAX' so needed to update it.
# What about meson build? "make" will be deprecated soon.
[L.H.] only when compiling the MUSDK as static LIBs, we face this issue. In meson we need to compile MUSDK as shared LIBS.
# This scheme won't work for distro build, Please spend the effort to analyze the conflict and fix the conflict. IMO, That would be the correct solution.


>
> Signed-off-by: Liron Himi <lironh@marvell.com>
> ---
>  config/defconfig_arm64-armada-linuxapp-gcc | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>
> diff --git a/config/defconfig_arm64-armada-linuxapp-gcc 
> b/config/defconfig_arm64-armada-linuxapp-gcc
> index 059180284..c09751cf0 100644
> --- a/config/defconfig_arm64-armada-linuxapp-gcc
> +++ b/config/defconfig_arm64-armada-linuxapp-gcc
> @@ -19,6 +19,23 @@ CONFIG_RTE_LIBRTE_PMD_MVSAM_CRYPTO=y
>
>  # Disable NXP as it is conflict with MUSDK  
> CONFIG_RTE_LIBRTE_DPAA_BUS=n
> +CONFIG_RTE_LIBRTE_COMMON_DPAAX=n
> +CONFIG_RTE_LIBRTE_FSLMC_BUS=n
> +CONFIG_RTE_LIBRTE_DPAA2_MEMPOOL=n
> +CONFIG_RTE_LIBRTE_DPAA2_PMD=n
> +CONFIG_RTE_LIBRTE_DPAA_BUS=n
> +CONFIG_RTE_LIBRTE_DPAA_MEMPOOL=n
> +CONFIG_RTE_LIBRTE_DPAA_PMD=n
> +CONFIG_RTE_LIBRTE_PMD_DPAA_EVENTDEV=n
> +CONFIG_RTE_LIBRTE_PMD_DPAA_SEC=n
> +CONFIG_RTE_LIBRTE_PMD_CAAM_JR=n
> +CONFIG_RTE_LIBRTE_PMD_DPAA2_EVENTDEV=n
> +CONFIG_RTE_LIBRTE_PMD_DPAA2_SEC=n
> +CONFIG_RTE_LIBRTE_PMD_DPAA2_CMDIF_RAWDEV=n
> +CONFIG_RTE_LIBRTE_PMD_DPAA2_QDMA_RAWDEV=n
> +CONFIG_RTE_LIBRTE_PFE_PMD=n
> +CONFIG_RTE_LIBRTE_ENETC_PMD=n
> +
>
>  # Doesn't support NUMA
>  CONFIG_RTE_EAL_NUMA_AWARE_HUGEPAGES=n
> --
> 2.23.0
>