On Wed, Oct 11, 2017 at 10:12:50AM -0500, Gustavo A. R. Silva wrote: > Hi Johan, > > Quoting Johan Hovold <jo...@kernel.org>:
> > Patch is fine, but next time make sure to run scripts/get_maintainer.sh > > to figure which people and lists to CC. > > > > I actually did: > > $ scripts/get_maintainer.pl --nokeywords --nogit --nogit-fallback > drivers/staging/greybus/es2.c > > and this was the output: > > linux-kernel@vger.kernel.org (open list) Very odd. If I run the exact same command I get the expected result: Johan Hovold <jo...@kernel.org> (maintainer:GREYBUS SUBSYSTEM) Alex Elder <el...@kernel.org> (maintainer:GREYBUS SUBSYSTEM) Greg Kroah-Hartman <gre...@linuxfoundation.org> (maintainer:GREYBUS SUBSYSTEM) greybus-...@lists.linaro.org (moderated list:GREYBUS SUBSYSTEM) de...@driverdev.osuosl.org (open list:STAGING SUBSYSTEM) linux-kernel@vger.kernel.org (open list) > > We also try to include the module (component) "es2" in the commit > > summary (unless you really are doing things subsystem wide), although > > that has not been enforced for staging clean ups recently. > > > > I got it. I'll take that into account for future patches. Looking at the commit history for the file in questions usually gives a hint about the style used for the corresponding subsystem. Thanks, Johan
