[2/3] ARM: dts: exynos: Remove syscon compatible from chipid node on Exynos5
diff mbox series

Message ID 20191028152050.10220-2-s.nawrocki@samsung.com
State Accepted
Headers show
Series
  • [1/3] soc: samsung: Drop "syscon" compatible requirement from the chipid driver
Related show

Commit Message

Sylwester Nawrocki Oct. 28, 2019, 3:20 p.m. UTC
The "syscon" compatible string was introduced in commit ("cdcce1ee977b
ARM: dts: exynos: Add "syscon" compatible string to chipid node on Exynos5")
to allow sharing of the CHIPID IO region between multiple drivers.
However, such sharing can be also done without an additional compatible
so remove the syscon entry.

Suggested-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
---
 arch/arm/boot/dts/exynos5.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Krzysztof Kozlowski Dec. 11, 2019, 6:20 p.m. UTC | #1
On Mon, Oct 28, 2019 at 04:20:49PM +0100, Sylwester Nawrocki wrote:
> The "syscon" compatible string was introduced in commit ("cdcce1ee977b
> ARM: dts: exynos: Add "syscon" compatible string to chipid node on Exynos5")
> to allow sharing of the CHIPID IO region between multiple drivers.
> However, such sharing can be also done without an additional compatible
> so remove the syscon entry.
> 
> Suggested-by: Krzysztof Kozlowski <krzk@kernel.org>
> Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
> ---
>  arch/arm/boot/dts/exynos5.dtsi | 2 +-

Thanks, applied.

Best regards,
Krzysztof

Patch
diff mbox series

diff --git a/arch/arm/boot/dts/exynos5.dtsi b/arch/arm/boot/dts/exynos5.dtsi
index 4801ca759feb..22eb951c614c 100644
--- a/arch/arm/boot/dts/exynos5.dtsi
+++ b/arch/arm/boot/dts/exynos5.dtsi
@@ -36,7 +36,7 @@ 
 		ranges;
 
 		chipid: chipid@10000000 {
-			compatible = "samsung,exynos4210-chipid", "syscon";
+			compatible = "samsung,exynos4210-chipid";
 			reg = <0x10000000 0x100>;
 		};