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

Attachment: signature.asc
Description: PGP signature

Reply via email to