[dpdk-stable] [PATCH] config/arm: fix typo in meson config

patches for DPDK stable branches
 help / color / mirror / Atom feed

* [dpdk-stable] [PATCH] config/arm: fix typo in meson config
       [not found] <7a4b591b-5e32-a3f8-a8ff-59a636217814@6wind.com>
@ 2021-02-01 22:41 ` Ferruh Yigit
  2021-02-01 23:22   ` Honnappa Nagarahalli
                     ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Ferruh Yigit @ 2021-02-01 22:41 UTC (permalink / raw)
  To: Jan Viktorin, Ruifeng Wang, Jerin Jacob, Bruce Richardson,
	Juraj Linkeš,
	Honnappa Nagarahalli, Vimal Chungath
  Cc: Ferruh Yigit, dev, Thomas Monjalon, stable, Thierry Herbelot

s/implementor/implementer

Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
Fixes: c51401832147 ("config/arm: add Graviton2")
Cc: stable@dpdk.org

Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
Cc: Juraj Linkeš <juraj.linkes@pantheon.tech>
Cc: Ruifeng Wang <ruifeng.wang@arm.com>
Cc: vcchunga@amazon.com
---
 config/arm/arm64_armv8_linux_clang_ubuntu1804 | 4 ++--
 config/arm/arm64_graviton2_linux_gcc          | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/config/arm/arm64_armv8_linux_clang_ubuntu1804 b/config/arm/arm64_armv8_linux_clang_ubuntu1804
index aa5ee0132065..ae3fb30c5da8 100644
--- a/config/arm/arm64_armv8_linux_clang_ubuntu1804
+++ b/config/arm/arm64_armv8_linux_clang_ubuntu1804
@@ -14,7 +14,7 @@ cpu = 'armv8-a'
 endian = 'little'
 
 [properties]
-implementor_id = 'generic'
-implementor_pn = 'default'
+implementer_id = 'generic'
+implementer_pn = 'default'
 c_args = ['-target', 'aarch64-linux-gnu', '--sysroot', '/usr/aarch64-linux-gnu']
 c_link_args = ['-target', 'aarch64-linux-gnu', '-fuse-ld=lld', '--gcc-toolchain=/usr']
diff --git a/config/arm/arm64_graviton2_linux_gcc b/config/arm/arm64_graviton2_linux_gcc
index cfe2397975fc..123f14eed29e 100644
--- a/config/arm/arm64_graviton2_linux_gcc
+++ b/config/arm/arm64_graviton2_linux_gcc
@@ -13,8 +13,8 @@ cpu = 'armv8-a'
 endian = 'little'
 
 [properties]
-implementor_id = '0x41'
-implementor_pn = '0xd0c'
+implementer_id = '0x41'
+implementer_pn = '0xd0c'
 max_lcores = 64
 max_numa_nodes = 1
 numa = false
-- 
2.29.2


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH] config/arm: fix typo in meson config
  2021-02-01 22:41 ` [dpdk-stable] [PATCH] config/arm: fix typo in meson config Ferruh Yigit
@ 2021-02-01 23:22   ` Honnappa Nagarahalli
  2021-02-02  2:19   ` Ruifeng Wang
  2021-02-02  7:12   ` Juraj Linkeš
  2 siblings, 0 replies; 5+ messages in thread
From: Honnappa Nagarahalli @ 2021-02-01 23:22 UTC (permalink / raw)
  To: Ferruh Yigit, Jan Viktorin, Ruifeng Wang, jerinj,
	Bruce Richardson, Juraj Linkeš,
	Vimal Chungath
  Cc: dev, thomas, stable, Thierry Herbelot, nd, Honnappa Nagarahalli, nd

<snip>

> Subject: [PATCH] config/arm: fix typo in meson config
> 
> s/implementor/implementer
> 
> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
> Fixes: c51401832147 ("config/arm: add Graviton2")
> Cc: stable@dpdk.org
> 
> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Reviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>

Thanks Ferruh. I remember discussing this on the list. I am fine with 'implementEr' as that is what is used in Arm public specs.

> ---
> Cc: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: Ruifeng Wang <ruifeng.wang@arm.com>
> Cc: vcchunga@amazon.com
> ---
>  config/arm/arm64_armv8_linux_clang_ubuntu1804 | 4 ++--
>  config/arm/arm64_graviton2_linux_gcc          | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> index aa5ee0132065..ae3fb30c5da8 100644
> --- a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> +++ b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> @@ -14,7 +14,7 @@ cpu = 'armv8-a'
>  endian = 'little'
> 
>  [properties]
> -implementor_id = 'generic'
> -implementor_pn = 'default'
> +implementer_id = 'generic'
> +implementer_pn = 'default'
>  c_args = ['-target', 'aarch64-linux-gnu', '--sysroot', '/usr/aarch64-linux-gnu']
> c_link_args = ['-target', 'aarch64-linux-gnu', '-fuse-ld=lld', '--gcc-
> toolchain=/usr'] diff --git a/config/arm/arm64_graviton2_linux_gcc
> b/config/arm/arm64_graviton2_linux_gcc
> index cfe2397975fc..123f14eed29e 100644
> --- a/config/arm/arm64_graviton2_linux_gcc
> +++ b/config/arm/arm64_graviton2_linux_gcc
> @@ -13,8 +13,8 @@ cpu = 'armv8-a'
>  endian = 'little'
> 
>  [properties]
> -implementor_id = '0x41'
> -implementor_pn = '0xd0c'
> +implementer_id = '0x41'
> +implementer_pn = '0xd0c'
>  max_lcores = 64
>  max_numa_nodes = 1
>  numa = false
> --
> 2.29.2


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH] config/arm: fix typo in meson config
  2021-02-01 22:41 ` [dpdk-stable] [PATCH] config/arm: fix typo in meson config Ferruh Yigit
  2021-02-01 23:22   ` Honnappa Nagarahalli
@ 2021-02-02  2:19   ` Ruifeng Wang
  2021-02-02  8:08     ` Ferruh Yigit
  2021-02-02  7:12   ` Juraj Linkeš
  2 siblings, 1 reply; 5+ messages in thread
From: Ruifeng Wang @ 2021-02-02  2:19 UTC (permalink / raw)
  To: Ferruh Yigit, Jan Viktorin, jerinj, Bruce Richardson,
	Juraj Linkeš,
	Honnappa Nagarahalli, Vimal Chungath
  Cc: dev, thomas, stable, Thierry Herbelot, nd

> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit@intel.com>
> Sent: Tuesday, February 2, 2021 6:42 AM
> To: Jan Viktorin <viktorin@rehivetech.com>; Ruifeng Wang
> <Ruifeng.Wang@arm.com>; jerinj@marvell.com; Bruce Richardson
> <bruce.richardson@intel.com>; Juraj Linkeš <juraj.linkes@pantheon.tech>;
> Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; Vimal Chungath
> <vcchunga@amazon.com>
> Cc: Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org;
> thomas@monjalon.net; stable@dpdk.org; Thierry Herbelot
> <thierry.herbelot@6wind.com>
> Subject: [PATCH] config/arm: fix typo in meson config
> 
> s/implementor/implementer
> 
> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
> Fixes: c51401832147 ("config/arm: add Graviton2")

'implementor' -> 'implementer' change was made in 7870ae8994c6.
It seems 7870ae8994c6 missed Graviton2 config file which was added earlier.

With the fix tag update,
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>

> Cc: stable@dpdk.org
> 
> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> Cc: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: Ruifeng Wang <ruifeng.wang@arm.com>
> Cc: vcchunga@amazon.com
> ---
>  config/arm/arm64_armv8_linux_clang_ubuntu1804 | 4 ++--
>  config/arm/arm64_graviton2_linux_gcc          | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> index aa5ee0132065..ae3fb30c5da8 100644
> --- a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> +++ b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> @@ -14,7 +14,7 @@ cpu = 'armv8-a'
>  endian = 'little'
> 
>  [properties]
> -implementor_id = 'generic'
> -implementor_pn = 'default'
> +implementer_id = 'generic'
> +implementer_pn = 'default'
>  c_args = ['-target', 'aarch64-linux-gnu', '--sysroot', '/usr/aarch64-linux-gnu']
> c_link_args = ['-target', 'aarch64-linux-gnu', '-fuse-ld=lld', '--gcc-
> toolchain=/usr'] diff --git a/config/arm/arm64_graviton2_linux_gcc
> b/config/arm/arm64_graviton2_linux_gcc
> index cfe2397975fc..123f14eed29e 100644
> --- a/config/arm/arm64_graviton2_linux_gcc
> +++ b/config/arm/arm64_graviton2_linux_gcc
> @@ -13,8 +13,8 @@ cpu = 'armv8-a'
>  endian = 'little'
> 
>  [properties]
> -implementor_id = '0x41'
> -implementor_pn = '0xd0c'
> +implementer_id = '0x41'
> +implementer_pn = '0xd0c'
>  max_lcores = 64
>  max_numa_nodes = 1
>  numa = false
> --
> 2.29.2


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH] config/arm: fix typo in meson config
  2021-02-02  2:19   ` Ruifeng Wang
@ 2021-02-02  8:08     ` Ferruh Yigit
  0 siblings, 0 replies; 5+ messages in thread
From: Ferruh Yigit @ 2021-02-02  8:08 UTC (permalink / raw)
  To: Ruifeng Wang, Jan Viktorin, jerinj, Bruce Richardson,
	Juraj Linkeš,
	Honnappa Nagarahalli, Vimal Chungath
  Cc: dev, thomas, stable, Thierry Herbelot, nd

On 2/2/2021 2:19 AM, Ruifeng Wang wrote:
>> -----Original Message-----
>> From: Ferruh Yigit <ferruh.yigit@intel.com>
>> Sent: Tuesday, February 2, 2021 6:42 AM
>> To: Jan Viktorin <viktorin@rehivetech.com>; Ruifeng Wang
>> <Ruifeng.Wang@arm.com>; jerinj@marvell.com; Bruce Richardson
>> <bruce.richardson@intel.com>; Juraj Linkeš <juraj.linkes@pantheon.tech>;
>> Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; Vimal Chungath
>> <vcchunga@amazon.com>
>> Cc: Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org;
>> thomas@monjalon.net; stable@dpdk.org; Thierry Herbelot
>> <thierry.herbelot@6wind.com>
>> Subject: [PATCH] config/arm: fix typo in meson config
>>
>> s/implementor/implementer
>>
>> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
>> Fixes: c51401832147 ("config/arm: add Graviton2")
> 
> 'implementor' -> 'implementer' change was made in 7870ae8994c6.
> It seems 7870ae8994c6 missed Graviton2 config file which was added earlier.
> 

Right, I missed the rename patch, so it should be:
Fixes: 7870ae8994c6 ("config/arm: rename variables")

And that rename patch does 'implementor_pn' -> 'part_number', so I will update 
this patch to do the same.

> With the fix tag update,
> Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
> 
>> Cc: stable@dpdk.org
>>
>> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> ---
>> Cc: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> Cc: Ruifeng Wang <ruifeng.wang@arm.com>
>> Cc: vcchunga@amazon.com
>> ---
>>   config/arm/arm64_armv8_linux_clang_ubuntu1804 | 4 ++--
>>   config/arm/arm64_graviton2_linux_gcc          | 4 ++--
>>   2 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> b/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> index aa5ee0132065..ae3fb30c5da8 100644
>> --- a/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> +++ b/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> @@ -14,7 +14,7 @@ cpu = 'armv8-a'
>>   endian = 'little'
>>
>>   [properties]
>> -implementor_id = 'generic'
>> -implementor_pn = 'default'
>> +implementer_id = 'generic'
>> +implementer_pn = 'default'
>>   c_args = ['-target', 'aarch64-linux-gnu', '--sysroot', '/usr/aarch64-linux-gnu']
>> c_link_args = ['-target', 'aarch64-linux-gnu', '-fuse-ld=lld', '--gcc-
>> toolchain=/usr'] diff --git a/config/arm/arm64_graviton2_linux_gcc
>> b/config/arm/arm64_graviton2_linux_gcc
>> index cfe2397975fc..123f14eed29e 100644
>> --- a/config/arm/arm64_graviton2_linux_gcc
>> +++ b/config/arm/arm64_graviton2_linux_gcc
>> @@ -13,8 +13,8 @@ cpu = 'armv8-a'
>>   endian = 'little'
>>
>>   [properties]
>> -implementor_id = '0x41'
>> -implementor_pn = '0xd0c'
>> +implementer_id = '0x41'
>> +implementer_pn = '0xd0c'
>>   max_lcores = 64
>>   max_numa_nodes = 1
>>   numa = false
>> --
>> 2.29.2
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH] config/arm: fix typo in meson config
  2021-02-01 22:41 ` [dpdk-stable] [PATCH] config/arm: fix typo in meson config Ferruh Yigit
  2021-02-01 23:22   ` Honnappa Nagarahalli
  2021-02-02  2:19   ` Ruifeng Wang
@ 2021-02-02  7:12   ` Juraj Linkeš
  2 siblings, 0 replies; 5+ messages in thread
From: Juraj Linkeš @ 2021-02-02  7:12 UTC (permalink / raw)
  To: Ferruh Yigit, Jan Viktorin, Ruifeng Wang, Jerin Jacob,
	Bruce Richardson, Honnappa Nagarahalli, Vimal Chungath
  Cc: dev, Thomas Monjalon, stable, Thierry Herbelot



> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit@intel.com>
> Sent: Monday, February 1, 2021 11:42 PM
> To: Jan Viktorin <viktorin@rehivetech.com>; Ruifeng Wang
> <ruifeng.wang@arm.com>; Jerin Jacob <jerinj@marvell.com>; Bruce Richardson
> <bruce.richardson@intel.com>; Juraj Linkeš <juraj.linkes@pantheon.tech>;
> Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>; Vimal Chungath
> <vcchunga@amazon.com>
> Cc: Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org; Thomas Monjalon
> <thomas@monjalon.net>; stable@dpdk.org; Thierry Herbelot
> <thierry.herbelot@6wind.com>
> Subject: [PATCH] config/arm: fix typo in meson config
> 
> s/implementor/implementer
> 
> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
> Fixes: c51401832147 ("config/arm: add Graviton2")
> Cc: stable@dpdk.org
> 
> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---

Thanks for submitting this,
Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-02-02  8:08 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <7a4b591b-5e32-a3f8-a8ff-59a636217814@6wind.com>
2021-02-01 22:41 ` [dpdk-stable] [PATCH] config/arm: fix typo in meson config Ferruh Yigit
2021-02-01 23:22   ` Honnappa Nagarahalli
2021-02-02  2:19   ` Ruifeng Wang
2021-02-02  8:08     ` Ferruh Yigit
2021-02-02  7:12   ` Juraj Linkeš

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).