[PATCH 06/16] arm: rmobile: Add basic R9A07G044L SoC support

[Marek Vasut]

On 10/4/23 15:26, Paul Barker wrote:
> On Wed, Oct 04, 2023 at 02:15:24PM +0200, Marek Vasut wrote:
>> On 10/4/23 09:59, Paul Barker wrote:
>>> On 03/10/2023 13:46, Marek Vasut wrote:
>>>> On 9/20/23 14:42, Paul Barker wrote:
>>>>> Add a config option for the R9A07G044L SoC used in the RZ/G2L so that we
>>>>> can make use of this in the subsequent driver patches.
>>>>>
>>>>> Signed-off-by: Paul Barker <paul.barker.ct at bp.renesas.com>
>>>>> Reviewed-by: Biju Das <biju.das.jz at bp.renesas.com>
>>>>> Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj at bp.renesas.com>
>>>>> ---
>>>>>     arch/arm/mach-rmobile/Kconfig.rzg2l | 5 +++++
>>>>>     1 file changed, 5 insertions(+)
>>>>>
>>>>> diff --git a/arch/arm/mach-rmobile/Kconfig.rzg2l b/arch/arm/mach-rmobile/Kconfig.rzg2l
>>>>> index 37ff6cd34160..266f82c18085 100644
>>>>> --- a/arch/arm/mach-rmobile/Kconfig.rzg2l
>>>>> +++ b/arch/arm/mach-rmobile/Kconfig.rzg2l
>>>>> @@ -3,6 +3,11 @@
>>>>>     if RZG2L
>>>>> +config R9A07G044L
>>>>> +	bool "Renesas R9A07G044L SoC"
>>>>> +	help
>>>>> +	  Enable support for the R9A07G044L SoC used in the RZ/G2L.
>>>>
>>>> I was under the impression that RZ/G2L is a SoC family, so the
>>>> R9A07G044L is part of that SoC family ? Maybe just reword this paragraph
>>>> and the commit message a bit to make it less confusing.
>>>
>>> The confusion is hard to avoid (see my reply re patch 5 of this series),
>>> but I'll see if I can improve this a bit in v2.
>>
>> So uh ... RZ/G2L is both the name of R9A07G044L SoC (what's the R9... model
>> number of e.g. RZ/G2LC?) and a family name ?
> 
> Yes. Definitely easy to confuse!
> 
> We have (with differences from the RZ/G2L model numbers highlighted):
>   * RZ/G2L  (R9A07G044L1 or R9A07G044L2)
>   * RZ/G2LC (R9A07G044*C*1 or R9A07G044*C*2)
>   * RZ/G2UL (R9A07G04*3U1*)
>   * RZ/V2L  (R9A07G0*5*4L1 or R9A07G0*5*4L2)
>   * RZ/Five (R9A07G04*3F0*)
> 
> The last digit indicates the number of cores (1 or 2), except for on the
> RZ/Five which has one core but the model number ends in a zero.

OK, got it, thanks !