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@xxxxxxxxxx>
> Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> ---
> 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@0 {
label = "boot-nand";
reg = <0 0x58a0000>;
+ nonsense;
};
partition@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.
