diff mbox

[1/1] ARM: kirkwood-ds112.dts: use unique machine name

Message ID 1454870066-3353-1-git-send-email-xypron.glpk@gmx.de (mailing list archive)
State New, archived
Headers show

Commit Message

Heinrich Schuchardt Feb. 7, 2016, 6:34 p.m. UTC
Downstream packages like Debian flash-kernel use
/proc/device-tree/model
to determine which dtb file to install.

Hence each dts in the Linux kernel should provide a unique model
identifier.

Commit 2d0a7addbd10 ("ARM: Kirkwood: Add support for many Synology NAS
devices") created the new files kirkwood-ds111.dts and kirkwood-ds112.dts
using the same model identifier.

This patch provides a unique model identifier for the
Synology DiskStation DS112.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 arch/arm/boot/dts/kirkwood-ds112.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Lunn Feb. 7, 2016, 7:26 p.m. UTC | #1
On Sun, Feb 07, 2016 at 07:34:26PM +0100, Heinrich Schuchardt wrote:
> Downstream packages like Debian flash-kernel use
> /proc/device-tree/model
> to determine which dtb file to install.
> 
> Hence each dts in the Linux kernel should provide a unique model
> identifier.
> 
> Commit 2d0a7addbd10 ("ARM: Kirkwood: Add support for many Synology NAS
> devices") created the new files kirkwood-ds111.dts and kirkwood-ds112.dts
> using the same model identifier.
> 
> This patch provides a unique model identifier for the
> Synology DiskStation DS112.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>

Fixes: 2d0a7addbd10 ("ARM: Kirkwood: Add support for many Synology NAS devices")

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Thanks
	Andrew	     

> ---
>  arch/arm/boot/dts/kirkwood-ds112.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/kirkwood-ds112.dts b/arch/arm/boot/dts/kirkwood-ds112.dts
> index bf4143c..b84af3d 100644
> --- a/arch/arm/boot/dts/kirkwood-ds112.dts
> +++ b/arch/arm/boot/dts/kirkwood-ds112.dts
> @@ -14,7 +14,7 @@
>  #include "kirkwood-synology.dtsi"
>  
>  / {
> -	model = "Synology DS111";
> +	model = "Synology DS112";
>  	compatible = "synology,ds111", "marvell,kirkwood";
>  
>  	memory {
> -- 
> 2.1.4
>
Gregory CLEMENT Feb. 8, 2016, 3:58 p.m. UTC | #2
Hi Heinrich,
 
 On dim., févr. 07 2016, Andrew Lunn <andrew@lunn.ch> wrote:

> On Sun, Feb 07, 2016 at 07:34:26PM +0100, Heinrich Schuchardt wrote:
>> Downstream packages like Debian flash-kernel use
>> /proc/device-tree/model
>> to determine which dtb file to install.
>> 
>> Hence each dts in the Linux kernel should provide a unique model
>> identifier.
>> 
>> Commit 2d0a7addbd10 ("ARM: Kirkwood: Add support for many Synology NAS
>> devices") created the new files kirkwood-ds111.dts and kirkwood-ds112.dts
>> using the same model identifier.
>> 
>> This patch provides a unique model identifier for the
>> Synology DiskStation DS112.
>> 
>> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>
> Fixes: 2d0a7addbd10 ("ARM: Kirkwood: Add support for many Synology NAS devices")
>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>

Applied on mvebu/fixes with the Fixes tag and Andrew Reviewed-by tag.

Thanks,

Gregory



>
>     Thanks
> 	Andrew	     
>
>> ---
>>  arch/arm/boot/dts/kirkwood-ds112.dts | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/arch/arm/boot/dts/kirkwood-ds112.dts b/arch/arm/boot/dts/kirkwood-ds112.dts
>> index bf4143c..b84af3d 100644
>> --- a/arch/arm/boot/dts/kirkwood-ds112.dts
>> +++ b/arch/arm/boot/dts/kirkwood-ds112.dts
>> @@ -14,7 +14,7 @@
>>  #include "kirkwood-synology.dtsi"
>>  
>>  / {
>> -	model = "Synology DS111";
>> +	model = "Synology DS112";
>>  	compatible = "synology,ds111", "marvell,kirkwood";
>>  
>>  	memory {
>> -- 
>> 2.1.4
>>
diff mbox

Patch

diff --git a/arch/arm/boot/dts/kirkwood-ds112.dts b/arch/arm/boot/dts/kirkwood-ds112.dts
index bf4143c..b84af3d 100644
--- a/arch/arm/boot/dts/kirkwood-ds112.dts
+++ b/arch/arm/boot/dts/kirkwood-ds112.dts
@@ -14,7 +14,7 @@ 
 #include "kirkwood-synology.dtsi"
 
 / {
-	model = "Synology DS111";
+	model = "Synology DS112";
 	compatible = "synology,ds111", "marvell,kirkwood";
 
 	memory {