diff mbox series

arm64: dts: renesas: r8a77961: Add iommus to SDHI/MMC nodes

Message ID 20210714090843.438076-1-yoshihiro.shimoda.uh@renesas.com (mailing list archive)
State Superseded
Delegated to: Geert Uytterhoeven
Headers show
Series arm64: dts: renesas: r8a77961: Add iommus to SDHI/MMC nodes | expand

Commit Message

Yoshihiro Shimoda July 14, 2021, 9:08 a.m. UTC
This patch adds iommus properties to the r8a77961 sdhi nodes.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a77961.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Yoshihiro Shimoda July 20, 2021, 10:27 a.m. UTC | #1
Hi Geert-san,

> From: Yoshihiro Shimoda, Sent: Wednesday, July 14, 2021 6:09 PM
> 
> This patch adds iommus properties to the r8a77961 sdhi nodes.
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

I realized that the r8a77961.dtsi is missing iommus properties on dmacN and avb nodes.
Can I make incremental patches? Or, can I add these properties into this patch as v2?

Best regards,
Yoshihiro Shimoda

> ---
>  arch/arm64/boot/dts/renesas/r8a77961.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77961.dtsi b/arch/arm64/boot/dts/renesas/r8a77961.dtsi
> index 91b501e0121e..6e80fee23891 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77961.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a77961.dtsi
> @@ -2280,6 +2280,7 @@ sdhi0: mmc@ee100000 {
>  			max-frequency = <200000000>;
>  			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
>  			resets = <&cpg 314>;
> +			iommus = <&ipmmu_ds1 32>;
>  			status = "disabled";
>  		};
> 
> @@ -2292,6 +2293,7 @@ sdhi1: mmc@ee120000 {
>  			max-frequency = <200000000>;
>  			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
>  			resets = <&cpg 313>;
> +			iommus = <&ipmmu_ds1 33>;
>  			status = "disabled";
>  		};
> 
> @@ -2304,6 +2306,7 @@ sdhi2: mmc@ee140000 {
>  			max-frequency = <200000000>;
>  			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
>  			resets = <&cpg 312>;
> +			iommus = <&ipmmu_ds1 34>;
>  			status = "disabled";
>  		};
> 
> @@ -2316,6 +2319,7 @@ sdhi3: mmc@ee160000 {
>  			max-frequency = <200000000>;
>  			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
>  			resets = <&cpg 311>;
> +			iommus = <&ipmmu_ds1 35>;
>  			status = "disabled";
>  		};
> 
> --
> 2.25.1
Geert Uytterhoeven July 20, 2021, 10:33 a.m. UTC | #2
Hi Shimoda-san,

On Tue, Jul 20, 2021 at 12:27 PM Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> > From: Yoshihiro Shimoda, Sent: Wednesday, July 14, 2021 6:09 PM
> >
> > This patch adds iommus properties to the r8a77961 sdhi nodes.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>
> I realized that the r8a77961.dtsi is missing iommus properties on dmacN and avb nodes.
> Can I make incremental patches? Or, can I add these properties into this patch as v2?

Feel free to send a v2 (with updated summary/description), as I haven't
reviewed this patch yet.

Gr{oetje,eeting}s,

                        Geert
Yoshihiro Shimoda July 21, 2021, 8:16 a.m. UTC | #3
Hi Geert-san,

> From: Geert Uytterhoeven, Sent: Tuesday, July 20, 2021 7:34 PM
> 
> On Tue, Jul 20, 2021 at 12:27 PM Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@renesas.com> wrote:
> > > From: Yoshihiro Shimoda, Sent: Wednesday, July 14, 2021 6:09 PM
> > >
> > > This patch adds iommus properties to the r8a77961 sdhi nodes.
> > >
> > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> >
> > I realized that the r8a77961.dtsi is missing iommus properties on dmacN and avb nodes.
> > Can I make incremental patches? Or, can I add these properties into this patch as v2?
> 
> Feel free to send a v2 (with updated summary/description), as I haven't
> reviewed this patch yet.

Thank you for your reply! I got it. I'll send a v2 patch with updated summary/description later.

Best regards,
Yoshihiro Shimoda
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/renesas/r8a77961.dtsi b/arch/arm64/boot/dts/renesas/r8a77961.dtsi
index 91b501e0121e..6e80fee23891 100644
--- a/arch/arm64/boot/dts/renesas/r8a77961.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77961.dtsi
@@ -2280,6 +2280,7 @@  sdhi0: mmc@ee100000 {
 			max-frequency = <200000000>;
 			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
 			resets = <&cpg 314>;
+			iommus = <&ipmmu_ds1 32>;
 			status = "disabled";
 		};
 
@@ -2292,6 +2293,7 @@  sdhi1: mmc@ee120000 {
 			max-frequency = <200000000>;
 			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
 			resets = <&cpg 313>;
+			iommus = <&ipmmu_ds1 33>;
 			status = "disabled";
 		};
 
@@ -2304,6 +2306,7 @@  sdhi2: mmc@ee140000 {
 			max-frequency = <200000000>;
 			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
 			resets = <&cpg 312>;
+			iommus = <&ipmmu_ds1 34>;
 			status = "disabled";
 		};
 
@@ -2316,6 +2319,7 @@  sdhi3: mmc@ee160000 {
 			max-frequency = <200000000>;
 			power-domains = <&sysc R8A77961_PD_ALWAYS_ON>;
 			resets = <&cpg 311>;
+			iommus = <&ipmmu_ds1 35>;
 			status = "disabled";
 		};