From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932479AbcHPBRO (ORCPT ); Mon, 15 Aug 2016 21:17:14 -0400 Received: from smtprelay.synopsys.com ([198.182.60.111]:47335 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932201AbcHPBRL (ORCPT ); Mon, 15 Aug 2016 21:17:11 -0400 Subject: Re: [PATCH] arc: Add "model" properly in device tree description of all boards To: Alexey Brodkin , References: <1471265776-29495-1-git-send-email-abrodkin@synopsys.com> CC: Rob Herring , Christian Ruppert , Arnd Bergmann , Vineet Gupta , , Jonas Gorski Newsgroups: gmane.linux.kernel.arc,gmane.linux.kernel From: Vineet Gupta Message-ID: Date: Mon, 15 Aug 2016 18:16:55 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <1471265776-29495-1-git-send-email-abrodkin@synopsys.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.9.129.216] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > Cc: Vineet Gupta > Cc: Jonas Gorski > Cc: Arnd Bergmann > Cc: Rob Herring > Cc: Christian Ruppert > --- > 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 { >