Message ID | 8f5651634df338743f95a7253a741f9ddc92487d.1683891609.git.michal.simek@amd.com |
---|---|
State | Accepted |
Commit | 64273b51a5156cae177394b3d9f036e0e61ca703 |
Headers | show |
Series | [v3] dt-bindings: dma: xilinx: Add power-domains to xlnx,zynqmp-dpdma | expand |
On 12-05-23, 13:40, Michal Simek wrote: > DP DMA has own power domain that's why describe required power-domain > property. Applied, thanks
diff --git a/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dpdma.yaml b/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dpdma.yaml index d6cbd95ec26d..2128f4645c98 100644 --- a/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dpdma.yaml +++ b/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dpdma.yaml @@ -41,6 +41,9 @@ properties: clock-names: const: axi_clk + power-domains: + maxItems: 1 + required: - "#dma-cells" - compatible @@ -48,12 +51,14 @@ required: - interrupts - clocks - clock-names + - power-domains additionalProperties: false examples: - | #include <dt-bindings/interrupt-controller/arm-gic.h> + #include <dt-bindings/power/xlnx-zynqmp-power.h> dma: dma-controller@fd4c0000 { compatible = "xlnx,zynqmp-dpdma"; @@ -63,6 +68,7 @@ examples: clocks = <&dpdma_clk>; clock-names = "axi_clk"; #dma-cells = <1>; + power-domains = <&zynqmp_firmware PD_DP>; }; ...
DP DMA has own power domain that's why describe required power-domain property. Signed-off-by: Michal Simek <michal.simek@amd.com> --- Changes in v3: - make power-domains as required property - also update commit message Changes in v2: - rewrite commit message - requested by Krzysztof The commit b06112cd5e08 ("arm64: dts: zynqmp: Add power domain for the DisplayPort DMA controller") added this property already in Linux that's why the patch is also fixing dts_check warnings. In v2 I got ack from Krzysztof but not adding it because of additional discussion about required property in v3. https://lore.kernel.org/r/029ba923-d13e-ea7c-018d-95e179dda2e5@linaro.org --- .../devicetree/bindings/dma/xilinx/xlnx,zynqmp-dpdma.yaml | 6 ++++++ 1 file changed, 6 insertions(+)