[PATCH 0/3] Fix a couple of missing unevaluated properties

Miquel Raynal miquel.raynal at bootlin.com
Tue Jun 6 09:23:28 PDT 2023


Hello,

miquel.raynal at bootlin.com wrote on Tue,  6 Jun 2023 12:30:54 +0200:

> As discussed with Krzysztof and Chris, it seems like each NAND
> controller binding should actually explicitely list all the allowed
> properties in a schema and use "unevaluatedProperties: false" in both
> the NAND controller and the NAND chip sections. While this restriction
> always applied on the controller side, this was not enforced on the
> chips side AFAIR.
> 
> Link: https://lore.kernel.org/all/a23dd485-a3d9-e31f-be3e-0ab293fcfc4a@linaro.org/

The discussion pointed above has lead to another observation which also
needs to be fixed in this series. A v2 is coming soon.

> While converting Marvell controller bindings, Chris explicitely pointed
> similar bindings which would also trigger errors when using
> "unevaluatedProperties: false" because of the reg/partitions/etc
> properties not being described. Here is an attempt at making these more
> precise and robust.
> 
> These change made me realize the qcom,boot-partition property
> description was broken, so here is a fix for it.
> 
> Thanks,
> Miquèl
> 
> Miquel Raynal (3):
>   dt-bindings: mtd: qcom: Fix a property position
>   dt-bindings: mtd: qcom: Prevent NAND chip unevaluated properties
>   dt-bindings: mtd: ingenic: Prevent NAND chip unevaluated properties
> 
>  .../devicetree/bindings/mtd/ingenic,nand.yaml | 11 +++++
>  .../devicetree/bindings/mtd/qcom,nandc.yaml   | 47 ++++++++++++-------
>  2 files changed, 42 insertions(+), 16 deletions(-)
> 


Thanks,
Miquèl



More information about the linux-mtd mailing list