diff mbox

kirkwood: fix spelling mistake

Message ID trinity-6911950b-e19a-4b44-aeed-adaccbc33164-1479414587776@3capp-gmx-bs50 (mailing list archive)
State New, archived
Headers show

Commit Message

p.wassi@gmx.at Nov. 17, 2016, 8:29 p.m. UTC
From: Paul Wassi <p.wassi@gmx.at>

Fix a spelling mistake in arch/arm/boot/dts/kirkwood-topkick.dts

Signed-off-by: Paul Wassi <p.wassi@gmx.at>
---
The manufacturer's name is Univer*s*al Scientific Industrial...
Compare with footer of page here:
http://www.usish.com/english/products_topkick1281p2.php

 arch/arm/boot/dts/kirkwood-topkick.dts |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Lunn Nov. 17, 2016, 9 p.m. UTC | #1
On Thu, Nov 17, 2016 at 09:29:47PM +0100, p.wassi@gmx.at wrote:
> From: Paul Wassi <p.wassi@gmx.at>
> 
> Fix a spelling mistake in arch/arm/boot/dts/kirkwood-topkick.dts
> 
> Signed-off-by: Paul Wassi <p.wassi@gmx.at>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

Thanks
    Andrew
Andrew Lunn Nov. 17, 2016, 9:02 p.m. UTC | #2
On Thu, Nov 17, 2016 at 09:29:47PM +0100, p.wassi@gmx.at wrote:
> From: Paul Wassi <p.wassi@gmx.at>

Hi Paul

You should also send the patch to the Marvell MVEBU
maintainers. Otherwise it can fall through the cracks.

	     Andrew
p.wassi@gmx.at Nov. 18, 2016, 6:02 a.m. UTC | #3
Hi Andrew,

> You should also send the patch to the Marvell MVEBU
> maintainers.

how do I find Marvell mvebu maintainers?
The list in
https://www.kernel.org/doc/linux/MAINTAINERS
(line 1429) references mvebu/kirkwood and gave me the
mailing list's address. Should the other two maintainers
be addressed directly?

Best regards,
P. Wassi
Gregory CLEMENT Nov. 18, 2016, 1:02 p.m. UTC | #4
Hi Paul,
 
 On jeu., nov. 17 2016, p.wassi@gmx.at wrote:

> From: Paul Wassi <p.wassi@gmx.at>
>
> Fix a spelling mistake in arch/arm/boot/dts/kirkwood-topkick.dts
>
> Signed-off-by: Paul Wassi <p.wassi@gmx.at>

Applied on mvebu/dt.

I fix the title, as for patcj modifying the dts, the title should start
by "ARM: dts: ". I also added the comment from the email which add more
inforamtion that the original commit log.

Thanks,

Gregory

> ---
> The manufacturer's name is Univer*s*al Scientific Industrial...
> Compare with footer of page here:
> http://www.usish.com/english/products_topkick1281p2.php
>
>  arch/arm/boot/dts/kirkwood-topkick.dts |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff -rupN a/arch/arm/boot/dts/kirkwood-topkick.dts b/arch/arm/boot/dts/kirkwood-topkick.dts
> --- a/arch/arm/boot/dts/kirkwood-topkick.dts
> +++ b/arch/arm/boot/dts/kirkwood-topkick.dts
> @@ -4,7 +4,7 @@
>  #include "kirkwood-6282.dtsi"
>  
>  / {
> -	model = "Univeral Scientific Industrial Co. Topkick-1281P2";
> +	model = "Universal Scientific Industrial Co. Topkick-1281P2";
>  	compatible = "usi,topkick-1281P2", "usi,topkick", "marvell,kirkwood-88f6282", "marvell,kirkwood";
>  
>  	memory {
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Gregory CLEMENT Nov. 18, 2016, 1:04 p.m. UTC | #5
Hi Paul,
 
 On ven., nov. 18 2016, p.wassi@gmx.at wrote:

> Hi Andrew,
>
>> You should also send the patch to the Marvell MVEBU
>> maintainers.
>
> how do I find Marvell mvebu maintainers?
> The list in
> https://www.kernel.org/doc/linux/MAINTAINERS
> (line 1429) references mvebu/kirkwood and gave me the
> mailing list's address. Should the other two maintainers
> be addressed directly?

About putting the maintainer in Cc. Using ./scripts/get_maintainer.pl
coudl help you to find the people to put in CC. For example for your
patch:
Jason Cooper <jason@lakedaemon.net> (maintainer:ARM/Marvell Kirkwood and Armada 370, 375, 38x,...)
Andrew Lunn <andrew@lunn.ch> (maintainer:ARM/Marvell Kirkwood and Armada 370, 375, 38x,...)
Gregory Clement <gregory.clement@free-electrons.com> (maintainer:ARM/Marvell Kirkwood and Armada 370, 375, 38x,...)
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> (maintainer:ARM/Marvell Kirkwood and Armada 370, 375, 38x,...)

So indeed you should put the 4 co-maintainers in Cc.

Thanks,

Gregory
diff mbox

Patch

diff -rupN a/arch/arm/boot/dts/kirkwood-topkick.dts b/arch/arm/boot/dts/kirkwood-topkick.dts
--- a/arch/arm/boot/dts/kirkwood-topkick.dts
+++ b/arch/arm/boot/dts/kirkwood-topkick.dts
@@ -4,7 +4,7 @@ 
 #include "kirkwood-6282.dtsi"
 
 / {
-	model = "Univeral Scientific Industrial Co. Topkick-1281P2";
+	model = "Universal Scientific Industrial Co. Topkick-1281P2";
 	compatible = "usi,topkick-1281P2", "usi,topkick", "marvell,kirkwood-88f6282", "marvell,kirkwood";
 
 	memory {