Re: [PATCH 1/6] dt-bindings: mtd: qcom,nandc: Document the SDX75 NAND

From: Kaushal Kumar
Date: Thu Mar 27 2025 - 06:42:43 EST


On 3/13/2025 8:18 PM, Krzysztof Kozlowski wrote:
On 13/03/2025 14:09, Kaushal Kumar wrote:
Document the QPIC NAND controller v2.1.1 being used in
SDX75 SoC and it uses BAM DMA.
Please wrap commit message according to Linux coding style / submission
process (neither too early nor over the limit):
https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597
Sure, will update in v2.
SDX75 NAND controller has DMA-coherent and iommu support
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)
That's a new compatible, so how can you have existing errors? Looks not
related.
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.
Signed-off-by: Kaushal Kumar <quic_kaushalk@xxxxxxxxxxx>
---
.../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:
Testing patches before sending is still on todo list for Qualcomm :/.
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.
+ - items:
+ - 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
Which controllers are DMA coherent?
I will add it under a sdx75 check since it is applicable to sdx75 only.

+
+ iommus:
+ minItems: 1
+ maxItems: 3
You need to list the items. Why is this flexible?
Agree, this need not be flexible. Will update in v2.

+
qcom,cmd-crci:
$ref: /schemas/types.yaml#/definitions/uint32
description:

Best regards,
Krzysztof