[PATCH 0/5] Enable DRA7 processor MMU DT nodes

[PATCH 0/5] Enable DRA7 processor MMU DT nodes

[Suman Anna]

Hi Tony,

The following series enables the DSP and IPU MMUs on various
TI DRA7xx/AM57xx boards. This is part 2 of the patches for
enabling the MMUs on DRA7 SoCs (OMAP4 and OMAP5 are already
supported on mainline).

Patches are baselined on your for-next branch with HEAD of
a70cb93b6c2f ("Merge branch 'omap-for-v4.14/dt-v3' into for-next").

I have tested the patches on current TI DRA7xx/AM57xx boards (DRA7,
DRA72, DRA71, DRA76 EVMs, AM571x & AM572x IDKs and AM57xx BeagleBoard-X15)
as well as the OMAP4 Panda and OMAP5 uEVM. The boards can boot fine
with just these patches, but requires the hwmod patches to be able
to use them (otherwise get the failures from _od_fail_runtime_{resume/suspend}
functions). There are no consumer devices for these yet, and they
just probe but are not clocked/enabled by default, so the current
System suspend is not affected.

regards
Suman

Suman Anna (5):
  ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes
  ARM: dts: dra72-evm-common: Enable relevant IPU and DSP MMU nodes
  ARM: dts: beagle-x15-common: Enable relevant IPU and DSP MMU nodes
  ARM: dts: am572x-idk: Enable relevant IPU and DSP MMU nodes
  ARM: dts: am571x-idk: Enable relevant IPU and DSP MMU nodes

 arch/arm/boot/dts/am571x-idk.dts                | 16 ++++++++++++++++
 arch/arm/boot/dts/am572x-idk.dts                | 24 ++++++++++++++++++++++++
 arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi | 24 ++++++++++++++++++++++++
 arch/arm/boot/dts/dra7-evm-common.dtsi          | 24 ++++++++++++++++++++++++
 arch/arm/boot/dts/dra72-evm-common.dtsi         | 16 ++++++++++++++++
 5 files changed, 104 insertions(+)

-- 
2.13.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 0/5] Enable DRA7 processor MMU DT nodes

[Suman Anna]

Hi Tony,

The following series enables the DSP and IPU MMUs on various
TI DRA7xx/AM57xx boards. This is part 2 of the patches for
enabling the MMUs on DRA7 SoCs (OMAP4 and OMAP5 are already
supported on mainline).

Patches are baselined on your for-next branch with HEAD of
a70cb93b6c2f ("Merge branch 'omap-for-v4.14/dt-v3' into for-next").

I have tested the patches on current TI DRA7xx/AM57xx boards (DRA7,
DRA72, DRA71, DRA76 EVMs, AM571x & AM572x IDKs and AM57xx BeagleBoard-X15)
as well as the OMAP4 Panda and OMAP5 uEVM. The boards can boot fine
with just these patches, but requires the hwmod patches to be able
to use them (otherwise get the failures from _od_fail_runtime_{resume/suspend}
functions). There are no consumer devices for these yet, and they
just probe but are not clocked/enabled by default, so the current
System suspend is not affected.

regards
Suman

Suman Anna (5):
  ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes
  ARM: dts: dra72-evm-common: Enable relevant IPU and DSP MMU nodes
  ARM: dts: beagle-x15-common: Enable relevant IPU and DSP MMU nodes
  ARM: dts: am572x-idk: Enable relevant IPU and DSP MMU nodes
  ARM: dts: am571x-idk: Enable relevant IPU and DSP MMU nodes

 arch/arm/boot/dts/am571x-idk.dts                | 16 ++++++++++++++++
 arch/arm/boot/dts/am572x-idk.dts                | 24 ++++++++++++++++++++++++
 arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi | 24 ++++++++++++++++++++++++
 arch/arm/boot/dts/dra7-evm-common.dtsi          | 24 ++++++++++++++++++++++++
 arch/arm/boot/dts/dra72-evm-common.dtsi         | 16 ++++++++++++++++
 5 files changed, 104 insertions(+)

-- 
2.13.1

[PATCH 1/5] ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
DSP2 have all been enabled. All these nodes are relevant and
valid for both the DRA7 EVM and DRA76 EVM boards, so are added
in the dra7 common evm dtsi file.

Signed-off-by: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
---
 arch/arm/boot/dts/dra7-evm-common.dtsi | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/boot/dts/dra7-evm-common.dtsi b/arch/arm/boot/dts/dra7-evm-common.dtsi
index 343e95f9a001..01a5d4e0b044 100644
--- a/arch/arm/boot/dts/dra7-evm-common.dtsi
+++ b/arch/arm/boot/dts/dra7-evm-common.dtsi
@@ -256,3 +256,27 @@
 		status = "okay";
 	};
 };
+
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu0_dsp2 {
+	status = "okay";
+};
+
+&mmu1_dsp2 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
-- 
2.13.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 1/5] ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
DSP2 have all been enabled. All these nodes are relevant and
valid for both the DRA7 EVM and DRA76 EVM boards, so are added
in the dra7 common evm dtsi file.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 arch/arm/boot/dts/dra7-evm-common.dtsi | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/boot/dts/dra7-evm-common.dtsi b/arch/arm/boot/dts/dra7-evm-common.dtsi
index 343e95f9a001..01a5d4e0b044 100644
--- a/arch/arm/boot/dts/dra7-evm-common.dtsi
+++ b/arch/arm/boot/dts/dra7-evm-common.dtsi
@@ -256,3 +256,27 @@
 		status = "okay";
 	};
 };
+
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu0_dsp2 {
+	status = "okay";
+};
+
+&mmu1_dsp2 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
-- 
2.13.1

[PATCH 2/5] ARM: dts: dra72-evm-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2 and DSP1 have
all been enabled. All these nodes are relevant and valid for both
the DRA72x EVM variants and the DRA71 EVM, so are added in the common
dts file. DSP2 MMU nodes are not present on DRA72x/DRA71x SoCs.

Signed-off-by: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
---
 arch/arm/boot/dts/dra72-evm-common.dtsi | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/arch/arm/boot/dts/dra72-evm-common.dtsi b/arch/arm/boot/dts/dra72-evm-common.dtsi
index 2e485a13dfd7..a72bcfd032d3 100644
--- a/arch/arm/boot/dts/dra72-evm-common.dtsi
+++ b/arch/arm/boot/dts/dra72-evm-common.dtsi
@@ -564,6 +564,22 @@
 	};
 };
 
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
+
 &pcie1_rc {
 	status = "okay";
 };
-- 
2.13.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/5] ARM: dts: dra72-evm-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2 and DSP1 have
all been enabled. All these nodes are relevant and valid for both
the DRA72x EVM variants and the DRA71 EVM, so are added in the common
dts file. DSP2 MMU nodes are not present on DRA72x/DRA71x SoCs.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 arch/arm/boot/dts/dra72-evm-common.dtsi | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/arch/arm/boot/dts/dra72-evm-common.dtsi b/arch/arm/boot/dts/dra72-evm-common.dtsi
index 2e485a13dfd7..a72bcfd032d3 100644
--- a/arch/arm/boot/dts/dra72-evm-common.dtsi
+++ b/arch/arm/boot/dts/dra72-evm-common.dtsi
@@ -564,6 +564,22 @@
 	};
 };
 
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
+
 &pcie1_rc {
 	status = "okay";
 };
-- 
2.13.1

[PATCH 3/5] ARM: dts: beagle-x15-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2, DSP1 and DSP2
have all been enabled. All these nodes are relevant and valid for
all the AM57xx BeagleBoard-X15 board variants, so are added in the
common dtsi file.

Signed-off-by: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
---
 arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi b/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi
index 49aeecd312b4..be9640b6e225 100644
--- a/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi
+++ b/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi
@@ -587,3 +587,27 @@
 		status = "okay";
 	};
 };
+
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu0_dsp2 {
+	status = "okay";
+};
+
+&mmu1_dsp2 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
-- 
2.13.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 3/5] ARM: dts: beagle-x15-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2, DSP1 and DSP2
have all been enabled. All these nodes are relevant and valid for
all the AM57xx BeagleBoard-X15 board variants, so are added in the
common dtsi file.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi b/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi
index 49aeecd312b4..be9640b6e225 100644
--- a/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi
+++ b/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi
@@ -587,3 +587,27 @@
 		status = "okay";
 	};
 };
+
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu0_dsp2 {
+	status = "okay";
+};
+
+&mmu1_dsp2 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
-- 
2.13.1

[PATCH 4/5] ARM: dts: am572x-idk: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
DSP2 have all been enabled. All these nodes are relevant and
valid for the AM572x IDK board.

Signed-off-by: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
---
 arch/arm/boot/dts/am572x-idk.dts | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/boot/dts/am572x-idk.dts b/arch/arm/boot/dts/am572x-idk.dts
index a578fe97ba3b..0f3be0de82b7 100644
--- a/arch/arm/boot/dts/am572x-idk.dts
+++ b/arch/arm/boot/dts/am572x-idk.dts
@@ -127,3 +127,27 @@
 		status = "okay";
 	};
 };
+
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu0_dsp2 {
+	status = "okay";
+};
+
+&mmu1_dsp2 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
-- 
2.13.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 4/5] ARM: dts: am572x-idk: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
DSP2 have all been enabled. All these nodes are relevant and
valid for the AM572x IDK board.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 arch/arm/boot/dts/am572x-idk.dts | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/boot/dts/am572x-idk.dts b/arch/arm/boot/dts/am572x-idk.dts
index a578fe97ba3b..0f3be0de82b7 100644
--- a/arch/arm/boot/dts/am572x-idk.dts
+++ b/arch/arm/boot/dts/am572x-idk.dts
@@ -127,3 +127,27 @@
 		status = "okay";
 	};
 };
+
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu0_dsp2 {
+	status = "okay";
+};
+
+&mmu1_dsp2 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
-- 
2.13.1

[PATCH 5/5] ARM: dts: am571x-idk: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2 and DSP1
have all been enabled. All these nodes are relevant and valid
for the AM571x IDK board. DSP2 MMU nodes are not present on
AM571x SoCs.

Signed-off-by: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
---
 arch/arm/boot/dts/am571x-idk.dts | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/arch/arm/boot/dts/am571x-idk.dts b/arch/arm/boot/dts/am571x-idk.dts
index debf9464403e..ea9f695e1282 100644
--- a/arch/arm/boot/dts/am571x-idk.dts
+++ b/arch/arm/boot/dts/am571x-idk.dts
@@ -91,6 +91,22 @@
 	};
 };
 
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
+
 &pcie1_rc {
 	status = "okay";
 	gpios = <&gpio3 23 GPIO_ACTIVE_HIGH>;
-- 
2.13.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 5/5] ARM: dts: am571x-idk: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

The MMU nodes for the remote processors IPU1, IPU2 and DSP1
have all been enabled. All these nodes are relevant and valid
for the AM571x IDK board. DSP2 MMU nodes are not present on
AM571x SoCs.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 arch/arm/boot/dts/am571x-idk.dts | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/arch/arm/boot/dts/am571x-idk.dts b/arch/arm/boot/dts/am571x-idk.dts
index debf9464403e..ea9f695e1282 100644
--- a/arch/arm/boot/dts/am571x-idk.dts
+++ b/arch/arm/boot/dts/am571x-idk.dts
@@ -91,6 +91,22 @@
 	};
 };
 
+&mmu0_dsp1 {
+	status = "okay";
+};
+
+&mmu1_dsp1 {
+	status = "okay";
+};
+
+&mmu_ipu1 {
+	status = "okay";
+};
+
+&mmu_ipu2 {
+	status = "okay";
+};
+
 &pcie1_rc {
 	status = "okay";
 	gpios = <&gpio3 23 GPIO_ACTIVE_HIGH>;
-- 
2.13.1

Re: [PATCH 1/5] ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes

[Tony Lindgren]

* Suman Anna <s-anna-l0cyMroinI0@public.gmane.org> [170821 16:52]:
> The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
> DSP2 have all been enabled. All these nodes are relevant and
> valid for both the DRA7 EVM and DRA76 EVM boards, so are added
> in the dra7 common evm dtsi file.
> 
> Signed-off-by: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
> ---
>  arch/arm/boot/dts/dra7-evm-common.dtsi | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/dra7-evm-common.dtsi b/arch/arm/boot/dts/dra7-evm-common.dtsi
> index 343e95f9a001..01a5d4e0b044 100644
> --- a/arch/arm/boot/dts/dra7-evm-common.dtsi
> +++ b/arch/arm/boot/dts/dra7-evm-common.dtsi
> @@ -256,3 +256,27 @@
>  		status = "okay";
>  	};
>  };
> +
> +&mmu0_dsp1 {
> +	status = "okay";
> +};
> +
> +&mmu1_dsp1 {
> +	status = "okay";
> +};
> +
> +&mmu0_dsp2 {
> +	status = "okay";
> +};
> +
> +&mmu1_dsp2 {
> +	status = "okay";
> +};
> +
> +&mmu_ipu1 {
> +	status = "okay";
> +};
> +
> +&mmu_ipu2 {
> +	status = "okay";
> +};

This is getting quite silly.. The default is "okay" so just remove the useless
"disabled" from the common file and all these patches are no longer needed.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 1/5] ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes

[Tony Lindgren]

* Suman Anna <s-anna@ti.com> [170821 16:52]:
> The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
> DSP2 have all been enabled. All these nodes are relevant and
> valid for both the DRA7 EVM and DRA76 EVM boards, so are added
> in the dra7 common evm dtsi file.
> 
> Signed-off-by: Suman Anna <s-anna@ti.com>
> ---
>  arch/arm/boot/dts/dra7-evm-common.dtsi | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/dra7-evm-common.dtsi b/arch/arm/boot/dts/dra7-evm-common.dtsi
> index 343e95f9a001..01a5d4e0b044 100644
> --- a/arch/arm/boot/dts/dra7-evm-common.dtsi
> +++ b/arch/arm/boot/dts/dra7-evm-common.dtsi
> @@ -256,3 +256,27 @@
>  		status = "okay";
>  	};
>  };
> +
> +&mmu0_dsp1 {
> +	status = "okay";
> +};
> +
> +&mmu1_dsp1 {
> +	status = "okay";
> +};
> +
> +&mmu0_dsp2 {
> +	status = "okay";
> +};
> +
> +&mmu1_dsp2 {
> +	status = "okay";
> +};
> +
> +&mmu_ipu1 {
> +	status = "okay";
> +};
> +
> +&mmu_ipu2 {
> +	status = "okay";
> +};

This is getting quite silly.. The default is "okay" so just remove the useless
"disabled" from the common file and all these patches are no longer needed.

Regards,

Tony

Re: [PATCH 1/5] ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

Hi Tony,

On 10/11/2017 01:22 PM, Tony Lindgren wrote:
> * Suman Anna <s-anna-l0cyMroinI0@public.gmane.org> [170821 16:52]:
>> The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
>> DSP2 have all been enabled. All these nodes are relevant and
>> valid for both the DRA7 EVM and DRA76 EVM boards, so are added
>> in the dra7 common evm dtsi file.
>>
>> Signed-off-by: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
>> ---
>>  arch/arm/boot/dts/dra7-evm-common.dtsi | 24 ++++++++++++++++++++++++
>>  1 file changed, 24 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/dra7-evm-common.dtsi b/arch/arm/boot/dts/dra7-evm-common.dtsi
>> index 343e95f9a001..01a5d4e0b044 100644
>> --- a/arch/arm/boot/dts/dra7-evm-common.dtsi
>> +++ b/arch/arm/boot/dts/dra7-evm-common.dtsi
>> @@ -256,3 +256,27 @@
>>  		status = "okay";
>>  	};
>>  };
>> +
>> +&mmu0_dsp1 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu1_dsp1 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu0_dsp2 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu1_dsp2 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu_ipu1 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu_ipu2 {
>> +	status = "okay";
>> +};
> 
> This is getting quite silly.. The default is "okay" so just remove the useless
> "disabled" from the common file and all these patches are no longer needed.

Yeah, agreed. Original intention was that some of the other remoteproc
related nodes can be customized per board, but there's nothing of that
sort on iommu nodes. Will post a new series. Thanks for the review.

regards
Suman
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 1/5] ARM: dts: dra7-evm-common: Enable relevant IPU and DSP MMU nodes

[Suman Anna]

Hi Tony,

On 10/11/2017 01:22 PM, Tony Lindgren wrote:
> * Suman Anna <s-anna@ti.com> [170821 16:52]:
>> The MMU nodes for the remote processors IPU1, IPU2, DSP1 and
>> DSP2 have all been enabled. All these nodes are relevant and
>> valid for both the DRA7 EVM and DRA76 EVM boards, so are added
>> in the dra7 common evm dtsi file.
>>
>> Signed-off-by: Suman Anna <s-anna@ti.com>
>> ---
>>  arch/arm/boot/dts/dra7-evm-common.dtsi | 24 ++++++++++++++++++++++++
>>  1 file changed, 24 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/dra7-evm-common.dtsi b/arch/arm/boot/dts/dra7-evm-common.dtsi
>> index 343e95f9a001..01a5d4e0b044 100644
>> --- a/arch/arm/boot/dts/dra7-evm-common.dtsi
>> +++ b/arch/arm/boot/dts/dra7-evm-common.dtsi
>> @@ -256,3 +256,27 @@
>>  		status = "okay";
>>  	};
>>  };
>> +
>> +&mmu0_dsp1 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu1_dsp1 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu0_dsp2 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu1_dsp2 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu_ipu1 {
>> +	status = "okay";
>> +};
>> +
>> +&mmu_ipu2 {
>> +	status = "okay";
>> +};
> 
> This is getting quite silly.. The default is "okay" so just remove the useless
> "disabled" from the common file and all these patches are no longer needed.

Yeah, agreed. Original intention was that some of the other remoteproc
related nodes can be customized per board, but there's nothing of that
sort on iommu nodes. Will post a new series. Thanks for the review.

regards
Suman