On 8/24/21 8:13 PM, Adam Ford wrote:
On Tue, Aug 24, 2021 at 1:02 PM Marek Vasut <ma...@denx.de> wrote:

On 8/24/21 6:43 PM, Adam Ford wrote:
There are two versaclock programmable clock chips used to provide
reference clocks for both USB and Ethernet.  Enable the driver
in the configs to run these clocks.

Signed-off-by: Adam Ford <aford...@gmail.com>

diff --git a/configs/r8a774a1_beacon_defconfig 
b/configs/r8a774a1_beacon_defconfig
index 7ba4ac05f5..002b91176f 100644
--- a/configs/r8a774a1_beacon_defconfig
+++ b/configs/r8a774a1_beacon_defconfig
@@ -39,7 +39,9 @@ CONFIG_VERSION_VARIABLE=y
   CONFIG_REGMAP=y
   CONFIG_SYSCON=y
   CONFIG_CLK=y
+CONFIG_CLK_CCF=y
   CONFIG_CLK_RENESAS=y
+CONFIG_CLK_VERSACLOCK=y

There is a lot of duplication here, look into
arch/arm/mach-rmobile/Kconfig.64 and for TARGET_BEACON_RZG2M (where are
the others?) add "select CLK_VERSACLOCK" . You can also fix it up and
add whatever other mandatory config options should be enabled for that
platform. And then you reduce the defconfig too.

Are you OK if i put that into board/beacon/beacon-rzg2m/Kconfig?
there is a check to see if TARGET_BEACON_RZG2H || TARGET_BEACON_RZG2M
|| TARGET_BEACON_RZG2N is set.  From there, i should be able to select
the common options that are required and it won't have to be
duplicated in either the defconfigs or the
arch/arm/mach-rmobile/Kconfig.64

Yeah, the less duplication the better.

Reply via email to