linux-kernel - RE: [PATCH] soc: renesas: Add support for reading product revision for RZ/G2L family

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Thu, 20 Jan 2022 09:40:31 +0000
From:   Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@...renesas.com>
To:     Geert Uytterhoeven <geert@...ux-m68k.org>
CC:     Magnus Damm <magnus.damm@...il.com>,
        Biju Das <biju.das.jz@...renesas.com>,
        Linux-Renesas <linux-renesas-soc@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] soc: renesas: Add support for reading product revision
 for RZ/G2L family

Hi Geert,

Thank you for the review.

> -----Original Message-----
> From: Geert Uytterhoeven <geert@...ux-m68k.org>
> Sent: 20 January 2022 08:45
> To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@...renesas.com>
> Cc: Magnus Damm <magnus.damm@...il.com>; Biju Das <biju.das.jz@...renesas.com>; Linux-Renesas <linux-
> renesas-soc@...r.kernel.org>; Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
> Subject: Re: [PATCH] soc: renesas: Add support for reading product revision for RZ/G2L family
> 
> Hi Prabhakar,
> 
> On Mon, Dec 20, 2021 at 6:04 PM Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com> wrote:
> > From: Biju Das <biju.das.jz@...renesas.com>
> >
> > As per RZ/G2L HW manual (Rev.1.00 Sep, 2021) DEV_ID [31:28] indicates
> > product revision. Use this information to populate the revision info
> > for RZ/G2L family.
> >
> > Signed-off-by: Biju Das <biju.das.jz@...renesas.com>
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> 
> Thanks for your patch!
> 
> > Below is the log from Renesas RZ/G2L SMARC EVK:
> >
> > root@...rc-rzg2l:~#
> > at /sys/devices/soc0/$i; donemachine family soc_id revision; do echo
> > -n "$i: ";ca
> 
> This looks a bit mangled ;-)
> 
Typical Tera Term.

> > machine: Renesas SMARC EVK based on r9a07g044l2
> > family: RZ/G2L
> > soc_id: r9a07g044
> > revision: Rev 1
> > root@...rc-rzg2l:~#
> >
> > Cheers,
> > Prabhakar
> > ---
> >  drivers/soc/renesas/renesas-soc.c | 31
> > +++++++++++++++++--------------
> >  1 file changed, 17 insertions(+), 14 deletions(-)
> >
> > diff --git a/drivers/soc/renesas/renesas-soc.c
> > b/drivers/soc/renesas/renesas-soc.c
> > index 8f82749f182f..6ecd2763d100 100644
> > --- a/drivers/soc/renesas/renesas-soc.c
> > +++ b/drivers/soc/renesas/renesas-soc.c
> > @@ -416,6 +416,17 @@ static int __init renesas_soc_init(void)
> >                 chipid = ioremap(family->reg, 4);
> >         }
> >
> > +       soc_dev_attr = kzalloc(sizeof(*soc_dev_attr), GFP_KERNEL);
> 
> This is not freed in case of SoC mismatch error below.
> 
Ouch.

> > +       if (!soc_dev_attr)
> > +               return -ENOMEM;
> > +
> > +       np = of_find_node_by_path("/");
> > +       of_property_read_string(np, "model", &soc_dev_attr->machine);
> > +       of_node_put(np);
> > +
> > +       soc_dev_attr->family = kstrdup_const(family->name, GFP_KERNEL);
> > +       soc_dev_attr->soc_id = kstrdup_const(soc_id, GFP_KERNEL);
> > +
> >         if (chipid) {
> >                 product = readl(chipid + id->offset);
> >                 iounmap(chipid);
> > @@ -430,6 +441,12 @@ static int __init renesas_soc_init(void)
> >
> >                         eshi = ((product >> 4) & 0x0f) + 1;
> >                         eslo = product & 0xf;
> > +                       soc_dev_attr->revision = kasprintf(GFP_KERNEL, "ES%u.%u",
> > +                                                          eshi, eslo);
> > +               }  else if (id == &id_rzg2l) {
> > +                       eshi =  ((product >> 28) & 0x0f);
> > +                       soc_dev_attr->revision = kasprintf(GFP_KERNEL, "Rev %u",
> > +                                                          eshi);
> 
> These are not freed in case of SoC mismatch error below.
> 
Will fix that.

Cheers,
Prabhakar