* [PATCH] arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage
@ 2020-06-28 18:08 Maxime Ripard
2020-07-09 16:39 ` Rob Herring
0 siblings, 1 reply; 2+ messages in thread
From: Maxime Ripard @ 2020-06-28 18:08 UTC (permalink / raw)
To: linux-arm-kernel, Chen-Yu Tsai, Maxime Ripard
Now that the IOMMU driver has been introduced, it prevents any access from
a DMA master going through it that hasn't properly mapped the pages, and
that link is set up through the iommus property.
Unfortunately we forgot to add that property to the video engine node when
adding the IOMMU node, so now any DMA access is broken.
Fixes: b3a0a2f910c7 ("arm64: dts: allwinner: h6: Add IOMMU")
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
---
.../bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml | 3 +++
arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 1 +
2 files changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
index 1dee641e3ea1..ef624327e33e 100644
--- a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
+++ b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
@@ -39,6 +39,9 @@ properties:
resets:
maxItems: 1
+ iommus:
+ maxItems: 1
+
ports:
type: object
description: |
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
index 78b1361dfbb9..9ce78a7b117d 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
@@ -161,6 +161,7 @@ video-codec@1c0e000 {
resets = <&ccu RST_BUS_VE>;
interrupts = <GIC_SPI 89 IRQ_TYPE_LEVEL_HIGH>;
allwinner,sram = <&ve_sram 1>;
+ iommus = <&iommu 3>;
};
gpu: gpu@1800000 {
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage
2020-06-28 18:08 [PATCH] arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage Maxime Ripard
@ 2020-07-09 16:39 ` Rob Herring
0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring @ 2020-07-09 16:39 UTC (permalink / raw)
To: Maxime Ripard
Cc: Chen-Yu Tsai, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE
On Sun, Jun 28, 2020 at 12:10 PM Maxime Ripard <maxime@cerno.tech> wrote:
>
> Now that the IOMMU driver has been introduced, it prevents any access from
> a DMA master going through it that hasn't properly mapped the pages, and
> that link is set up through the iommus property.
>
> Unfortunately we forgot to add that property to the video engine node when
> adding the IOMMU node, so now any DMA access is broken.
>
> Fixes: b3a0a2f910c7 ("arm64: dts: allwinner: h6: Add IOMMU")
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> ---
> .../bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml | 3 +++
> arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 1 +
> 2 files changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
> index 1dee641e3ea1..ef624327e33e 100644
> --- a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
> +++ b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
> @@ -39,6 +39,9 @@ properties:
> resets:
> maxItems: 1
>
> + iommus:
> + maxItems: 1
> +
This is breaking linux-next because there is already an 'iommus' property.
Rob
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-07-09 16:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-28 18:08 [PATCH] arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage Maxime Ripard
2020-07-09 16:39 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).