diff mbox

[v3,1/2] ARM: dts: Fix sysreg node name in exynos4.dtsi

Message ID 1383632302-23535-1-git-send-email-sachin.kamat@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sachin Kamat Nov. 5, 2013, 6:18 a.m. UTC
Fix the name as per DT node naming convention.
- rename the node to syscon which is a more generic name.
- append the register value to the node name.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Reviewed-by: Tomasz Figa <t.figa@samsung.com>
---
Changes since v2:
None.
---
 arch/arm/boot/dts/exynos4.dtsi |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

On 05/11/13 07:18, Sachin Kamat wrote:
> Fix the name as per DT node naming convention.
> - rename the node to syscon which is a more generic name.
> - append the register value to the node name.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> Reviewed-by: Tomasz Figa <t.figa@samsung.com>

Thanks Sachin.

Reviewed-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
--
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
kgene@kernel.org Nov. 12, 2013, 10:40 a.m. UTC | #2
Sachin Kamat wrote:
> 
> Fix the name as per DT node naming convention.
> - rename the node to syscon which is a more generic name.
> - append the register value to the node name.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> Reviewed-by: Tomasz Figa <t.figa@samsung.com>
> ---
> Changes since v2:
> None.
> ---
>  arch/arm/boot/dts/exynos4.dtsi |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/exynos4.dtsi
> b/arch/arm/boot/dts/exynos4.dtsi
> index cb0f70cc552b..b7378154c0a1 100644
> --- a/arch/arm/boot/dts/exynos4.dtsi
> +++ b/arch/arm/boot/dts/exynos4.dtsi
> @@ -99,7 +99,7 @@
>  		reg = <0x10440000 0x1000>;
>  	};
> 
> -	sys_reg: sysreg {
> +	sys_reg: syscon@10010000 {
>  		compatible = "samsung,exynos4-sysreg", "syscon";
>  		reg = <0x10010000 0x400>;
>  	};
> --
> 1.7.9.5

Looks good to me, thanks for this patch and reviews from Sylwester and
Tomasz.

Applied.
- Kukjin

--
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
Sachin Kamat Dec. 2, 2013, 4:21 a.m. UTC | #3
On 12 November 2013 16:10, Kukjin Kim <kgene@kernel.org> wrote:
> Sachin Kamat wrote:
>>
>> Fix the name as per DT node naming convention.
>> - rename the node to syscon which is a more generic name.
>> - append the register value to the node name.
>>
>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>> Reviewed-by: Tomasz Figa <t.figa@samsung.com>
>> ---
>> Changes since v2:
>> None.
>> ---
>>  arch/arm/boot/dts/exynos4.dtsi |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos4.dtsi
>> b/arch/arm/boot/dts/exynos4.dtsi
>> index cb0f70cc552b..b7378154c0a1 100644
>> --- a/arch/arm/boot/dts/exynos4.dtsi
>> +++ b/arch/arm/boot/dts/exynos4.dtsi
>> @@ -99,7 +99,7 @@
>>               reg = <0x10440000 0x1000>;
>>       };
>>
>> -     sys_reg: sysreg {
>> +     sys_reg: syscon@10010000 {
>>               compatible = "samsung,exynos4-sysreg", "syscon";
>>               reg = <0x10010000 0x400>;
>>       };
>> --
>> 1.7.9.5
>
> Looks good to me, thanks for this patch and reviews from Sylwester and
> Tomasz.
>
> Applied.

Kukjin,

I don't see these patches in your tree yet. Please check.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
index cb0f70cc552b..b7378154c0a1 100644
--- a/arch/arm/boot/dts/exynos4.dtsi
+++ b/arch/arm/boot/dts/exynos4.dtsi
@@ -99,7 +99,7 @@ 
 		reg = <0x10440000 0x1000>;
 	};
 
-	sys_reg: sysreg {
+	sys_reg: syscon@10010000 {
 		compatible = "samsung,exynos4-sysreg", "syscon";
 		reg = <0x10010000 0x400>;
 	};