[PATCH v2 06/17] dt-bindings: mtd: qcom: Prevent NAND chip unevaluated properties

[Chris Packham]

On 7/06/23 05:52, Miquel Raynal wrote:
> List all the possible properties in the NAND chip as per the example and
> set unevaluatedProperties to false in the NAND chip section.
>
> Cc: Manivannan Sadhasivam <mani at kernel.org>
> Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> ---
>   Documentation/devicetree/bindings/mtd/qcom,nandc.yaml | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml b/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
> index 7217d8a64d14..581973f452ff 100644
> --- a/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
> +++ b/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
> @@ -34,7 +34,9 @@ properties:
>   patternProperties:
>     "^nand@[a-f0-9]$":
>       type: object
> +    $ref: raw-nand-chip.yaml
>       properties:
> +
>         nand-bus-width:
>           const: 8
>   
> @@ -61,6 +63,8 @@ patternProperties:
>   
>             Refer to the ipq8064 example on how to use this special binding.
>   
> +    unevaluatedProperties: false
> +
>   allOf:
>     - $ref: "nand-controller.yaml#"
>   

This doesn't appear to be working as intended. If I add the following

diff --git a/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml 
b/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
index 581973f452ff..7a910a62c7b8 100644
--- a/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
+++ b/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
@@ -220,6 +220,7 @@ examples:
                  partition at 0 {
                      label = "boot-nand";
                      reg = <0 0x58a0000>;
+                    nonsense;
                  };

                  partition at 58a0000 {


Then I'd expect to get a complaint from `make mrproper; make 
DT_CHECKER_FLAGS=-m dt_binding_check DT_SCHEMA_FILES=qcom,nandc.yaml` 
but it seems to let this through.