[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YZzpmPfnZ/I2ZdZk@sirena.org.uk>
Date: Tue, 23 Nov 2021 13:16:08 +0000
From: Mark Brown <broonie@...nel.org>
To: Adam Ward <Adam.Ward.opensource@...semi.com>
Cc: Rob Herring <robh+dt@...nel.org>,
Liam Girdwood <lgirdwood@...il.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
Support Opensource <Support.Opensource@...semi.com>
Subject: Re: [PATCH 1/2] DA9121: add DA914x binding info
On Tue, Nov 23, 2021 at 01:06:23PM +0000, Adam Ward wrote:
> > On Tue, Nov 23, 2021 at 12:30PM +0000, Mark Brown wrote:
> > This is removing the existing DA9140 compatible?
> No. It's never been recognised in the driver itself and shouldn't have been there.
> Detritus from an obsolete test.
So by "no" you mean "yes" then? In any case you shouldn't be putting a
change like this in the middle of an unrelated commit without mentioning
it in the commit log, if you feel the need to remove this compatible you
should make a commit doing so. Please see submitting-patches.rst for
details on splitting changes up.
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists