Hi Niklas,
Thank you for the patch.
On Thursday 21 January 2016 15:01:34 Niklas Söderlund wrote:
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> ---
> arch/arm/boot/dts/r8a7790.dtsi | 30 ++++++++++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
> diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
> index e07ae5d..23af349 100644
> --- a/arch/arm/boot/dts/r8a7790.dtsi
> +++ b/arch/arm/boot/dts/r8a7790.dtsi
> @@ -294,6 +294,21 @@
> power-domains = <&cpg_clocks>;
> #dma-cells = <1>;
> dma-channels = <15>;
> + iommus = <&ipmmu_ds 0>,
> + <&ipmmu_ds 1>,
> + <&ipmmu_ds 2>,
> + <&ipmmu_ds 3>,
> + <&ipmmu_ds 4>,
> + <&ipmmu_ds 5>,
> + <&ipmmu_ds 6>,
> + <&ipmmu_ds 7>,
> + <&ipmmu_ds 8>,
> + <&ipmmu_ds 9>,
> + <&ipmmu_ds 10>,
> + <&ipmmu_ds 11>,
> + <&ipmmu_ds 12>,
> + <&ipmmu_ds 13>,
> + <&ipmmu_ds 14>;
I wonder whether this should be done unconditionally. Whether to use the IOMMU
seems more like a policy decision to me.
> };
>
> dmac1: dma-controller@e6720000 {
> @@ -325,6 +340,21 @@
> power-domains = <&cpg_clocks>;
> #dma-cells = <1>;
> dma-channels = <15>;
> + iommus = <&ipmmu_ds 15>,
> + <&ipmmu_ds 16>,
> + <&ipmmu_ds 17>,
> + <&ipmmu_ds 18>,
> + <&ipmmu_ds 19>,
> + <&ipmmu_ds 20>,
> + <&ipmmu_ds 21>,
> + <&ipmmu_ds 22>,
> + <&ipmmu_ds 23>,
> + <&ipmmu_ds 24>,
> + <&ipmmu_ds 25>,
> + <&ipmmu_ds 26>,
> + <&ipmmu_ds 27>,
> + <&ipmmu_ds 28>,
> + <&ipmmu_ds 29>;
> };
>
> audma0: dma-controller@ec700000 {
--
Regards,
Laurent Pinchart
