Messages in this thread | | | From | Naga Sureshkumar Relli <> | Subject | RE: [[LINUX PATCH v10] 4/4] mtd: rawnand: pl353: Add basic driver for arm pl353 smc nand interface | Date | Fri, 29 Jun 2018 04:15:24 +0000 |
| |
Hi Linus,
> -----Original Message----- > From: Linus Walleij [mailto:linus.walleij@linaro.org] > Sent: Thursday, June 28, 2018 11:44 PM > To: Naga Sureshkumar Relli <nagasure@xilinx.com> > Cc: Boris Brezillon <boris.brezillon@bootlin.com>; Richard Weinberger <richard@nod.at>; > David Woodhouse <dwmw2@infradead.org>; Brian Norris > <computersforpeace@gmail.com>; Mark Vasut <marek.vasut@gmail.com>; Florian Fainelli > <f.fainelli@gmail.com>; Markus Mayer <mmayer@broadcom.com>; Roger Quadros > <rogerq@ti.com>; Ladislav Michl <ladis@linux-mips.org>; ada@thorsis.com; > honghui.zhang@mediatek.com; Miquèl Raynal <miquel.raynal@bootlin.com>; linux- > mtd@lists.infradead.org; linux-kernel@vger.kernel.org; nagasureshkumarrelli@gmail.com; > Michal Simek <michals@xilinx.com> > Subject: Re: [[LINUX PATCH v10] 4/4] mtd: rawnand: pl353: Add basic driver for arm > pl353 smc nand interface > > On Thu, Jun 28, 2018 at 2:13 PM Naga Sureshkumar Relli <nagasure@xilinx.com> wrote: > > > > > This driver has the same problem as the other patches: > > > use the ARM_AMBA primecell magic numbers detection, and the PrimeCell bus. > > > > Here the child is NAND controller and the parent is PL353 SMC, so do > > we need to update this also as AMBA driver? > > No, sorry. The parent is spawning another device and you should indeed use a platform device > for that. > Ok, you mean platform driver model is sufficient for both pl353-smc.c and pl353_nand.c drivers? Or AMBA model is for pl353-smc and platform driver model is for pl353-nand?
Thanks, Naga Sureshkumar Relli.
> Forget this comment. > > Yours, > Linus Walleij
| |