linux-kernel - RE: [[LINUX PATCH v10] 1/4] Devicetree: Add pl353 smc controller devicetree binding information

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <MWHPR02MB2623D909645D0AEB426C205CAF4F0@MWHPR02MB2623.namprd02.prod.outlook.com>
Date:   Thu, 28 Jun 2018 07:56:55 +0000
From:   Naga Sureshkumar Relli <nagasure@...inx.com>
To:     Linus Walleij <linus.walleij@...aro.org>
CC:     Boris Brezillon <boris.brezillon@...tlin.com>,
        Richard Weinberger <richard@....at>,
        David Woodhouse <dwmw2@...radead.org>,
        Brian Norris <computersforpeace@...il.com>,
        Mark Vasut <marek.vasut@...il.com>,
        Florian Fainelli <f.fainelli@...il.com>,
        Markus Mayer <mmayer@...adcom.com>,
        Roger Quadros <rogerq@...com>,
        Ladislav Michl <ladis@...ux-mips.org>,
        "ada@...rsis.com" <ada@...rsis.com>,
        "honghui.zhang@...iatek.com" <honghui.zhang@...iatek.com>,
        Miquèl Raynal <miquel.raynal@...tlin.com>,
        "linux-mtd@...ts.infradead.org" <linux-mtd@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "nagasureshkumarrelli@...il.com" <nagasureshkumarrelli@...il.com>,
        Michal Simek <michals@...inx.com>
Subject: RE: [[LINUX PATCH v10] 1/4] Devicetree: Add pl353 smc controller
 devicetree binding information

Hi Linus,

Thanks for the review.

> -----Original Message-----
> From: Linus Walleij [mailto:linus.walleij@...aro.org]
> Sent: Thursday, June 28, 2018 12:24 PM
> To: Naga Sureshkumar Relli <nagasure@...inx.com>
> Cc: Boris Brezillon <boris.brezillon@...tlin.com>; Richard Weinberger <richard@....at>;
> David Woodhouse <dwmw2@...radead.org>; Brian Norris
> <computersforpeace@...il.com>; Mark Vasut <marek.vasut@...il.com>; Florian Fainelli
> <f.fainelli@...il.com>; Markus Mayer <mmayer@...adcom.com>; Roger Quadros
> <rogerq@...com>; Ladislav Michl <ladis@...ux-mips.org>; ada@...rsis.com;
> honghui.zhang@...iatek.com; Miquèl Raynal <miquel.raynal@...tlin.com>; linux-
> mtd@...ts.infradead.org; linux-kernel@...r.kernel.org; nagasureshkumarrelli@...il.com;
> Michal Simek <michals@...inx.com>
> Subject: Re: [[LINUX PATCH v10] 1/4] Devicetree: Add pl353 smc controller devicetree
> binding information
> 
> On Thu, Jun 21, 2018 at 8:43 AM Naga Sureshkumar Relli
> <naga.sureshkumar.relli@...inx.com> wrote:
> 
> Thanks for your patch!
> 
> > Add pl353 static memory controller devicetree binding information.
> >
> > Signed-off-by: Naga Sureshkumar Relli
> > <naga.sureshkumar.relli@...inx.com>
> > +Device tree bindings for ARM PL353 static memory controller
> > +
> > +PL353 static memory controller supports two kinds of memory
> > +interfaces.i.e NAND and SRAM/NOR interfaces.
> > +The actual devices are instantiated from the child nodes of pl353 smc node.
> 
> See and reference the special PrimeCell DT bindings:
> Documentation/devicetree/bindings/arm/primecell.txt
> 
> > +Required properties:
> > +- compatible           : Must be "arm,pl353-smc-r2p1"
> 
> Should be:
> 
> "arm,pl353-smc-r2p1", "arm,primecell";
> 
> It will be possible for the Linux AMBA core to probe the device from just reading out the
> primecell ID registers, so the first compatible string will not even be used by many OS:es.
Thanks for pointing the primecell.txt, will append the "arm,primecell" entry in next version.
> 
> > +- reg                  : Controller registers map and length.
> > +- clock-names          : List of input clock names - "ref_clk", "aper_clk"
> > +                         (See clock bindings for details).
> 
> The AMBA primecell peripheral clock (clocking the silicon) must be named "apb_pclk".
Ok, will update in next version.
> 
> > +- clocks               : Clock phandles (see clock bindings for details).
> > +- address-cells                : Must be 1.
> > +- size-cells           : Must be 1.
> > +
> > +Child nodes:
> > + For NAND the "arm,pl353-nand-r2p1" and for NOR the "cfi-flash"
> > +drivers are supported as child nodes.
> > +
> > +for NAND partition information please refer the below file
> > +Documentation/devicetree/bindings/mtd/partition.txt
> > +
> > +Example:
> > +       smcc: memory-controller@...0e000
> > +                       compatible = "arm,pl353-smc-r2p1"
> 
> Add , "arm,primecell";
Ok. Will update in next version.
> 
> > +                       clock-names = "memclk", "aclk";
> 
> This doesn't even have the same name as in the bindings above, I think the latter should be
> named "apb_pclk".
Ok, it’s a mistake, I will correct it in next version.
> 
> Otherwise it looks good!
> 
> Yours,
> Linus Walleij

Thanks for your review.

Thanks,
Naga Sureshkumar Relli