diff mbox series

dt-bindings: spi: zynqmp-qspi: Add power-domains and iommus properties

Message ID 87563d1a920458dff134bb92b27645471f4d9fca.1683035258.git.michal.simek@amd.com
State New
Headers
Series dt-bindings: spi: zynqmp-qspi: Add power-domains and iommus properties |

Commit Message

Michal Simek May 2, 2023, 1:47 p.m. UTC 
  Describe optional power-domain property to fix dts_check warnings. The
similar change was done by commit 0aec3958c88f ("dt-bindings: gpio: zynq:
Add power-domains").
And also by commit 84fa8f159022 ("dt-bindings: arm: coresight-tmc: Add
'iommu' property") for iommus property.

Signed-off-by: Michal Simek <michal.simek@amd.com>
---

 Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Krzysztof Kozlowski May 2, 2023, 8:17 p.m. UTC | #1 
On 02/05/2023 15:47, Michal Simek wrote:
> Describe optional power-domain property to fix dts_check warnings. The
> similar change was done by commit 0aec3958c88f ("dt-bindings: gpio: zynq:
> Add power-domains").
> And also by commit 84fa8f159022 ("dt-bindings: arm: coresight-tmc: Add
> 'iommu' property") for iommus property.

I am a bit pedantic, but you cannot use such arguments for this change.
Just because some other unrelated device (Coresight or GPIO controller)
has IOMMU does not mean your device has it as well. You know, some
device has reset-gpios, so shall you add it here?

The argument could be something around: The foo bar uses IOMMU and
belongs to a power domain.

Please re-phrase the commit msg.


Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml b/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml
index 20f77246d365..226d8b493b57 100644
--- a/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml
+++ b/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml
@@ -32,6 +32,12 @@  properties:
   clocks:
     maxItems: 2
 
+  iommus:
+    maxItems: 1
+
+  power-domains:
+    maxItems: 1
+
 required:
   - compatible
   - reg