linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: "Niklas Söderlund" <niklas.soderlund@ragnatech.se>
To: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Magnus Damm <magnus.damm@gmail.com>,
	linux-renesas-soc@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] arm64: dts: renesas: rcar-gen3: Add SoC model to comment headers
Date: Mon, 7 Jun 2021 23:02:32 +0200	[thread overview]
Message-ID: <YL6JaF8GEiuqnh4M@oden.dyn.berto.se> (raw)
In-Reply-To: <2cb7113d307b1bfdb33cd96245202b2d5d867818.1623087738.git.geert+renesas@glider.be>

Hi Geert,

Thanks for your work.

On 2021-06-07 19:42:38 +0200, Geert Uytterhoeven wrote:
> Make sure the R-Car Gen3 SoC model present is documented in the comment
> header of each board DTS.  This makes it easier to identify boards that
> are available with different SoC or SiP options.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

nit: Would it make sens to break the line length limit for this to make 
it easier for grep to be more useful? In either case,

Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

> ---
> To be queued in renesas-devel for v5.14.
> 
>  arch/arm64/boot/dts/renesas/r8a77950-ulcb-kf.dts | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77950-ulcb.dts    | 3 ++-
>  arch/arm64/boot/dts/renesas/r8a77951-ulcb-kf.dts | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77951-ulcb.dts    | 3 ++-
>  arch/arm64/boot/dts/renesas/r8a77960-ulcb-kf.dts | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77960-ulcb.dts    | 3 ++-
>  arch/arm64/boot/dts/renesas/r8a77961-ulcb-kf.dts | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77965-ulcb-kf.dts | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77965-ulcb.dts    | 3 ++-
>  arch/arm64/boot/dts/renesas/r8a77970-eagle.dts   | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77980-condor.dts  | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts   | 2 +-
>  arch/arm64/boot/dts/renesas/r8a77995-draak.dts   | 2 +-
>  arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts  | 2 +-
>  14 files changed, 18 insertions(+), 14 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77950-ulcb-kf.dts b/arch/arm64/boot/dts/renesas/r8a77950-ulcb-kf.dts
> index dcaaf12cec40dab4..85f008ef63ded5d6 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77950-ulcb-kf.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77950-ulcb-kf.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the H3ULCB Kingfisher board
> + * Device Tree Source for the H3ULCB Kingfisher board with R-Car H3 ES1.x
>   *
>   * Copyright (C) 2017 Renesas Electronics Corp.
>   * Copyright (C) 2017 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77950-ulcb.dts b/arch/arm64/boot/dts/renesas/r8a77950-ulcb.dts
> index 38a6d6a108d488ed..f1df707b98a6e269 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77950-ulcb.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77950-ulcb.dts
> @@ -1,6 +1,7 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the H3ULCB (R-Car Starter Kit Premier) board
> + * Device Tree Source for the H3ULCB (R-Car Starter Kit Premier) board with
> + * R-Car H3 ES1.x
>   *
>   * Copyright (C) 2016 Renesas Electronics Corp.
>   * Copyright (C) 2016 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77951-ulcb-kf.dts b/arch/arm64/boot/dts/renesas/r8a77951-ulcb-kf.dts
> index 11f943a67703f0a8..2e58a27aa2766603 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77951-ulcb-kf.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77951-ulcb-kf.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the H3ULCB Kingfisher board
> + * Device Tree Source for the H3ULCB Kingfisher board with R-Car H3 ES2.0+
>   *
>   * Copyright (C) 2017 Renesas Electronics Corp.
>   * Copyright (C) 2017 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77951-ulcb.dts b/arch/arm64/boot/dts/renesas/r8a77951-ulcb.dts
> index 8ad8f2a539771de8..21ce300b54abecef 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77951-ulcb.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77951-ulcb.dts
> @@ -1,6 +1,7 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the H3ULCB (R-Car Starter Kit Premier) board
> + * Device Tree Source for the H3ULCB (R-Car Starter Kit Premier) board with
> + * R-Car H3 ES2.0+
>   *
>   * Copyright (C) 2016 Renesas Electronics Corp.
>   * Copyright (C) 2016 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77960-ulcb-kf.dts b/arch/arm64/boot/dts/renesas/r8a77960-ulcb-kf.dts
> index 2151c37d77a6c942..02d61360692c0ac6 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77960-ulcb-kf.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77960-ulcb-kf.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the M3ULCB Kingfisher board
> + * Device Tree Source for the M3ULCB Kingfisher board with R-Car M3-W
>   *
>   * Copyright (C) 2017 Renesas Electronics Corp.
>   * Copyright (C) 2017 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77960-ulcb.dts b/arch/arm64/boot/dts/renesas/r8a77960-ulcb.dts
> index d041042a56192ab2..d936fb9281be6edd 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77960-ulcb.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77960-ulcb.dts
> @@ -1,6 +1,7 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the M3ULCB (R-Car Starter Kit Pro) board
> + * Device Tree Source for the M3ULCB (R-Car Starter Kit Pro) board with R-Car
> + * M3-W
>   *
>   * Copyright (C) 2016 Renesas Electronics Corp.
>   * Copyright (C) 2016 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77961-ulcb-kf.dts b/arch/arm64/boot/dts/renesas/r8a77961-ulcb-kf.dts
> index 6ec958348eb025d9..d66eb27ee8c48189 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77961-ulcb-kf.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77961-ulcb-kf.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the M3ULCB Kingfisher board
> + * Device Tree Source for the M3ULCB Kingfisher board with R-Car M3-W+
>   *
>   * Copyright (C) 2020 Eugeniu Rosca <rosca.eugeniu@gmail.com>
>   */
> diff --git a/arch/arm64/boot/dts/renesas/r8a77965-ulcb-kf.dts b/arch/arm64/boot/dts/renesas/r8a77965-ulcb-kf.dts
> index 12aa08fd6fd87f7e..a601968c5727e55a 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77965-ulcb-kf.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77965-ulcb-kf.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the M3NULCB Kingfisher board
> + * Device Tree Source for the M3NULCB Kingfisher board with R-Car M3-N
>   *
>   * Copyright (C) 2018 Renesas Electronics Corp.
>   * Copyright (C) 2018 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77965-ulcb.dts b/arch/arm64/boot/dts/renesas/r8a77965-ulcb.dts
> index 964078b6cc49e959..87b013279c74dc48 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77965-ulcb.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77965-ulcb.dts
> @@ -1,6 +1,7 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the M3NULCB (R-Car Starter Kit Pro) board
> + * Device Tree Source for the M3NULCB (R-Car Starter Kit Pro) board with R-Car
> + * M3-N
>   *
>   * Copyright (C) 2018 Renesas Electronics Corp.
>   * Copyright (C) 2018 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> index 5c84681703edad2e..d24da54f312b9085 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the Eagle board
> + * Device Tree Source for the Eagle board with R-Car V3M
>   *
>   * Copyright (C) 2016-2017 Renesas Electronics Corp.
>   * Copyright (C) 2017 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77980-condor.dts b/arch/arm64/boot/dts/renesas/r8a77980-condor.dts
> index 7bde0a549c09884f..edf7f2a2f958787c 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77980-condor.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77980-condor.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the Condor board
> + * Device Tree Source for the Condor board with R-Car V3H
>   *
>   * Copyright (C) 2018 Renesas Electronics Corp.
>   * Copyright (C) 2018 Cogent Embedded, Inc.
> diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> index 4715e4a4abe064cd..236671401bc1e4de 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the ebisu board
> + * Device Tree Source for the Ebisu board with R-Car E3
>   *
>   * Copyright (C) 2018 Renesas Electronics Corp.
>   */
> diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> index 6783c3ad08567e4b..9290ea49e2dd3c7a 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the Draak board
> + * Device Tree Source for the Draak board with R-Car D3
>   *
>   * Copyright (C) 2016-2018 Renesas Electronics Corp.
>   * Copyright (C) 2017 Glider bvba
> diff --git a/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts b/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
> index 687f019e79f059a1..dc671ff57ec7678a 100644
> --- a/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * Device Tree Source for the Falcon CPU and BreakOut boards
> + * Device Tree Source for the Falcon CPU and BreakOut boards with R-Car V3U
>   *
>   * Copyright (C) 2020 Renesas Electronics Corp.
>   */
> -- 
> 2.25.1
> 

-- 
Regards,
Niklas Söderlund

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2021-06-07 21:05 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-07 17:42 [PATCH] arm64: dts: renesas: rcar-gen3: Add SoC model to comment headers Geert Uytterhoeven
2021-06-07 21:02 ` Niklas Söderlund [this message]
2021-06-08  6:45   ` Geert Uytterhoeven

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=YL6JaF8GEiuqnh4M@oden.dyn.berto.se \
    --to=niklas.soderlund@ragnatech.se \
    --cc=geert+renesas@glider.be \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    /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;
as well as URLs for NNTP newsgroup(s).