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

From: Nithin Dabilpuram <nithind1988@gmail.com>
To: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
Cc: Jerin Jacob <jerinjacobk@gmail.com>, dpdk-dev <dev@dpdk.org>,
	Jan Viktorin <viktorin@rehivetech.com>,
	Ruifeng Wang <Ruifeng.Wang@arm.com>,
	"jerinj@marvell.com" <jerinj@marvell.com>,
	Sunil Kumar Kori <skori@marvell.com>,
	Satha Koteswara Rao Kottidi <skoteshwar@marvell.com>,
	Pavan Nikhilesh <pbhagavatula@marvell.com>,
	Kiran Kumar K <kirankumark@marvell.com>,
	Satheesh Paul <psatheesh@marvell.com>,
	Ashwin Sekhar Thalakalath Kottilveetil <asekhar@marvell.com>,
	nd <nd@arm.com>, "thomas@monjalon.net" <thomas@monjalon.net>
Subject: Re: [dpdk-dev] [PATCH] config/arm: add support for Marvell CN10K
Date: Thu, 1 Apr 2021 15:44:11 +0530	[thread overview]
Message-ID: <YGWc84U+3TgVMoMc@gmail.com> (raw)
In-Reply-To: <YGWbvZuL0dPHbYgh@gmail.com>

On Thu, Apr 01, 2021 at 03:39:01PM +0530, Nithin Dabilpuram wrote:
> On Thu, Apr 01, 2021 at 01:20:44AM +0000, Honnappa Nagarahalli wrote:
> > <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-send-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.

> 
> > 
> > > 
> > > 
> > > > ---
> > > >  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
> > > >