[PATCH v4 07/10] ARM: rzn1: basic support for Renesas RZ/N1 SoC

[Marek Vasut]

On 4/17/23 22:44, Ralph Siemsen wrote:
> On Mon, Apr 17, 2023 at 10:30:26PM +0200, Marek Vasut wrote:
>> I mean, if you drop 'CLK' and 'CLK_RENESAS' from this list, do they 
>> still get selected because 'CLK_R8A06G032' is in this list ? If yes, 
>> then drop them from this list, else keep them.
> 
> If I drop the first two, I get some kconfig warnings:
> 
> WARNING: unmet direct dependencies detected for CLK_R9A06G032
>    Depends on [n]: CLK_RENESAS [=n]
>    Selected by [y]:
>    - ARCH_RZN1 [=y] && <choice>
> 
> And there is no output on the UART.

Aha, so one has to list all of them deps. In that case, no change needed 
here.

>>> I don't have a strong preference, if moving to mach-rmobile makes 
>>> sense, let me know and I can give it a go.
>>
>> I think it does make sense to keep all the renesas stuff in 
>> mach-rmobile . We can rename that to mach-renesas in some subsequent 
>> patch.
> 
> Okay, I will try moving it. Does this also then mean I should abandon 
> ARCH_RZN1 in favour of ARCH_RMOBILE? This could be more challenging...

I think so, RZA1 is also not rmobile and I think it has its own RZA1 
thing already, see:
arch/arm/mach-rmobile/Kconfig:config RZA1

So in your case, it would be RZN1 , just like that.