From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756889AbdJKQIR (ORCPT ); Wed, 11 Oct 2017 12:08:17 -0400 Received: from mail-lf0-f67.google.com ([209.85.215.67]:45913 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751527AbdJKQIQ (ORCPT ); Wed, 11 Oct 2017 12:08:16 -0400 X-Google-Smtp-Source: ABhQp+TSLUtHPSHhuBmmQHpwmyyj/OJ6+H0FaAHELZyVUAomv2ddHQ65W6FhyVoCtlE7hqxYSry/5A== Date: Wed, 11 Oct 2017 18:08:15 +0200 From: Johan Hovold To: "Gustavo A. R. Silva" Cc: Johan Hovold , Greg Kroah-Hartman , linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: greybus: mark expected switch fall-through in check_urb_status Message-ID: <20171011160815.GA2655@localhost> References: <20171010201041.GA1464@embeddedor.com> <20171011074501.GQ4269@localhost> <20171011101250.Horde.ShAACqxWq_0okyS3bLhkOF7@gator4166.hostgator.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171011101250.Horde.ShAACqxWq_0okyS3bLhkOF7@gator4166.hostgator.com> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 11, 2017 at 10:12:50AM -0500, Gustavo A. R. Silva wrote: > Hi Johan, > > Quoting Johan Hovold : > > 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 (maintainer:GREYBUS SUBSYSTEM) Alex Elder (maintainer:GREYBUS SUBSYSTEM) Greg Kroah-Hartman (maintainer:GREYBUS SUBSYSTEM) greybus-dev@lists.linaro.org (moderated list:GREYBUS SUBSYSTEM) devel@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