Re: [PATCH v2] dt-bindings: mtd: avoid automatically select from mtd.yaml

From: Miquel Raynal
Date: Mon Feb 05 2024 - 09:50:32 EST


Hi,

robh@xxxxxxxxxx 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 check
> > issues reported because it conflicts with sram devices(eg, qcom,imem.yaml,
> > rules in folder sram/*)
> >
> > To avoid the automatically created "select" in mtd.yaml, adding:
> >
> > select: false
> >
> > Suggested-by: Bjorn Andersson <quic_bjorande@xxxxxxxxxxx>
> > Fixes: 7bdc671822e9 ("dt-bindings: mtd: physmap: Reuse the generic definitions")
> > Signed-off-by: Zhenhua Huang <quic_zhenhuah@xxxxxxxxxxx>
> > ---
> > Hi Bjorn,
> >
> > As the idea is from your comment, I added "Suggested-by" you.
> > Please tell me if that's not suitable. Thanks.
> >
> > Documentation/devicetree/bindings/mtd/mtd.yaml | 2 ++
> > 1 file changed, 2 insertions(+)
> >
>
> Acked-by: Rob Herring <robh@xxxxxxxxxx>

Applied to mtd/next, thanks!

Miquèl