| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <f25cf01d-1352-463d-7a6d-64b12402443e@synopsys.com>
Date: Mon, 15 Aug 2016 18:16:55 -0700
From: Vineet Gupta <Vineet.Gupta1@...opsys.com>
To: Alexey Brodkin <Alexey.Brodkin@...opsys.com>,
<linux-snps-arc@...ts.infradead.org>
CC: Rob Herring <robh@...nel.org>,
Christian Ruppert <christian.ruppert@...tech.com>,
Arnd Bergmann <arnd@...db.de>,
Vineet Gupta <Vineet.Gupta1@...opsys.com>,
<linux-kernel@...r.kernel.org>,
Jonas Gorski <jonas.gorski@...il.com>
Subject: Re: [PATCH] arc: Add "model" properly in device tree description of
all boards
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@...opsys.com>
> Cc: Vineet Gupta <vgupta@...opsys.com>
> Cc: Jonas Gorski <jonas.gorski@...il.com>
> Cc: Arnd Bergmann <arnd@...db.de>
> Cc: Rob Herring <robh@...nel.org>
> Cc: Christian Ruppert <christian.ruppert@...tech.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 {
>
Powered by blists - more mailing lists