On 02/10/2023 07:56, Kuninori Morimoto wrote:
> Add initial support for the R-Car S4 Starter Kit with R8A779F4
> SoC support. Based on a patch in the BSP.
>
> Signed-off-by: Michael Dege <michael.dege@xxxxxxxxxxx>
> Signed-off-by: Yusuke Goda <yusuke.goda.sx@xxxxxxxxxxx>
> Signed-off-by: Tam Nguyen <tam.nguyen.xa@xxxxxxxxxxx>
> Signed-off-by: Hai Pham <hai.pham.ud@xxxxxxxxxxx>
> Co-developed-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> ---
> arch/arm64/boot/dts/renesas/Makefile | 1 +
> arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts | 240 ++++++++++++++++++
> 2 files changed, 241 insertions(+)
> create mode 100644 arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts
>
> diff --git a/arch/arm64/boot/dts/renesas/Makefile b/arch/arm64/boot/dts/renesas/Makefile
> index 7114cbbd8713..751cb0c65dcd 100644
> --- a/arch/arm64/boot/dts/renesas/Makefile
> +++ b/arch/arm64/boot/dts/renesas/Makefile
> @@ -61,6 +61,7 @@ dtb-$(CONFIG_ARCH_R8A77995) += r8a77995-draak.dtb
> dtb-$(CONFIG_ARCH_R8A779A0) += r8a779a0-falcon.dtb
>
> dtb-$(CONFIG_ARCH_R8A779F0) += r8a779f0-spider.dtb
> +dtb-$(CONFIG_ARCH_R8A779F0) += r8a779f4-s4sk.dtb
>
> dtb-$(CONFIG_ARCH_R8A779G0) += r8a779g0-white-hawk.dtb
> dtb-$(CONFIG_ARCH_R8A779G0) += r8a779g0-white-hawk-ard-audio-da7212.dtbo
> diff --git a/arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts b/arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts
> new file mode 100644
> index 000000000000..abfda5c6ca16
> --- /dev/null
> +++ b/arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts
> @@ -0,0 +1,240 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +/*
> + * Device Tree Source for the R-Car S4 Starter Kit board
> + *
> + * Copyright (C) 2023 Renesas Electronics Corp.
> + */
> +
> +/dts-v1/;
> +#include <dt-bindings/gpio/gpio.h>
> +#include "r8a779f4.dtsi"
> +
> +/ {
> + model = "R-Car S4 Starter Kit board";
> + compatible = "renesas,s4sk", "renesas,r8a779f4", "renesas,r8a779f0";
> +
> + aliases {
> + serial0 = &hscif0;
> + serial1 = &hscif1;
> + eth0 = &rswitch;
> + };
> +
> + chosen {
> + bootargs = "ignore_loglevel rw root=/dev/nfs ip=on";
This does not look really hardware-independent and suitable for entire
community, but rather like your personal setup. Please drop entire
bootargs or share why this is needed for every user in the world of
mainline kernel.
> + stdout-path = "serial0:921600n8";
> + };
> +
Best regards,
Krzysztof
