All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arc: Add "model" properly in device tree description of all boards
@ 2016-08-15 12:56 ` Alexey Brodkin
  0 siblings, 0 replies; 4+ messages in thread
From: Alexey Brodkin @ 2016-08-15 12:56 UTC (permalink / raw)
  To: linux-snps-arc
  Cc: linux-kernel, Alexey Brodkin, Vineet Gupta, Jonas Gorski,
	Arnd Bergmann, Rob Herring, Christian Ruppert

As it was discussed quite some time ago (see
https://lkml.org/lkml/2015/11/5/862) it's a good practice to add
"model" property in .dts. Moreover as per ePAPR "model" property is
required and should look like "manufacturer,model" so we do here.

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
Cc: Vineet Gupta <vgupta@synopsys.com>
Cc: Jonas Gorski <jonas.gorski@gmail.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Rob Herring <robh@kernel.org>
Cc: Christian Ruppert <christian.ruppert@alitech.com>
---
 arch/arc/boot/dts/abilis_tb100_dvk.dts | 1 +
 arch/arc/boot/dts/abilis_tb101_dvk.dts | 1 +
 arch/arc/boot/dts/axs101.dts           | 1 +
 arch/arc/boot/dts/axs103.dts           | 1 +
 arch/arc/boot/dts/axs103_idu.dts       | 1 +
 arch/arc/boot/dts/nsim_700.dts         | 1 +
 arch/arc/boot/dts/nsim_hs.dts          | 1 +
 arch/arc/boot/dts/nsim_hs_idu.dts      | 1 +
 arch/arc/boot/dts/nsimosci.dts         | 1 +
 arch/arc/boot/dts/nsimosci_hs.dts      | 1 +
 arch/arc/boot/dts/nsimosci_hs_idu.dts  | 1 +
 arch/arc/boot/dts/vdk_hs38.dts         | 1 +
 arch/arc/boot/dts/vdk_hs38_smp.dts     | 1 +
 13 files changed, 13 insertions(+)

diff --git a/arch/arc/boot/dts/abilis_tb100_dvk.dts b/arch/arc/boot/dts/abilis_tb100_dvk.dts
index 3dd6ed9..3acf04d 100644
--- a/arch/arc/boot/dts/abilis_tb100_dvk.dts
+++ b/arch/arc/boot/dts/abilis_tb100_dvk.dts
@@ -24,6 +24,7 @@
 /include/ "abilis_tb100.dtsi"
 
 / {
+	model = "abilis,tb100";
 	chosen {
 		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
 	};
diff --git a/arch/arc/boot/dts/abilis_tb101_dvk.dts b/arch/arc/boot/dts/abilis_tb101_dvk.dts
index 1cf51c2..37d88c5 100644
--- a/arch/arc/boot/dts/abilis_tb101_dvk.dts
+++ b/arch/arc/boot/dts/abilis_tb101_dvk.dts
@@ -24,6 +24,7 @@
 /include/ "abilis_tb101.dtsi"
 
 / {
+	model = "abilis,tb101";
 	chosen {
 		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
 	};
diff --git a/arch/arc/boot/dts/axs101.dts b/arch/arc/boot/dts/axs101.dts
index 3f9b058..d9b9b9d 100644
--- a/arch/arc/boot/dts/axs101.dts
+++ b/arch/arc/boot/dts/axs101.dts
@@ -13,6 +13,7 @@
 /include/ "axs10x_mb.dtsi"
 
 / {
+	model = "snps,axs101";
 	compatible = "snps,axs101", "snps,arc-sdp";
 
 	chosen {
diff --git a/arch/arc/boot/dts/axs103.dts b/arch/arc/boot/dts/axs103.dts
index e6d0e31..ec7fb27 100644
--- a/arch/arc/boot/dts/axs103.dts
+++ b/arch/arc/boot/dts/axs103.dts
@@ -16,6 +16,7 @@
 /include/ "axs10x_mb.dtsi"
 
 / {
+	model = "snps,axs103";
 	compatible = "snps,axs103", "snps,arc-sdp";
 
 	chosen {
diff --git a/arch/arc/boot/dts/axs103_idu.dts b/arch/arc/boot/dts/axs103_idu.dts
index f999fef..070c297 100644
--- a/arch/arc/boot/dts/axs103_idu.dts
+++ b/arch/arc/boot/dts/axs103_idu.dts
@@ -16,6 +16,7 @@
 /include/ "axs10x_mb.dtsi"
 
 / {
+	model = "snps,axs103-smp";
 	compatible = "snps,axs103", "snps,arc-sdp";
 
 	chosen {
diff --git a/arch/arc/boot/dts/nsim_700.dts b/arch/arc/boot/dts/nsim_700.dts
index 6397051..ce0ccd20 100644
--- a/arch/arc/boot/dts/nsim_700.dts
+++ b/arch/arc/boot/dts/nsim_700.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton.dtsi"
 
 / {
+	model = "snps,nsim";
 	compatible = "snps,nsim";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/nsim_hs.dts b/arch/arc/boot/dts/nsim_hs.dts
index bf05fe5..fa6fde8 100644
--- a/arch/arc/boot/dts/nsim_hs.dts
+++ b/arch/arc/boot/dts/nsim_hs.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs.dtsi"
 
 / {
+	model = "snps,nsim";
 	compatible = "snps,nsim_hs";
 	#address-cells = <2>;
 	#size-cells = <2>;
diff --git a/arch/arc/boot/dts/nsim_hs_idu.dts b/arch/arc/boot/dts/nsim_hs_idu.dts
index 99eabe1..1f02808 100644
--- a/arch/arc/boot/dts/nsim_hs_idu.dts
+++ b/arch/arc/boot/dts/nsim_hs_idu.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs_idu.dtsi"
 
 / {
+	model = "snps,nsim-smp";
 	compatible = "snps,nsim_hs";
 	interrupt-parent = <&core_intc>;
 
diff --git a/arch/arc/boot/dts/nsimosci.dts b/arch/arc/boot/dts/nsimosci.dts
index e659a34..bcf6031 100644
--- a/arch/arc/boot/dts/nsimosci.dts
+++ b/arch/arc/boot/dts/nsimosci.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton.dtsi"
 
 / {
+	model = "snps,nsimosci";
 	compatible = "snps,nsimosci";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/nsimosci_hs.dts b/arch/arc/boot/dts/nsimosci_hs.dts
index 16ce5d6..6609240 100644
--- a/arch/arc/boot/dts/nsimosci_hs.dts
+++ b/arch/arc/boot/dts/nsimosci_hs.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs.dtsi"
 
 / {
+	model = "snps,nsimosci";
 	compatible = "snps,nsimosci_hs";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/nsimosci_hs_idu.dts b/arch/arc/boot/dts/nsimosci_hs_idu.dts
index ce8dfbc..4926c73 100644
--- a/arch/arc/boot/dts/nsimosci_hs_idu.dts
+++ b/arch/arc/boot/dts/nsimosci_hs_idu.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs_idu.dtsi"
 
 / {
+	model = "snps,nsimosci-smp";
 	compatible = "snps,nsimosci_hs";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/vdk_hs38.dts b/arch/arc/boot/dts/vdk_hs38.dts
index 5d803dd..ab2968e 100644
--- a/arch/arc/boot/dts/vdk_hs38.dts
+++ b/arch/arc/boot/dts/vdk_hs38.dts
@@ -13,6 +13,7 @@
 /include/ "vdk_axs10x_mb.dtsi"
 
 / {
+	model = "snps,vdk";
 	compatible = "snps,axs103";
 
 	chosen {
diff --git a/arch/arc/boot/dts/vdk_hs38_smp.dts b/arch/arc/boot/dts/vdk_hs38_smp.dts
index 2ba60c39..a39fd38 100644
--- a/arch/arc/boot/dts/vdk_hs38_smp.dts
+++ b/arch/arc/boot/dts/vdk_hs38_smp.dts
@@ -13,6 +13,7 @@
 /include/ "vdk_axs10x_mb.dtsi"
 
 / {
+	model = "snps,vdk-smp";
 	compatible = "snps,axs103";
 
 	chosen {
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* [PATCH] arc: Add "model" properly in device tree description of all boards
@ 2016-08-15 12:56 ` Alexey Brodkin
  0 siblings, 0 replies; 4+ messages in thread
From: Alexey Brodkin @ 2016-08-15 12:56 UTC (permalink / raw)
  To: linux-snps-arc

As it was discussed quite some time ago (see
https://lkml.org/lkml/2015/11/5/862) it's a good practice to add
"model" property in .dts. Moreover as per ePAPR "model" property is
required and should look like "manufacturer,model" so we do here.

Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
Cc: Vineet Gupta <vgupta at synopsys.com>
Cc: Jonas Gorski <jonas.gorski at gmail.com>
Cc: Arnd Bergmann <arnd at arndb.de>
Cc: Rob Herring <robh at kernel.org>
Cc: Christian Ruppert <christian.ruppert at alitech.com>
---
 arch/arc/boot/dts/abilis_tb100_dvk.dts | 1 +
 arch/arc/boot/dts/abilis_tb101_dvk.dts | 1 +
 arch/arc/boot/dts/axs101.dts           | 1 +
 arch/arc/boot/dts/axs103.dts           | 1 +
 arch/arc/boot/dts/axs103_idu.dts       | 1 +
 arch/arc/boot/dts/nsim_700.dts         | 1 +
 arch/arc/boot/dts/nsim_hs.dts          | 1 +
 arch/arc/boot/dts/nsim_hs_idu.dts      | 1 +
 arch/arc/boot/dts/nsimosci.dts         | 1 +
 arch/arc/boot/dts/nsimosci_hs.dts      | 1 +
 arch/arc/boot/dts/nsimosci_hs_idu.dts  | 1 +
 arch/arc/boot/dts/vdk_hs38.dts         | 1 +
 arch/arc/boot/dts/vdk_hs38_smp.dts     | 1 +
 13 files changed, 13 insertions(+)

diff --git a/arch/arc/boot/dts/abilis_tb100_dvk.dts b/arch/arc/boot/dts/abilis_tb100_dvk.dts
index 3dd6ed9..3acf04d 100644
--- a/arch/arc/boot/dts/abilis_tb100_dvk.dts
+++ b/arch/arc/boot/dts/abilis_tb100_dvk.dts
@@ -24,6 +24,7 @@
 /include/ "abilis_tb100.dtsi"
 
 / {
+	model = "abilis,tb100";
 	chosen {
 		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
 	};
diff --git a/arch/arc/boot/dts/abilis_tb101_dvk.dts b/arch/arc/boot/dts/abilis_tb101_dvk.dts
index 1cf51c2..37d88c5 100644
--- a/arch/arc/boot/dts/abilis_tb101_dvk.dts
+++ b/arch/arc/boot/dts/abilis_tb101_dvk.dts
@@ -24,6 +24,7 @@
 /include/ "abilis_tb101.dtsi"
 
 / {
+	model = "abilis,tb101";
 	chosen {
 		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
 	};
diff --git a/arch/arc/boot/dts/axs101.dts b/arch/arc/boot/dts/axs101.dts
index 3f9b058..d9b9b9d 100644
--- a/arch/arc/boot/dts/axs101.dts
+++ b/arch/arc/boot/dts/axs101.dts
@@ -13,6 +13,7 @@
 /include/ "axs10x_mb.dtsi"
 
 / {
+	model = "snps,axs101";
 	compatible = "snps,axs101", "snps,arc-sdp";
 
 	chosen {
diff --git a/arch/arc/boot/dts/axs103.dts b/arch/arc/boot/dts/axs103.dts
index e6d0e31..ec7fb27 100644
--- a/arch/arc/boot/dts/axs103.dts
+++ b/arch/arc/boot/dts/axs103.dts
@@ -16,6 +16,7 @@
 /include/ "axs10x_mb.dtsi"
 
 / {
+	model = "snps,axs103";
 	compatible = "snps,axs103", "snps,arc-sdp";
 
 	chosen {
diff --git a/arch/arc/boot/dts/axs103_idu.dts b/arch/arc/boot/dts/axs103_idu.dts
index f999fef..070c297 100644
--- a/arch/arc/boot/dts/axs103_idu.dts
+++ b/arch/arc/boot/dts/axs103_idu.dts
@@ -16,6 +16,7 @@
 /include/ "axs10x_mb.dtsi"
 
 / {
+	model = "snps,axs103-smp";
 	compatible = "snps,axs103", "snps,arc-sdp";
 
 	chosen {
diff --git a/arch/arc/boot/dts/nsim_700.dts b/arch/arc/boot/dts/nsim_700.dts
index 6397051..ce0ccd20 100644
--- a/arch/arc/boot/dts/nsim_700.dts
+++ b/arch/arc/boot/dts/nsim_700.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton.dtsi"
 
 / {
+	model = "snps,nsim";
 	compatible = "snps,nsim";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/nsim_hs.dts b/arch/arc/boot/dts/nsim_hs.dts
index bf05fe5..fa6fde8 100644
--- a/arch/arc/boot/dts/nsim_hs.dts
+++ b/arch/arc/boot/dts/nsim_hs.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs.dtsi"
 
 / {
+	model = "snps,nsim";
 	compatible = "snps,nsim_hs";
 	#address-cells = <2>;
 	#size-cells = <2>;
diff --git a/arch/arc/boot/dts/nsim_hs_idu.dts b/arch/arc/boot/dts/nsim_hs_idu.dts
index 99eabe1..1f02808 100644
--- a/arch/arc/boot/dts/nsim_hs_idu.dts
+++ b/arch/arc/boot/dts/nsim_hs_idu.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs_idu.dtsi"
 
 / {
+	model = "snps,nsim-smp";
 	compatible = "snps,nsim_hs";
 	interrupt-parent = <&core_intc>;
 
diff --git a/arch/arc/boot/dts/nsimosci.dts b/arch/arc/boot/dts/nsimosci.dts
index e659a34..bcf6031 100644
--- a/arch/arc/boot/dts/nsimosci.dts
+++ b/arch/arc/boot/dts/nsimosci.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton.dtsi"
 
 / {
+	model = "snps,nsimosci";
 	compatible = "snps,nsimosci";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/nsimosci_hs.dts b/arch/arc/boot/dts/nsimosci_hs.dts
index 16ce5d6..6609240 100644
--- a/arch/arc/boot/dts/nsimosci_hs.dts
+++ b/arch/arc/boot/dts/nsimosci_hs.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs.dtsi"
 
 / {
+	model = "snps,nsimosci";
 	compatible = "snps,nsimosci_hs";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/nsimosci_hs_idu.dts b/arch/arc/boot/dts/nsimosci_hs_idu.dts
index ce8dfbc..4926c73 100644
--- a/arch/arc/boot/dts/nsimosci_hs_idu.dts
+++ b/arch/arc/boot/dts/nsimosci_hs_idu.dts
@@ -10,6 +10,7 @@
 /include/ "skeleton_hs_idu.dtsi"
 
 / {
+	model = "snps,nsimosci-smp";
 	compatible = "snps,nsimosci_hs";
 	#address-cells = <1>;
 	#size-cells = <1>;
diff --git a/arch/arc/boot/dts/vdk_hs38.dts b/arch/arc/boot/dts/vdk_hs38.dts
index 5d803dd..ab2968e 100644
--- a/arch/arc/boot/dts/vdk_hs38.dts
+++ b/arch/arc/boot/dts/vdk_hs38.dts
@@ -13,6 +13,7 @@
 /include/ "vdk_axs10x_mb.dtsi"
 
 / {
+	model = "snps,vdk";
 	compatible = "snps,axs103";
 
 	chosen {
diff --git a/arch/arc/boot/dts/vdk_hs38_smp.dts b/arch/arc/boot/dts/vdk_hs38_smp.dts
index 2ba60c39..a39fd38 100644
--- a/arch/arc/boot/dts/vdk_hs38_smp.dts
+++ b/arch/arc/boot/dts/vdk_hs38_smp.dts
@@ -13,6 +13,7 @@
 /include/ "vdk_axs10x_mb.dtsi"
 
 / {
+	model = "snps,vdk-smp";
 	compatible = "snps,axs103";
 
 	chosen {
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH] arc: Add "model" properly in device tree description of all boards
  2016-08-15 12:56 ` Alexey Brodkin
@ 2016-08-16  1:16   ` Vineet Gupta
  -1 siblings, 0 replies; 4+ messages in thread
From: Vineet Gupta @ 2016-08-16  1:16 UTC (permalink / raw)
  To: Alexey Brodkin, linux-snps-arc
  Cc: Rob Herring, Christian Ruppert, Arnd Bergmann, Vineet Gupta,
	linux-kernel, Jonas Gorski

On 08/15/2016 05:56 AM, Alexey Brodkin wrote:
> As it was discussed quite some time ago (see
> https://lkml.org/lkml/2015/11/5/862) it's a good practice to add
> "model" property in .dts. Moreover as per ePAPR "model" property is
> required and should look like "manufacturer,model" so we do here.
> 
> Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
> Cc: Vineet Gupta <vgupta@synopsys.com>
> Cc: Jonas Gorski <jonas.gorski@gmail.com>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Rob Herring <robh@kernel.org>
> Cc: Christian Ruppert <christian.ruppert@alitech.com>
> ---
>  arch/arc/boot/dts/abilis_tb100_dvk.dts | 1 +
>  arch/arc/boot/dts/abilis_tb101_dvk.dts | 1 +
>  arch/arc/boot/dts/axs101.dts           | 1 +
>  arch/arc/boot/dts/axs103.dts           | 1 +
>  arch/arc/boot/dts/axs103_idu.dts       | 1 +
>  arch/arc/boot/dts/nsim_700.dts         | 1 +
>  arch/arc/boot/dts/nsim_hs.dts          | 1 +
>  arch/arc/boot/dts/nsim_hs_idu.dts      | 1 +
>  arch/arc/boot/dts/nsimosci.dts         | 1 +
>  arch/arc/boot/dts/nsimosci_hs.dts      | 1 +
>  arch/arc/boot/dts/nsimosci_hs_idu.dts  | 1 +
>  arch/arc/boot/dts/vdk_hs38.dts         | 1 +
>  arch/arc/boot/dts/vdk_hs38_smp.dts     | 1 +
>  13 files changed, 13 insertions(+)
> 
> diff --git a/arch/arc/boot/dts/abilis_tb100_dvk.dts b/arch/arc/boot/dts/abilis_tb100_dvk.dts
> index 3dd6ed9..3acf04d 100644
> --- a/arch/arc/boot/dts/abilis_tb100_dvk.dts
> +++ b/arch/arc/boot/dts/abilis_tb100_dvk.dts
> @@ -24,6 +24,7 @@
>  /include/ "abilis_tb100.dtsi"
>  
>  / {
> +	model = "abilis,tb100";
>  	chosen {
>  		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
>  	};
> diff --git a/arch/arc/boot/dts/abilis_tb101_dvk.dts b/arch/arc/boot/dts/abilis_tb101_dvk.dts
> index 1cf51c2..37d88c5 100644
> --- a/arch/arc/boot/dts/abilis_tb101_dvk.dts
> +++ b/arch/arc/boot/dts/abilis_tb101_dvk.dts
> @@ -24,6 +24,7 @@
>  /include/ "abilis_tb101.dtsi"
>  
>  / {
> +	model = "abilis,tb101";
>  	chosen {
>  		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
>  	};
> diff --git a/arch/arc/boot/dts/axs101.dts b/arch/arc/boot/dts/axs101.dts
> index 3f9b058..d9b9b9d 100644
> --- a/arch/arc/boot/dts/axs101.dts
> +++ b/arch/arc/boot/dts/axs101.dts
> @@ -13,6 +13,7 @@
>  /include/ "axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,axs101";
>  	compatible = "snps,axs101", "snps,arc-sdp";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/axs103.dts b/arch/arc/boot/dts/axs103.dts
> index e6d0e31..ec7fb27 100644
> --- a/arch/arc/boot/dts/axs103.dts
> +++ b/arch/arc/boot/dts/axs103.dts
> @@ -16,6 +16,7 @@
>  /include/ "axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,axs103";
>  	compatible = "snps,axs103", "snps,arc-sdp";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/axs103_idu.dts b/arch/arc/boot/dts/axs103_idu.dts
> index f999fef..070c297 100644
> --- a/arch/arc/boot/dts/axs103_idu.dts
> +++ b/arch/arc/boot/dts/axs103_idu.dts
> @@ -16,6 +16,7 @@
>  /include/ "axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,axs103-smp";
>  	compatible = "snps,axs103", "snps,arc-sdp";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/nsim_700.dts b/arch/arc/boot/dts/nsim_700.dts
> index 6397051..ce0ccd20 100644
> --- a/arch/arc/boot/dts/nsim_700.dts
> +++ b/arch/arc/boot/dts/nsim_700.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton.dtsi"
>  
>  / {
> +	model = "snps,nsim";
>  	compatible = "snps,nsim";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/nsim_hs.dts b/arch/arc/boot/dts/nsim_hs.dts
> index bf05fe5..fa6fde8 100644
> --- a/arch/arc/boot/dts/nsim_hs.dts
> +++ b/arch/arc/boot/dts/nsim_hs.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs.dtsi"
>  
>  / {
> +	model = "snps,nsim";

snps,nsim_hs ?

>  	compatible = "snps,nsim_hs";
>  	#address-cells = <2>;
>  	#size-cells = <2>;
> diff --git a/arch/arc/boot/dts/nsim_hs_idu.dts b/arch/arc/boot/dts/nsim_hs_idu.dts
> index 99eabe1..1f02808 100644
> --- a/arch/arc/boot/dts/nsim_hs_idu.dts
> +++ b/arch/arc/boot/dts/nsim_hs_idu.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs_idu.dtsi"
>  
>  / {
> +	model = "snps,nsim-smp";

snps,nsim_hs-smp ?

>  	compatible = "snps,nsim_hs";
>  	interrupt-parent = <&core_intc>;
>  
> diff --git a/arch/arc/boot/dts/nsimosci.dts b/arch/arc/boot/dts/nsimosci.dts
> index e659a34..bcf6031 100644
> --- a/arch/arc/boot/dts/nsimosci.dts
> +++ b/arch/arc/boot/dts/nsimosci.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton.dtsi"
>  
>  / {
> +	model = "snps,nsimosci";
>  	compatible = "snps,nsimosci";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/nsimosci_hs.dts b/arch/arc/boot/dts/nsimosci_hs.dts
> index 16ce5d6..6609240 100644
> --- a/arch/arc/boot/dts/nsimosci_hs.dts
> +++ b/arch/arc/boot/dts/nsimosci_hs.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs.dtsi"
>  
>  / {
> +	model = "snps,nsimosci";

snps,nsimosci_hs ?

>  	compatible = "snps,nsimosci_hs";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/nsimosci_hs_idu.dts b/arch/arc/boot/dts/nsimosci_hs_idu.dts
> index ce8dfbc..4926c73 100644
> --- a/arch/arc/boot/dts/nsimosci_hs_idu.dts
> +++ b/arch/arc/boot/dts/nsimosci_hs_idu.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs_idu.dtsi"
>  
>  / {
> +	model = "snps,nsimosci-smp";

snps,nsimosci_hs-smp ?

>  	compatible = "snps,nsimosci_hs";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/vdk_hs38.dts b/arch/arc/boot/dts/vdk_hs38.dts
> index 5d803dd..ab2968e 100644
> --- a/arch/arc/boot/dts/vdk_hs38.dts
> +++ b/arch/arc/boot/dts/vdk_hs38.dts
> @@ -13,6 +13,7 @@
>  /include/ "vdk_axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,vdk";

AFAIk there are VDK models for ARM so better to prefix these with arc or some such !

>  	compatible = "snps,axs103";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/vdk_hs38_smp.dts b/arch/arc/boot/dts/vdk_hs38_smp.dts
> index 2ba60c39..a39fd38 100644
> --- a/arch/arc/boot/dts/vdk_hs38_smp.dts
> +++ b/arch/arc/boot/dts/vdk_hs38_smp.dts
> @@ -13,6 +13,7 @@
>  /include/ "vdk_axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,vdk-smp";
>  	compatible = "snps,axs103";
>  
>  	chosen {
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [PATCH] arc: Add "model" properly in device tree description of all boards
@ 2016-08-16  1:16   ` Vineet Gupta
  0 siblings, 0 replies; 4+ messages in thread
From: Vineet Gupta @ 2016-08-16  1:16 UTC (permalink / raw)
  To: linux-snps-arc

On 08/15/2016 05:56 AM, Alexey Brodkin wrote:
> As it was discussed quite some time ago (see
> https://lkml.org/lkml/2015/11/5/862) it's a good practice to add
> "model" property in .dts. Moreover as per ePAPR "model" property is
> required and should look like "manufacturer,model" so we do here.
> 
> Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
> Cc: Vineet Gupta <vgupta at synopsys.com>
> Cc: Jonas Gorski <jonas.gorski at gmail.com>
> Cc: Arnd Bergmann <arnd at arndb.de>
> Cc: Rob Herring <robh at kernel.org>
> Cc: Christian Ruppert <christian.ruppert at alitech.com>
> ---
>  arch/arc/boot/dts/abilis_tb100_dvk.dts | 1 +
>  arch/arc/boot/dts/abilis_tb101_dvk.dts | 1 +
>  arch/arc/boot/dts/axs101.dts           | 1 +
>  arch/arc/boot/dts/axs103.dts           | 1 +
>  arch/arc/boot/dts/axs103_idu.dts       | 1 +
>  arch/arc/boot/dts/nsim_700.dts         | 1 +
>  arch/arc/boot/dts/nsim_hs.dts          | 1 +
>  arch/arc/boot/dts/nsim_hs_idu.dts      | 1 +
>  arch/arc/boot/dts/nsimosci.dts         | 1 +
>  arch/arc/boot/dts/nsimosci_hs.dts      | 1 +
>  arch/arc/boot/dts/nsimosci_hs_idu.dts  | 1 +
>  arch/arc/boot/dts/vdk_hs38.dts         | 1 +
>  arch/arc/boot/dts/vdk_hs38_smp.dts     | 1 +
>  13 files changed, 13 insertions(+)
> 
> diff --git a/arch/arc/boot/dts/abilis_tb100_dvk.dts b/arch/arc/boot/dts/abilis_tb100_dvk.dts
> index 3dd6ed9..3acf04d 100644
> --- a/arch/arc/boot/dts/abilis_tb100_dvk.dts
> +++ b/arch/arc/boot/dts/abilis_tb100_dvk.dts
> @@ -24,6 +24,7 @@
>  /include/ "abilis_tb100.dtsi"
>  
>  / {
> +	model = "abilis,tb100";
>  	chosen {
>  		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
>  	};
> diff --git a/arch/arc/boot/dts/abilis_tb101_dvk.dts b/arch/arc/boot/dts/abilis_tb101_dvk.dts
> index 1cf51c2..37d88c5 100644
> --- a/arch/arc/boot/dts/abilis_tb101_dvk.dts
> +++ b/arch/arc/boot/dts/abilis_tb101_dvk.dts
> @@ -24,6 +24,7 @@
>  /include/ "abilis_tb101.dtsi"
>  
>  / {
> +	model = "abilis,tb101";
>  	chosen {
>  		bootargs = "earlycon=uart8250,mmio32,0xff100000,9600n8 console=ttyS0,9600n8";
>  	};
> diff --git a/arch/arc/boot/dts/axs101.dts b/arch/arc/boot/dts/axs101.dts
> index 3f9b058..d9b9b9d 100644
> --- a/arch/arc/boot/dts/axs101.dts
> +++ b/arch/arc/boot/dts/axs101.dts
> @@ -13,6 +13,7 @@
>  /include/ "axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,axs101";
>  	compatible = "snps,axs101", "snps,arc-sdp";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/axs103.dts b/arch/arc/boot/dts/axs103.dts
> index e6d0e31..ec7fb27 100644
> --- a/arch/arc/boot/dts/axs103.dts
> +++ b/arch/arc/boot/dts/axs103.dts
> @@ -16,6 +16,7 @@
>  /include/ "axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,axs103";
>  	compatible = "snps,axs103", "snps,arc-sdp";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/axs103_idu.dts b/arch/arc/boot/dts/axs103_idu.dts
> index f999fef..070c297 100644
> --- a/arch/arc/boot/dts/axs103_idu.dts
> +++ b/arch/arc/boot/dts/axs103_idu.dts
> @@ -16,6 +16,7 @@
>  /include/ "axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,axs103-smp";
>  	compatible = "snps,axs103", "snps,arc-sdp";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/nsim_700.dts b/arch/arc/boot/dts/nsim_700.dts
> index 6397051..ce0ccd20 100644
> --- a/arch/arc/boot/dts/nsim_700.dts
> +++ b/arch/arc/boot/dts/nsim_700.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton.dtsi"
>  
>  / {
> +	model = "snps,nsim";
>  	compatible = "snps,nsim";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/nsim_hs.dts b/arch/arc/boot/dts/nsim_hs.dts
> index bf05fe5..fa6fde8 100644
> --- a/arch/arc/boot/dts/nsim_hs.dts
> +++ b/arch/arc/boot/dts/nsim_hs.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs.dtsi"
>  
>  / {
> +	model = "snps,nsim";

snps,nsim_hs ?

>  	compatible = "snps,nsim_hs";
>  	#address-cells = <2>;
>  	#size-cells = <2>;
> diff --git a/arch/arc/boot/dts/nsim_hs_idu.dts b/arch/arc/boot/dts/nsim_hs_idu.dts
> index 99eabe1..1f02808 100644
> --- a/arch/arc/boot/dts/nsim_hs_idu.dts
> +++ b/arch/arc/boot/dts/nsim_hs_idu.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs_idu.dtsi"
>  
>  / {
> +	model = "snps,nsim-smp";

snps,nsim_hs-smp ?

>  	compatible = "snps,nsim_hs";
>  	interrupt-parent = <&core_intc>;
>  
> diff --git a/arch/arc/boot/dts/nsimosci.dts b/arch/arc/boot/dts/nsimosci.dts
> index e659a34..bcf6031 100644
> --- a/arch/arc/boot/dts/nsimosci.dts
> +++ b/arch/arc/boot/dts/nsimosci.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton.dtsi"
>  
>  / {
> +	model = "snps,nsimosci";
>  	compatible = "snps,nsimosci";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/nsimosci_hs.dts b/arch/arc/boot/dts/nsimosci_hs.dts
> index 16ce5d6..6609240 100644
> --- a/arch/arc/boot/dts/nsimosci_hs.dts
> +++ b/arch/arc/boot/dts/nsimosci_hs.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs.dtsi"
>  
>  / {
> +	model = "snps,nsimosci";

snps,nsimosci_hs ?

>  	compatible = "snps,nsimosci_hs";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/nsimosci_hs_idu.dts b/arch/arc/boot/dts/nsimosci_hs_idu.dts
> index ce8dfbc..4926c73 100644
> --- a/arch/arc/boot/dts/nsimosci_hs_idu.dts
> +++ b/arch/arc/boot/dts/nsimosci_hs_idu.dts
> @@ -10,6 +10,7 @@
>  /include/ "skeleton_hs_idu.dtsi"
>  
>  / {
> +	model = "snps,nsimosci-smp";

snps,nsimosci_hs-smp ?

>  	compatible = "snps,nsimosci_hs";
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> diff --git a/arch/arc/boot/dts/vdk_hs38.dts b/arch/arc/boot/dts/vdk_hs38.dts
> index 5d803dd..ab2968e 100644
> --- a/arch/arc/boot/dts/vdk_hs38.dts
> +++ b/arch/arc/boot/dts/vdk_hs38.dts
> @@ -13,6 +13,7 @@
>  /include/ "vdk_axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,vdk";

AFAIk there are VDK models for ARM so better to prefix these with arc or some such !

>  	compatible = "snps,axs103";
>  
>  	chosen {
> diff --git a/arch/arc/boot/dts/vdk_hs38_smp.dts b/arch/arc/boot/dts/vdk_hs38_smp.dts
> index 2ba60c39..a39fd38 100644
> --- a/arch/arc/boot/dts/vdk_hs38_smp.dts
> +++ b/arch/arc/boot/dts/vdk_hs38_smp.dts
> @@ -13,6 +13,7 @@
>  /include/ "vdk_axs10x_mb.dtsi"
>  
>  / {
> +	model = "snps,vdk-smp";
>  	compatible = "snps,axs103";
>  
>  	chosen {
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-08-16  1:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-15 12:56 [PATCH] arc: Add "model" properly in device tree description of all boards Alexey Brodkin
2016-08-15 12:56 ` Alexey Brodkin
2016-08-16  1:16 ` Vineet Gupta
2016-08-16  1:16   ` Vineet Gupta

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.