Received-SPF: pass (google.com: domain of linux-kernel+bounces-52845-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Mon, 5 Feb 2024 15:50:11 +0100
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Rob Herring <robh@kernel.org>
Cc: Zhenhua Huang <quic_zhenhuah@quicinc.com>,
 krzysztof.kozlowski+dt@linaro.org, quic_tingweiz@quicinc.com,
 richard@nod.at, vigneshr@ti.com, conor+dt@kernel.org,
 linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
 quic_bjorande@quicinc.com, devicetree@vger.kernel.org, robh+dt@kernel.org
Subject: Re: [PATCH v2] dt-bindings: mtd: avoid automatically select from
 mtd.yaml
Message-ID: <20240205155011.7f27bb07@xps-13>
In-Reply-To: <170542500495.126061.10553621105906619365.robh@kernel.org>
References: <1704885705-7486-1-git-send-email-quic_zhenhuah@quicinc.com>
	<170542500495.126061.10553621105906619365.robh@kernel.org>
Organization: Bootlin
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hi,

robh@kernel.org wrote on Tue, 16 Jan 2024 11:10:05 -0600:

> On Wed, 10 Jan 2024 19:21:45 +0800, Zhenhua Huang wrote:
> > The mtd binding is always $ref'ed by other bindings, default selector
> > should be from other binding files which ref'ed it. Now, "$nodename" in
> > mtd.yaml turns into a "select" automatically such that a few binding ch=
eck
> > issues reported because it conflicts with sram devices(eg, qcom,imem.ya=
ml,
> > rules in folder sram/*)
> >=20
> > To avoid the automatically created "select" in mtd.yaml, adding:
> >=20
> > select: false
> >=20
> > Suggested-by: Bjorn Andersson <quic_bjorande@quicinc.com>
> > Fixes: 7bdc671822e9 ("dt-bindings: mtd: physmap: Reuse the generic defi=
nitions")
> > Signed-off-by: Zhenhua Huang <quic_zhenhuah@quicinc.com>
> > ---
> > Hi Bjorn,
> >=20
> > As the idea is from your comment, I added "Suggested-by" you.
> > Please tell me if that's not suitable. Thanks.
> >=20
> >  Documentation/devicetree/bindings/mtd/mtd.yaml | 2 ++
> >  1 file changed, 2 insertions(+)
> >  =20
>=20
> Acked-by: Rob Herring <robh@kernel.org>

Applied to mtd/next, thanks!

Miqu=C3=A8l