config/arm: add support for Marvell CN10K
Checks
Commit Message
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
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
>
<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
> >
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
> > >
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
> > > >
<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
> > > > >
new file mode 100644
@@ -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