On 13/03/2025 14:09, Kaushal Kumar wrote:Sure, will update in v2.
Document the QPIC NAND controller v2.1.1 being used inPlease wrap commit message according to Linux coding style / submission
SDX75 SoC and it uses BAM DMA.
process (neither too early nor over the limit):
https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597
It is not an existing error. It is the error that would come unless the new properties are updated. Anyhow sorry for the confusion, will remove this in v2.SDX75 NAND controller has DMA-coherent and iommu supportThat's a new compatible, so how can you have existing errors? Looks not
so define them in the properties section, without which
'dtbs_check' reports the following error:
nand-controller@1cc8000: Unevaluated properties are not
allowed ('dma-coherent', 'iommus' were unexpected)
related.
I had run the "make dt_binding_check". Possibly due to my older version of dtschema , it did not get caught. After the schema update, now it is getting caught. Will update in v2.Signed-off-by: Kaushal Kumar <quic_kaushalk@xxxxxxxxxxx>Testing patches before sending is still on todo list for Qualcomm :/.
---
.../devicetree/bindings/mtd/qcom,nandc.yaml | 23 ++++++++++++++-----
1 file changed, 17 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml b/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
index 35b4206ea918..8b77e8837205 100644
--- a/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
+++ b/Documentation/devicetree/bindings/mtd/qcom,nandc.yaml
@@ -11,12 +11,17 @@ maintainers:
properties:
compatible:
- enum:
- - qcom,ipq806x-nand
- - qcom,ipq4019-nand
- - qcom,ipq6018-nand
- - qcom,ipq8074-nand
- - qcom,sdx55-nand
+ OneOf:
I will add it under a sdx75 check since it is applicable to sdx75 only.+ - items:Which controllers are DMA coherent?
+ - enum:
+ - qcom,sdx75-nand
+ - const: qcom,sdx55-nand
+ - items:
+ - const: qcom,ipq806x-nand
+ - const: qcom,ipq4019-nand
+ - const: qcom,ipq6018-nand
+ - const: qcom,ipq8074-nand
+ - const: qcom,sdx55-nand
reg:
maxItems: 1
@@ -31,6 +36,12 @@ properties:
- const: core
- const: aon
+ dma-coherent: true
Agree, this need not be flexible. Will update in v2.
+You need to list the items. Why is this flexible?
+ iommus:
+ minItems: 1
+ maxItems: 3
+
qcom,cmd-crci:
$ref: /schemas/types.yaml#/definitions/uint32
description:
Best regards,
Krzysztof