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...@bp.renesas.com> > > > > Reviewed-by: Biju Das <biju.das...@bp.renesas.com> > > > > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad...@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. Thanks, Paul
signature.asc
Description: PGP signature