mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Michal Simek <michal.simek@xilinx.com>
To: Luis Araneda <luaraneda@gmail.com>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Michal Simek <michal.simek@xilinx.com>
Cc: <devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH 3/5] arm: dts: zynq: correct and improve the model property of dt files
Date: Thu, 28 Jun 2018 08:39:09 +0200	[thread overview]
Message-ID: <05232c62-0b3f-d2b1-3e28-98f0154942ef@xilinx.com> (raw)
In-Reply-To: <20180628054205.17737-4-luaraneda@gmail.com>

On 28.6.2018 07:42, Luis Araneda wrote:
> Replace the current value of the model property by a more accurate
> description of each board (which includes the manufacturer), as some
> of the boards had the same value ("Xilinx Zynq")
> 
> Signed-off-by: Luis Araneda <luaraneda@gmail.com>
> ---
>  arch/arm/boot/dts/zynq-cc108.dts       | 2 +-
>  arch/arm/boot/dts/zynq-microzed.dts    | 2 +-
>  arch/arm/boot/dts/zynq-parallella.dts  | 2 +-
>  arch/arm/boot/dts/zynq-zc702.dts       | 2 +-
>  arch/arm/boot/dts/zynq-zc706.dts       | 2 +-
>  arch/arm/boot/dts/zynq-zc770-xm010.dts | 2 +-
>  arch/arm/boot/dts/zynq-zc770-xm011.dts | 2 +-
>  arch/arm/boot/dts/zynq-zc770-xm012.dts | 2 +-
>  arch/arm/boot/dts/zynq-zc770-xm013.dts | 2 +-
>  arch/arm/boot/dts/zynq-zed.dts         | 2 +-
>  arch/arm/boot/dts/zynq-zybo-z7.dts     | 2 +-
>  arch/arm/boot/dts/zynq-zybo.dts        | 2 +-
>  12 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/zynq-cc108.dts b/arch/arm/boot/dts/zynq-cc108.dts
> index 1a0f631c1..8b9ab9bba 100644
> --- a/arch/arm/boot/dts/zynq-cc108.dts
> +++ b/arch/arm/boot/dts/zynq-cc108.dts
> @@ -12,8 +12,8 @@
>  /include/ "zynq-7000.dtsi"
>  
>  / {
> +	model = "Xilinx CC108 board";
>  	compatible = "xlnx,zynq-cc108", "xlnx,zynq-7000";
> -	model = "Xilinx Zynq";
>  
>  	aliases {
>  		ethernet0 = &gem0;
> diff --git a/arch/arm/boot/dts/zynq-microzed.dts b/arch/arm/boot/dts/zynq-microzed.dts
> index e40f40a7f..3cc957428 100644
> --- a/arch/arm/boot/dts/zynq-microzed.dts
> +++ b/arch/arm/boot/dts/zynq-microzed.dts
> @@ -7,7 +7,7 @@
>  /include/ "zynq-7000.dtsi"
>  
>  / {
> -	model = "Zynq MicroZED Development Board";
> +	model = "Avnet MicroZed board";
>  	compatible = "avnet,zynq-microzed", "xlnx,zynq-7000";
>  
>  	aliases {
> diff --git a/arch/arm/boot/dts/zynq-parallella.dts b/arch/arm/boot/dts/zynq-parallella.dts
> index c05f4b67d..54592aeb9 100644
> --- a/arch/arm/boot/dts/zynq-parallella.dts
> +++ b/arch/arm/boot/dts/zynq-parallella.dts
> @@ -12,7 +12,7 @@
>  /include/ "zynq-7000.dtsi"
>  
>  / {
> -	model = "Adapteva Parallella Board";
> +	model = "Adapteva Parallella board";
>  	compatible = "adapteva,parallella", "xlnx,zynq-7000";
>  
>  	aliases {
> diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
> index f2330b0cb..3f84b94db 100644
> --- a/arch/arm/boot/dts/zynq-zc702.dts
> +++ b/arch/arm/boot/dts/zynq-zc702.dts
> @@ -7,7 +7,7 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> -	model = "Zynq ZC702 Development Board";
> +	model = "Xilinx ZC702 board";
>  	compatible = "xlnx,zynq-zc702", "xlnx,zynq-7000";
>  
>  	aliases {
> diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
> index 3ad1260ff..a15908cac 100644
> --- a/arch/arm/boot/dts/zynq-zc706.dts
> +++ b/arch/arm/boot/dts/zynq-zc706.dts
> @@ -7,7 +7,7 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> -	model = "Zynq ZC706 Development Board";
> +	model = "Xilinx ZC706 board";
>  	compatible = "xlnx,zynq-zc706", "xlnx,zynq-7000";
>  
>  	aliases {
> diff --git a/arch/arm/boot/dts/zynq-zc770-xm010.dts b/arch/arm/boot/dts/zynq-zc770-xm010.dts
> index 6884f1ad6..0e1bfdd34 100644
> --- a/arch/arm/boot/dts/zynq-zc770-xm010.dts
> +++ b/arch/arm/boot/dts/zynq-zc770-xm010.dts
> @@ -8,8 +8,8 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> +	model = "Xilinx ZC770 XM010 board";
>  	compatible = "xlnx,zynq-zc770-xm010", "xlnx,zynq-7000";
> -	model = "Xilinx Zynq";
>  
>  	aliases {
>  		ethernet0 = &gem0;
> diff --git a/arch/arm/boot/dts/zynq-zc770-xm011.dts b/arch/arm/boot/dts/zynq-zc770-xm011.dts
> index b78883cee..b7f65862c 100644
> --- a/arch/arm/boot/dts/zynq-zc770-xm011.dts
> +++ b/arch/arm/boot/dts/zynq-zc770-xm011.dts
> @@ -8,8 +8,8 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> +	model = "Xilinx ZC770 XM011 board";
>  	compatible = "xlnx,zynq-zc770-xm011", "xlnx,zynq-7000";
> -	model = "Xilinx Zynq";
>  
>  	aliases {
>  		i2c0 = &i2c1;
> diff --git a/arch/arm/boot/dts/zynq-zc770-xm012.dts b/arch/arm/boot/dts/zynq-zc770-xm012.dts
> index c3169d636..d2359b789 100644
> --- a/arch/arm/boot/dts/zynq-zc770-xm012.dts
> +++ b/arch/arm/boot/dts/zynq-zc770-xm012.dts
> @@ -8,8 +8,8 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> +	model = "Xilinx ZC770 XM012 board";
>  	compatible = "xlnx,zynq-zc770-xm012", "xlnx,zynq-7000";
> -	model = "Xilinx Zynq";
>  
>  	aliases {
>  		i2c0 = &i2c0;
> diff --git a/arch/arm/boot/dts/zynq-zc770-xm013.dts b/arch/arm/boot/dts/zynq-zc770-xm013.dts
> index 8bb66859d..651913f1a 100644
> --- a/arch/arm/boot/dts/zynq-zc770-xm013.dts
> +++ b/arch/arm/boot/dts/zynq-zc770-xm013.dts
> @@ -8,8 +8,8 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> +	model = "Xilinx ZC770 XM013 board";
>  	compatible = "xlnx,zynq-zc770-xm013", "xlnx,zynq-7000";
> -	model = "Xilinx Zynq";
>  
>  	aliases {
>  		ethernet0 = &gem1;
> diff --git a/arch/arm/boot/dts/zynq-zed.dts b/arch/arm/boot/dts/zynq-zed.dts
> index 98381cc85..bf3e26dbf 100644
> --- a/arch/arm/boot/dts/zynq-zed.dts
> +++ b/arch/arm/boot/dts/zynq-zed.dts
> @@ -7,7 +7,7 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> -	model = "Zynq Zed Development Board";
> +	model = "Avnet ZedBoard board";
>  	compatible = "avnet,zynq-zed", "xlnx,zynq-7000";
>  
>  	aliases {
> diff --git a/arch/arm/boot/dts/zynq-zybo-z7.dts b/arch/arm/boot/dts/zynq-zybo-z7.dts
> index 1e713dc98..be40be9b7 100644
> --- a/arch/arm/boot/dts/zynq-zybo-z7.dts
> +++ b/arch/arm/boot/dts/zynq-zybo-z7.dts
> @@ -3,7 +3,7 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> -	model = "Zynq ZYBO Z7 Development Board";
> +	model = "Digilent Zybo Z7 board";
>  	compatible = "digilent,zynq-zybo-z7", "xlnx,zynq-7000";
>  
>  	aliases {
> diff --git a/arch/arm/boot/dts/zynq-zybo.dts b/arch/arm/boot/dts/zynq-zybo.dts
> index a6c00e7fa..0439e9f1e 100644
> --- a/arch/arm/boot/dts/zynq-zybo.dts
> +++ b/arch/arm/boot/dts/zynq-zybo.dts
> @@ -7,7 +7,7 @@
>  #include "zynq-7000.dtsi"
>  
>  / {
> -	model = "Zynq ZYBO Development Board";
> +	model = "Digilent Zybo board";
>  	compatible = "digilent,zynq-zybo", "xlnx,zynq-7000";
>  
>  	aliases {
> 

Not a problem with this patch.

M


  reply	other threads:[~2018-06-28  6:39 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-28  5:42 [RFC PATCH 0/5] arm: dts: zynq: improvements and cleanup Luis Araneda
2018-06-28  5:42 ` [RFC PATCH 1/5] arm: dts: zynq: Set correct manufacturer for ZedBoard and MicroZed boards Luis Araneda
2018-06-28  6:36   ` Michal Simek
2018-07-03  6:51     ` Luis Araneda
2018-06-28  5:42 ` [RFC PATCH 2/5] dt-bindings: xilinx: Add missing zynq boards Luis Araneda
2018-06-28  6:44   ` Michal Simek
2018-07-03  6:53     ` Luis Araneda
2018-07-03 23:04   ` Rob Herring
2018-07-05 21:10     ` Luis Araneda
2018-06-28  5:42 ` [RFC PATCH 3/5] arm: dts: zynq: correct and improve the model property of dt files Luis Araneda
2018-06-28  6:39   ` Michal Simek [this message]
2018-06-28  5:42 ` [RFC PATCH 4/5] arm: dts: zynq: rename board zed to zedboard Luis Araneda
2018-06-28  6:40   ` Michal Simek
2018-07-03  6:56     ` Luis Araneda
2018-06-28  5:42 ` [RFC PATCH 5/5] arm: dts: zynq: remove "zynq" prefix from compatible property Luis Araneda
2018-06-28  6:41   ` Michal Simek
2018-07-03  7:18     ` Luis Araneda
2018-07-03 23:07       ` Rob Herring
2018-07-05 21:12         ` Luis Araneda

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=05232c62-0b3f-d2b1-3e28-98f0154942ef@xilinx.com \
    --to=michal.simek@xilinx.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luaraneda@gmail.com \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®