diff mbox series

config/arm: add support for Marvell CN10K

Message ID 20210331172320.29130-1-ndabilpuram@marvell.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series config/arm: add support for Marvell CN10K |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/travis-robot success travis build: passed
ci/github-robot success github build: passed
ci/iol-abi-testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS

Commit Message

Nithin Dabilpuram March 31, 2021, 5:23 p.m. UTC 
  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.

Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
---
 config/arm/arm64_cn10k_linux_gcc | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 config/arm/arm64_cn10k_linux_gcc

Comments

Jerin Jacob March 31, 2021, 5:33 p.m. UTC | #1 
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>


> ---
>  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
>
Honnappa Nagarahalli April 1, 2021, 1:20 a.m. UTC | #2 
<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/

> 
> 
> > ---
> >  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
> >
Nithin Dabilpuram April 1, 2021, 10:09 a.m. UTC | #3 
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

> 
> > 
> > 
> > > ---
> > >  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
> > >
Nithin Dabilpuram April 1, 2021, 10:14 a.m. UTC | #4 
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
> > > >
Honnappa Nagarahalli April 1, 2021, 2:28 p.m. UTC | #5 
<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
> > > > >
diff mbox series

Patch

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