ARM: dts: exynos: Fix IOMMU support for GScaler devices on Exynos5250

Message ID 20180302160742.16947-1-m.szyprowski@samsung.com
State New
Headers show
Series
  • ARM: dts: exynos: Fix IOMMU support for GScaler devices on Exynos5250
Related show

Commit Message

Marek Szyprowski March 2, 2018, 4:07 p.m.
The proper name for the property, which assign given device to IOMMU is
'iommus', not 'iommu'. Fix incorrect name and let all GScaler devices
to be properly handled when IOMMU support is enabled.

Reported-by: Andrzej Hajda <a.hajda@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

Fixes: 6cbfdd73a94f ("ARM: dts: add sysmmu nodes for exynos5250")
CC: stable@vger.kernel.org # v4.8+
---
This issue has been introduced in kernel v4.2-rc1, but this patch applies
cleanly only on v4.8+ kernels.
---
 arch/arm/boot/dts/exynos5250.dtsi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

-- 
2.15.0

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

Comments

Krzysztof Kozlowski March 5, 2018, 6:14 p.m. | #1
On Fri, Mar 02, 2018 at 05:07:42PM +0100, Marek Szyprowski wrote:
> The proper name for the property, which assign given device to IOMMU is

> 'iommus', not 'iommu'. Fix incorrect name and let all GScaler devices

> to be properly handled when IOMMU support is enabled.

> 

> Reported-by: Andrzej Hajda <a.hajda@samsung.com>

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

> Fixes: 6cbfdd73a94f ("ARM: dts: add sysmmu nodes for exynos5250")

> CC: stable@vger.kernel.org # v4.8+

> ---

> This issue has been introduced in kernel v4.2-rc1, but this patch applies

> cleanly only on v4.8+ kernels.

> ---

>  arch/arm/boot/dts/exynos5250.dtsi | 8 ++++----

>  1 file changed, 4 insertions(+), 4 deletions(-)

> 


Thanks, applied.

Best regards,
Krzysztof

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

Patch

diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index 56626d1a4235..cf89385e7888 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -655,7 +655,7 @@ 
 			power-domains = <&pd_gsc>;
 			clocks = <&clock CLK_GSCL0>;
 			clock-names = "gscl";
-			iommu = <&sysmmu_gsc0>;
+			iommus = <&sysmmu_gsc0>;
 		};
 
 		gsc_1:  gsc@13e10000 {
@@ -665,7 +665,7 @@ 
 			power-domains = <&pd_gsc>;
 			clocks = <&clock CLK_GSCL1>;
 			clock-names = "gscl";
-			iommu = <&sysmmu_gsc1>;
+			iommus = <&sysmmu_gsc1>;
 		};
 
 		gsc_2:  gsc@13e20000 {
@@ -675,7 +675,7 @@ 
 			power-domains = <&pd_gsc>;
 			clocks = <&clock CLK_GSCL2>;
 			clock-names = "gscl";
-			iommu = <&sysmmu_gsc2>;
+			iommus = <&sysmmu_gsc2>;
 		};
 
 		gsc_3:  gsc@13e30000 {
@@ -685,7 +685,7 @@ 
 			power-domains = <&pd_gsc>;
 			clocks = <&clock CLK_GSCL3>;
 			clock-names = "gscl";
-			iommu = <&sysmmu_gsc3>;
+			iommus = <&sysmmu_gsc3>;
 		};
 
 		hdmi: hdmi@14530000 {