diff mbox series

[v1] ARM: dts: elpida_ecb240abacn: Change Elpida compatible

Message ID 20211019234059.29877-1-digetx@gmail.com (mailing list archive)
State New, archived
Headers show
Series [v1] ARM: dts: elpida_ecb240abacn: Change Elpida compatible | expand

Commit Message

Dmitry Osipenko Oct. 19, 2021, 11:40 p.m. UTC
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 <digetx@gmail.com>
---
 arch/arm/boot/dts/elpida_ecb240abacn.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Dmitry Osipenko Oct. 27, 2021, 12:19 p.m. UTC | #1
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 <digetx@gmail.com>
> ---
>  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!
Tony Lindgren Dec. 9, 2021, 7:10 a.m. UTC | #2
Hi,

* Dmitry Osipenko <digetx@gmail.com> [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 <digetx@gmail.com>
> > ---
> >  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
diff mbox series

Patch

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