Re: [Outreachy kernel] who to send first patch to

[Greg KH <gregkh@linuxfoundation.org>]

On Thu, Feb 15, 2018 at 02:48:30PM -0300, Ezequiel Garcia wrote:
> On 15 February 2018 at 14:42, Ezequiel Garcia
> <ezequiel@vanguardiasur.com.ar> wrote:
> > On 15 February 2018 at 14:32, Dafna Hirschfeld <dafna3@gmail.com> wrote:
> >> Hi,
> >> I follow the first-patch tutorial.
> >> When running the get_maintainer.sh script, I get Greg Kroah-Hartman
> >> <gregkh@linuxfoundation.org>.
> >> Should I send the patch to this mailing list and add the maintainer as a CC,
> >> or should I send two separate mails, one to this mailing list and one to the
> >> maintainer ?
> >>
> >
> > Just like Shraddha said, you sould send just *one* mail.
> >
> > You can Cc as much developers as you want, but you'll usually want to keep
> > the Cc list short, Ccing only the relevant maintainers.
> >
> > Let's work with an example. I just crafted a small patch removing an unused
> > header from arch/arm/mach-mv78xx0/common.c:
> >
> > commit 42dc417bb0c0547a926880494ac3f79829bd3035 (HEAD -> orion-nand)
> > Author: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
> > Date:   Thu Feb 15 14:39:09 2018 -0300
> >
> >     ARM: mv78xx0: remove unused header
> >
> >     This file doesn't setup an orion-nand device, and so
> >     it does not need the orion-nand header.
> >
> >     Signed-off-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
> >
> > diff --git a/arch/arm/mach-mv78xx0/common.c b/arch/arm/mach-mv78xx0/common.c
> > index f72e1e9f5fc5..89c95e365e90 100644
> > --- a/arch/arm/mach-mv78xx0/common.c
> > +++ b/arch/arm/mach-mv78xx0/common.c
> > @@ -19,7 +19,6 @@
> >  #include <asm/mach/map.h>
> >  #include <asm/mach/time.h>
> >  #include <linux/platform_data/usb-ehci-orion.h>
> > -#include <linux/platform_data/mtd-orion_nand.h>
> >  #include <plat/time.h>
> >  #include <plat/common.h>
> >  #include <plat/addr-map.h>
> >
> 
> Oops... sent the mail by mistake. So, like I was saying: I have that
> patch ready.
> Now I need to see who I'll sent to:
> 
> ./scripts/get_maintainer.pl 0001-ARM-mv78xx0-remove-unused-header.patch
> Jason Cooper <jason@lakedaemon.net> (maintainer:ARM/Marvell
> Dove/MV78xx0/Orion SOC support)
> Andrew Lunn <andrew@lunn.ch> (maintainer:ARM/Marvell
> Dove/MV78xx0/Orion SOC support)
> Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> (maintainer:ARM/Marvell Dove/MV78xx0/Orion SOC support)
> Gregory Clement <gregory.clement@free-electrons.com>
> (maintainer:ARM/Marvell Dove/MV78xx0/Orion SOC support)
> Russell King <linux@armlinux.org.uk> (maintainer:ARM PORT)
> linux-arm-kernel@lists.infradead.org (moderated list:ARM SUB-ARCHITECTURES)
> linux-kernel@vger.kernel.org (open list)
> 
> OK, that's a long list. I'll send the patch to just the SOC
> maintainers and the ARM mailing list, not all of them listed above.
> 
> git send-email  --to jason@lakedaemon.net --to andrew@lunn.ch --to
> sebastian.hesselbarth@gmail.com --to
> gregory.clement@free-electrons.com
> 0001-ARM-mv78xx0-remove-unused-header.patch --cc
> linux-arm-kernel@lists.infradead.org
> 
> In your case, you'll most likely pick Cc the outreachy mailing list
> and send it to Greg KH (as Greg just said).

For staging patches, please also cc: the public mailing list for the
staging drivers, and any other maintainers of the drivers, they want to
see these patches as well.

thanks,

greg k-h