Re: [PATCH 0/5] arm64: renesas: Split/rename R-Car H3 support

["Niklas Söderlund" <niklas.soderlund@ragnatech.se>]

Hi Geert,

Nice cleanup of the otherwise sometimes confusing names :-)

For the whole series,

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

On 2019-12-17 19:38:36 +0100, Geert Uytterhoeven wrote:
> 	Hi all,
> 
> Despite using the same compatible values ("r8a7795"-based) because of
> historical reasons, R-Car H3 ES1.x (R8A77950) and R-Car H3 ES2.0+
> (R8A77951) are really different SoCs, with different part numbers.
> 
> Hence this patch series splits the config symbols for R-Car H3, and
> renames the related DTS files, to maintain a clear separation between
> early (ES1.x) and later (ES2.0+) SoC revisions.  This will pave the way
> for configuring out support for early SoC revisions, which can reduce
> kernel size, especially in the pin control subsystem.
> This is similar to the recent split of R8A7796 symbols for R-Car M3-W
> (R8A77960) and M3-W+ (R8A77961)[1], and the related DTS file renames[2],
> but different due to the sharing of compatible values between R-Car H3
> ES1.x and H3 ES2.0+.
> 
> This series also includes the rename/cleanup of the ULCB DTS file names,
> as suggested by Eugeniu Rosca.  As DTS files are already being renamed
> for v5.6[2], it makes sense to combine them with other renames, to avoid
> inconveniencing the user with multiple renames in multiple kernel
> versions.
> 
> The pin control part will be handled in a separate patch.
> 
> For your convenience, all of this is available in the
> topic/r8a7795-rename-v1 branch of my renesas-drivers git repository at
> git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git.
> 
> Thanks for your comments!
> 
> References:
>   [1] "[PATCH v2 00/11] arm64: renesas: Add r8a77961 support"
>       https://lore.kernel.org/linux-renesas-soc/20191023123342.13100-1-geert+renesas@glider.be/
>   [2] "[PATCH/RFC] arm64: dts: renesas: Rename r8a7796* to r8a77960*"
>       https://lore.kernel.org/linux-renesas-soc/20191211131311.23661-1-geert+renesas@glider.be/
> 
> Geert Uytterhoeven (5):
>   arm64: dts: renesas: Rename r8a7795{-es1,}* to r8a7795[01]*
>   arm64: dts: renesas: Drop redudant SoC prefixes from ULCB DTS file
>     names
>   arm64: dts: renesas: Sort DTBs in Makefile
>   arm64: dts: renesas: Prepare for split of ARCH_R8A7795 into
>     ARCH_R8A7795[01]
>   soc: renesas: Add ARCH_R8A7795[01] for existing R-Car H3
> 
>  arch/arm64/boot/dts/renesas/Makefile          | 21 ++++++++++---------
>  ...salvator-x.dts => r8a77950-salvator-x.dts} |  4 ++--
>  ...795-h3ulcb-kf.dts => r8a77950-ulcb-kf.dts} |  4 ++--
>  ...a7795-es1-h3ulcb.dts => r8a77950-ulcb.dts} |  4 ++--
>  .../{r8a7795-es1.dtsi => r8a77950.dtsi}       |  4 ++--
>  ...salvator-x.dts => r8a77951-salvator-x.dts} |  4 ++--
>  ...lvator-xs.dts => r8a77951-salvator-xs.dts} |  8 +++----
>  ...es1-h3ulcb-kf.dts => r8a77951-ulcb-kf.dts} |  4 ++--
>  .../{r8a7795-h3ulcb.dts => r8a77951-ulcb.dts} |  4 ++--
>  .../renesas/{r8a7795.dtsi => r8a77951.dtsi}   |  2 +-
>  ...960-m3ulcb-kf.dts => r8a77960-ulcb-kf.dts} |  2 +-
>  ...{r8a77960-m3ulcb.dts => r8a77960-ulcb.dts} |  0
>  ...65-m3nulcb-kf.dts => r8a77965-ulcb-kf.dts} |  2 +-
>  ...r8a77965-m3nulcb.dts => r8a77965-ulcb.dts} |  0
>  drivers/soc/renesas/Kconfig                   |  8 +++++++
>  15 files changed, 40 insertions(+), 31 deletions(-)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-es1-salvator-x.dts => r8a77950-salvator-x.dts} (96%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-h3ulcb-kf.dts => r8a77950-ulcb-kf.dts} (75%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-es1-h3ulcb.dts => r8a77950-ulcb.dts} (89%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-es1.dtsi => r8a77950.dtsi} (98%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-salvator-x.dts => r8a77951-salvator-x.dts} (96%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-salvator-xs.dts => r8a77951-salvator-xs.dts} (96%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-es1-h3ulcb-kf.dts => r8a77951-ulcb-kf.dts} (75%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795-h3ulcb.dts => r8a77951-ulcb.dts} (92%)
>  rename arch/arm64/boot/dts/renesas/{r8a7795.dtsi => r8a77951.dtsi} (99%)
>  rename arch/arm64/boot/dts/renesas/{r8a77960-m3ulcb-kf.dts => r8a77960-ulcb-kf.dts} (92%)
>  rename arch/arm64/boot/dts/renesas/{r8a77960-m3ulcb.dts => r8a77960-ulcb.dts} (100%)
>  rename arch/arm64/boot/dts/renesas/{r8a77965-m3nulcb-kf.dts => r8a77965-ulcb-kf.dts} (92%)
>  rename arch/arm64/boot/dts/renesas/{r8a77965-m3nulcb.dts => r8a77965-ulcb.dts} (100%)
> 
> -- 
> 2.17.1
> 
> Gr{oetje,eeting}s,
> 
> 						Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> 							    -- Linus Torvalds

-- 
Regards,
Niklas Söderlund