2021-01-12 13:48:41

by Adam Ford

[permalink] [raw]
Subject: [PATCH] arm64: defconfig: Enable CLK_RCAR_USB2_CLOCK_SEL

The RZ/G2 Series has the optional CLK_RCAR_USB2_CLOCK_SEL.
Enable it by default. It's disabled by default in the
the device tree, so it should be safe to enable it here.

Signed-off-by: Adam Ford <[email protected]>

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 838301650a79..84fcfa7a1865 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -932,6 +932,7 @@ CONFIG_SM_GCC_8250=y
CONFIG_SM_GPUCC_8150=y
CONFIG_SM_GPUCC_8250=y
CONFIG_QCOM_HFPLL=y
+CONFIG_CLK_RCAR_USB2_CLOCK_SEL=y
CONFIG_HWSPINLOCK=y
CONFIG_HWSPINLOCK_QCOM=y
CONFIG_ARM_MHU=y
--
2.25.1


2021-01-19 00:52:14

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable CLK_RCAR_USB2_CLOCK_SEL

Hi Adam,

On Tue, Jan 12, 2021 at 2:46 PM Adam Ford <[email protected]> wrote:
> The RZ/G2 Series has the optional CLK_RCAR_USB2_CLOCK_SEL.
> Enable it by default. It's disabled by default in the
> the device tree, so it should be safe to enable it here.
>
> Signed-off-by: Adam Ford <[email protected]>

Thanks for your patch!

Reviewed-by: Geert Uytterhoeven <[email protected]>

I will accept this patch once the Beacon Kit DTS starts using this, and
the patch description can be changed to "... enabled because it's used by
Beacon Kit".

> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -932,6 +932,7 @@ CONFIG_SM_GCC_8250=y
> CONFIG_SM_GPUCC_8150=y
> CONFIG_SM_GPUCC_8250=y
> CONFIG_QCOM_HFPLL=y
> +CONFIG_CLK_RCAR_USB2_CLOCK_SEL=y
> CONFIG_HWSPINLOCK=y
> CONFIG_HWSPINLOCK_QCOM=y
> CONFIG_ARM_MHU=y

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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