Re: [dpdk-dev] [PATCH] config/arm: add support for Marvell CN10K

[Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>]

<snip>

> > > > Subject: Re: [dpdk-dev] [PATCH] config/arm: add support for
> > > > Marvell CN10K
> > > >
> > > > On Wed, Mar 31, 2021 at 10:53 PM Nithin Dabilpuram
> > > > <ndabilpuram@marvell.com> wrote:
> > > > >
> > > > > From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> > > > >
> > > > > Add config support to cross compile for Marvell CN10K SoC.
> > > > > Marvell CN10K SoC is based on ARM Neoverse N2 cores.
> > > >
> > > > cores->core
> > > >
> > > > >
> > > > > Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
> > > > > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> > > >
> > > > With the above change:
> > > > Acked-by: Jerin Jacob <jerinj@marvell.com>
> > > I prefer this patch to be based on top of [1].
> > >
> > > [1]
> > > http://patches.dpdk.org/project/dpdk/patch/1617186371-18699-2-git-se
> > > nd-email-juraj.linkes@pantheon.tech/
> >
> > There is no conflicting change I believe for me to take care of. The
> > change for enable
> 
> Sorry, for incomplete message.
> 
> I think there is no conflicting change here. I hope my understanding is correct.
> Since there is no dependency b/w the above patch and this one, I leave it to
> the maintainers to decide the merge order.
Agree, there is no conflicting change. However, merging the above mentioned patch would require further enhancements to your patch. The above mentioned patch is pending since some time.

> 
> 
> >
> > >
> > > >
> > > >
> > > > > ---
> > > > >  config/arm/arm64_cn10k_linux_gcc | 20 ++++++++++++++++++++
> > > > >  1 file changed, 20 insertions(+)  create mode 100644
> > > > > config/arm/arm64_cn10k_linux_gcc
> > > > >
> > > > > diff --git a/config/arm/arm64_cn10k_linux_gcc
> > > > > b/config/arm/arm64_cn10k_linux_gcc
> > > > > new file mode 100644
> > > > > index 0000000..4f8e7cb
> > > > > --- /dev/null
> > > > > +++ b/config/arm/arm64_cn10k_linux_gcc
> > > > > @@ -0,0 +1,20 @@
> > > > > +[binaries]
> > > > > +c = 'aarch64-linux-gnu-gcc'
> > > > > +cpp = 'aarch64-linux-gnu-cpp'
> > > > > +ar = 'aarch64-linux-gnu-gcc-ar'
> > > > > +strip = 'aarch64-linux-gnu-strip'
> > > > > +pkgconfig = 'aarch64-linux-gnu-pkg-config'
> > > > > +pcap-config = ''
> > > > > +
> > > > > +[host_machine]
> > > > > +system = 'linux'
> > > > > +cpu_family = 'aarch64'
> > > > > +cpu = 'armv8.6-a'
> > > > > +endian = 'little'
> > > > > +
> > > > > +[properties]
> > > > > +implementer_id = '0x41'
> > > > > +part_number = '0xd49'
> > > > > +max_lcores = 36
> > > > > +max_numa_nodes = 1
> > > > > +numa = false
> > > > > --
> > > > > 2.8.4
> > > > >