Unverified Commit 62a8ccf3 authored Jun 28, 2020 by Maxime Ripard

arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage

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: b3a0a2f9 ("arm64: dts: allwinner: h6: Add IOMMU")
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Link: https://lore.kernel.org/r/20200628180804.79026-1-maxime@cerno.tech

parent 92025b90

Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml

+3 −0

Original line number	Diff line number	Diff line
		@@ -47,6 +47,9 @@ properties:
		$ref: /schemas/types.yaml#/definitions/phandle-array
		description: Phandle to the device SRAM

		iommus:
		maxItems: 1

		memory-region:
		description:
		CMA pool to use for buffers allocation instead of the default

arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi

+1 −0

Original line number	Diff line number	Diff line
		@@ -161,6 +161,7 @@
		resets = <&ccu RST_BUS_VE>;
		interrupts = <GIC_SPI 89 IRQ_TYPE_LEVEL_HIGH>;
		allwinner,sram = <&ve_sram 1>;
		iommus = <&iommu 3>;
		};

		gpu: gpu@1800000 {

Admin message