Re: [PATCH v5 1/3] dt-bindings: mtd: spinand: Add randomizer enable/disable properties

From: Cheng Ming Lin

Date: Mon Feb 23 2026 - 00:51:27 EST


Hi Krzysztof,

Krzysztof Kozlowski <krzk@xxxxxxxxxx> 於 2026年2月12日週四 下午3:33寫道:
>
> On Wed, Feb 11, 2026 at 06:05:51PM +0800, Cheng Ming Lin wrote:
> > diff --git a/Documentation/devicetree/bindings/mtd/nand-chip.yaml b/Documentation/devicetree/bindings/mtd/nand-chip.yaml
> > index 609d4a4ddd80..2fcbc4b73e95 100644
> > --- a/Documentation/devicetree/bindings/mtd/nand-chip.yaml
> > +++ b/Documentation/devicetree/bindings/mtd/nand-chip.yaml
> > @@ -67,6 +67,14 @@ properties:
> > the secure regions present.
> > $ref: /schemas/types.yaml#/definitions/uint64-matrix
> >
> > + nand-randomizer-enable:
> > + description: Enable the randomizer feature.
> > + type: boolean
> > +
> > + nand-randomizer-disable:
> > + description: Disable the randomizer feature.
> > + type: boolean
>
> Read again feedback from Miquel - two EXCLUSIVE properties. You miss
> "not" with "required":
> https://lore.kernel.org/all/20230118163208.GA117919-robh@xxxxxxxxxx/

Thank you for the review and the reference link.

I understand now. I will add the "not" with "required" constraints to
properly enforce the mutual exclusivity for these two properties.
This will be updated in the upcoming v6.

>
> Best regards,
> Krzysztof
>

Thanks,
Cheng Ming Lin