2021-10-19 23:43:18

by Dmitry Osipenko

[permalink] [raw]
Subject: [PATCH v1] ARM: dts: elpida_ecb240abacn: Change Elpida compatible

Vendor prefix shouldn't start with capital letter. The Elpida Memory
compatible was never used in practice, hence just correct the compatible.

Signed-off-by: Dmitry Osipenko <[email protected]>
---
arch/arm/boot/dts/elpida_ecb240abacn.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/elpida_ecb240abacn.dtsi b/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
index d87ee4794f83..9698801cbcfb 100644
--- a/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
+++ b/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
@@ -5,7 +5,7 @@

/ {
elpida_ECB240ABACN: lpddr2 {
- compatible = "Elpida,ECB240ABACN","jedec,lpddr2-s4";
+ compatible = "elpida,ECB240ABACN","jedec,lpddr2-s4";
density = <2048>;
io-width = <32>;

--
2.32.0


2021-10-27 21:27:33

by Dmitry Osipenko

[permalink] [raw]
Subject: Re: [PATCH v1] ARM: dts: elpida_ecb240abacn: Change Elpida compatible

20.10.2021 02:40, Dmitry Osipenko пишет:
> Vendor prefix shouldn't start with capital letter. The Elpida Memory
> compatible was never used in practice, hence just correct the compatible.
>
> Signed-off-by: Dmitry Osipenko <[email protected]>
> ---
> arch/arm/boot/dts/elpida_ecb240abacn.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/elpida_ecb240abacn.dtsi b/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
> index d87ee4794f83..9698801cbcfb 100644
> --- a/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
> +++ b/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
> @@ -5,7 +5,7 @@
>
> / {
> elpida_ECB240ABACN: lpddr2 {
> - compatible = "Elpida,ECB240ABACN","jedec,lpddr2-s4";
> + compatible = "elpida,ECB240ABACN","jedec,lpddr2-s4";
> density = <2048>;
> io-width = <32>;
>
>

This patch should go via OMAP tree since there is only OMAP board which
uses this dtsi. To make it more clear, "elpida" variant is now
documented in the bindings [1][2].

[1]
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=9e17f71e9c334f14ad6a8ec3edc09c7a4244e12f

[2]
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=001b8b2594db4ea24fbea4c161e665f858917fce

If there is no interest in taking the patch via OMAP tree, then it may
go via the DT git tree. Rob, you should pick it up then.

Thanks!

2021-12-09 07:11:05

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH v1] ARM: dts: elpida_ecb240abacn: Change Elpida compatible

Hi,

* Dmitry Osipenko <[email protected]> [211027 12:19]:
> 20.10.2021 02:40, Dmitry Osipenko пишет:
> > Vendor prefix shouldn't start with capital letter. The Elpida Memory
> > compatible was never used in practice, hence just correct the compatible.
> >
> > Signed-off-by: Dmitry Osipenko <[email protected]>
> > ---
> > arch/arm/boot/dts/elpida_ecb240abacn.dtsi | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/elpida_ecb240abacn.dtsi b/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
> > index d87ee4794f83..9698801cbcfb 100644
> > --- a/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
> > +++ b/arch/arm/boot/dts/elpida_ecb240abacn.dtsi
> > @@ -5,7 +5,7 @@
> >
> > / {
> > elpida_ECB240ABACN: lpddr2 {
> > - compatible = "Elpida,ECB240ABACN","jedec,lpddr2-s4";
> > + compatible = "elpida,ECB240ABACN","jedec,lpddr2-s4";
> > density = <2048>;
> > io-width = <32>;
> >
> >
>
> This patch should go via OMAP tree since there is only OMAP board which
> uses this dtsi. To make it more clear, "elpida" variant is now
> documented in the bindings [1][2].
>
> [1]
> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=9e17f71e9c334f14ad6a8ec3edc09c7a4244e12f
>
> [2]
> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=001b8b2594db4ea24fbea4c161e665f858917fce
>
> If there is no interest in taking the patch via OMAP tree, then it may
> go via the DT git tree. Rob, you should pick it up then.

Applying into omap-for-v5.17/dt thanks.

Regards,

Tony