Commit 411a1215 authored by Miquel Raynal's avatar Miquel Raynal
Browse files

dt-bindings: mtd: qcom: Fix a property position 



qcom,boot-partitions is a NAND chip property, not a NAND controller
property. Move the description of the property into the NAND chip
section and just enable the property in the if/else block.

Fixes: 5278cc93 ("dt-bindings: mtd: qcom_nandc: document qcom,boot-partitions binding")
Cc: Manivannan Sadhasivam <mani@kernel.org>
Signed-off-by: default avatarMiquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: default avatarRob Herring <robh@kernel.org>
Reviewed-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/linux-mtd/20230619092916.3028470-6-miquel.raynal@bootlin.com
parent efdd2963

Documentation/devicetree/bindings/mtd/qcom,nandc.yaml

+25 −16
Original line number Diff line number Diff line
@@ -45,6 +45,22 @@ patternProperties: 
        enum:

          - 512



      qcom,boot-partitions:

        $ref: /schemas/types.yaml#/definitions/uint32-matrix

        items:

          items:

            - description: offset

            - description: size

        description:

          Boot partition use a different layout where the 4 bytes of spare

          data are not protected by ECC. Use this to declare these special

          partitions by defining first the offset and then the size.



          It's in the form of <offset1 size1 offset2 size2 offset3 ...>

          and should be declared in ascending order.



          Refer to the ipq8064 example on how to use this special binding.



allOf:

  - $ref: nand-controller.yaml#
@@ -107,22 +123,15 @@ allOf: 
              - qcom,ipq806x-nand



    then:

      patternProperties:

        "^nand@[a-f0-9]$":

          properties:

        qcom,boot-partitions:

          $ref: /schemas/types.yaml#/definitions/uint32-matrix

          items:

            items:

              - description: offset

              - description: size

          description:

            Boot partition use a different layout where the 4 bytes of spare

            data are not protected by ECC. Use this to declare these special

            partitions by defining first the offset and then the size.



            It's in the form of <offset1 size1 offset2 size2 offset3 ...>

            and should be declared in ascending order.



            Refer to the ipq8064 example on how to use this special binding.

            qcom,boot-partitions: true

    else:

      patternProperties:

        "^nand@[a-f0-9]$":

          properties:

            qcom,boot-partitions: false



required:

  - compatible