[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <BY2PR02MB1411D166CF8D31421D183A5DAF9A0@BY2PR02MB1411.namprd02.prod.outlook.com>
Date: Tue, 8 May 2018 06:35:46 +0000
From: Naga Sureshkumar Relli <nagasure@...inx.com>
To: Julia Cartwright <julia@...com>
CC: "nagasureshkumarrelli@...il.com" <nagasureshkumarrelli@...il.com>,
"boris.brezillon@...tlin.com" <boris.brezillon@...tlin.com>,
"rogerq@...com" <rogerq@...com>,
"lee.jones@...aro.org" <lee.jones@...aro.org>,
"alexandre.belloni@...e-electrons.com"
<alexandre.belloni@...e-electrons.com>,
"nicolas.ferre@...rochip.com" <nicolas.ferre@...rochip.com>,
"ladis@...ux-mips.org" <ladis@...ux-mips.org>,
"ada@...rsis.com" <ada@...rsis.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [LINUX PATCH v8 2/2] memory: pl353: Add driver for arm pl353
static memory controller
> -----Original Message-----
> From: Julia Cartwright [mailto:julia@...com]
> Sent: Monday, May 7, 2018 10:23 PM
> To: Naga Sureshkumar Relli <nagasure@...inx.com>
> Cc: nagasureshkumarrelli@...il.com; boris.brezillon@...tlin.com; rogerq@...com;
> lee.jones@...aro.org; alexandre.belloni@...e-electrons.com; nicolas.ferre@...rochip.com;
> ladis@...ux-mips.org; ada@...rsis.com; linux-kernel@...r.kernel.org
> Subject: Re: [LINUX PATCH v8 2/2] memory: pl353: Add driver for arm pl353 static
> memory controller
>
> On Mon, May 07, 2018 at 10:12:28AM +0000, Naga Sureshkumar Relli wrote:
> > Hi Julia,
> >
> > Thanks for reviewing the patch and Sorry for my late reply. This patch
> > went to junk folder, hence I didn't catch this patch.
> >
> > From: Julia Cartwright [mailto:julia@...com]
> [..]
> > >
> > > It would be easier to follow if you constructed your two patchsets
> > > with git format-patch -- thread.
> > >
> >
> > I am using the same but with out --thread.
> >
> > > Or, merge them into a single patchset, especially considering the dependency between
> patches.
> >
> > But both are different patches, one for Device tree documentation and other for driver
> update.
>
> Yes, I'm not proposing you merge _patches_ but _patchsets_. You have two patchsets, one for
> the SMC driver, and another for the NAND. Given that they depend on one another, it's
> helpful for reviewers if you sent them all together, with a cover letter which describes the
> entire patchset, changelog, it's dependencies, revision changelog, etc.
>
> Something like:
>
> [PATCH v9 0/4] rawnand: memory: add support for PL353 static memory controller +
> NAND
> [PATCH v9 1/4] Devicetree: Add pl353 smc controller devicetree binding information
> [PATCH v9 2/4] memory: pl353: Add driver for arm pl353 static memory controller
> [PATCH v9 3/4] Documentation: nand: pl353: Add documentation for controller and
> driver
> [PATCH v9 4/4] mtd: rawnand: pl353: Add basic driver for arm pl353 smc nand interface
>
> Anyway, just with --thread enabled would be an improvement.
Ok. Got it. But both are different layers hence I sent like that.
>
> [..]
> > > > --- a/drivers/memory/Kconfig
> > > > +++ b/drivers/memory/Kconfig
> > > > @@ -152,6 +152,13 @@ config DA8XX_DDRCTL
> > > > This driver is for the DDR2/mDDR Memory Controller present on
> > > > Texas Instruments da8xx SoCs. It's used to tweak various memory
> > > > controller configuration options.
> > > > +config PL35X_SMC
> > > > + bool "ARM PL35X Static Memory Controller(SMC) driver"
> > >
> > > Is there any reason why this can't be tristate?
> >
> > There is a Nand driver which uses this driver. i.e The NAND driver
> > Depends on this driver.
>
> That's true, but it's irrelevant to question I asked. It is perfectly valid for both the SMC and
> NAND drivers to be tristate, why are you not allowing this configuration?
Yes, I will update it in next version of patch set.
Thanks,
Naga Sureshkumar Relli
>
> Julia
Powered by blists - more mailing lists