Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 5 Jan 2023 16:27:32 +0000
From:   Lee Jones <lee@kernel.org>
To:     Tzung-Bi Shih <tzungbi@kernel.org>
Cc:     Mark Hasemeyer <markhas@chromium.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Raul Rangel <rrangel@chromium.org>,
        Bhanu Prakash Maiya <bhanumaiya@chromium.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Benson Leung <bleung@chromium.org>,
        Guenter Roeck <groeck@chromium.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        chrome-platform@lists.linux.dev, devicetree@vger.kernel.org
Subject: Re: [PATCH v10 2/3] dt-bindings: mfd: Add compatible string for UART
 support
Message-ID: <Y7b6dC6dDNO2bVjr@google.com>
References: <20221207104005.v10.1.If7926fcbad397bc6990dd725690229bed403948c@changeid>
 <20221207104005.v10.2.I9e018ecb8bdf341648cb64417085978ff0d22a46@changeid>
 <Y7WkeCi7/x/t37JM@google.com>
 <Y7YjmtwByTR+8tbZ@google.com>
 <Y7bdmiZ9lqKZXCfq@google.com>
 <Y7buUdgLfnQqnG+x@google.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <Y7buUdgLfnQqnG+x@google.com>
Precedence: bulk

On Thu, 05 Jan 2023, Tzung-Bi Shih wrote:

> On Thu, Jan 05, 2023 at 02:24:26PM +0000, Lee Jones wrote:
> > On Thu, 05 Jan 2023, Tzung-Bi Shih wrote:
> > 
> > > On Wed, Jan 04, 2023 at 04:08:24PM +0000, Lee Jones wrote:
> > > > On Wed, 07 Dec 2022, Mark Hasemeyer wrote:
> > > > 
> > > > > From: Bhanu Prakash Maiya <bhanumaiya@chromium.org>
> > > > > 
> > > > > Add a compatible string to support the UART implementation of the cros
> > > > > ec interface. The driver does not support the reg and interrupt
> > > > > properties, so exempt them from being required for UART compatible nodes.
> > > > > 
> > > > > Signed-off-by: Bhanu Prakash Maiya <bhanumaiya@chromium.org>
> > > > > Co-developed-by: Mark Hasemeyer <markhas@chromium.org>
> > > > > Signed-off-by: Mark Hasemeyer <markhas@chromium.org>
> > > > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > > > > ---
> > > > > 
> > > > > Changes in v10:
> > > > > - No change
> > > > > 
> > > [...]
> > > > 
> > > > I changed the subject line and applied the patch, thanks.
> > > 
> > > Pardon me.  I didn't know you would pick the patch so that I have queued it
> > > into chrome-platform[1].  Would you like me to drop the commit?
> > 
> > Yes, please do.
> 
> Dropped in chrome-platform.
> 
> > You should gain permission from the maintainer before you apply patches.
> > 
> >   % scripts/get_maintainer.pl -f Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
> >      Lee Jones <lee@kernel.org> (supporter:MULTIFUNCTION DEVICES (MFD))
> >      Rob Herring <robh+dt@kernel.org> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> >      Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> >      Matthias Brugger <matthias.bgg@gmail.com> (maintainer:ARM/Mediatek SoC support)
> 
> I see.  I guess for the case, I should get both MFD and DT bindings
> maintainers' Acks before applying the patch.

Yes.  The situation isn't overly clear and differs from subsystem to
subsystem and maintainer to maintainer.  In the MFD case, Rob and
Krzysztof conduct the first reviews.  99 times out of 100, if they're
happy, I'm happy and I merge the changes via the MFD tree.  Same goes
for Backlight and more recently, LEDs.

-- 
Lee Jones [李琼斯]