lkml.org 
[lkml]   [2015]   [May]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 00/12] ARM: berlin: add nand support
On Sat, May 16, 2015 at 06:37:51PM -0300, Ezequiel Garcia wrote:
> Hi Antoine,
>
> This is a long patchset, which involves some very invasive changes
> to the driver. So, I think it'll be better to split it in different
> patchsets.
>
> On 05/11/2015 11:58 AM, Antoine Tenart wrote:
> [..]
> >
> > Antoine Tenart (12):
>
> These two clock patches are related to Berlin support only.
>
> > mtd: pxa3xx_nand: add a non mandatory ECC clock
> > Documentation: bindings: document the clocks for pxa3xx-nand
>
> The next four patches rework the timing setup and remove the legacy
> device detection. I think we should try to get them merged in first place.
>
> > mtd: pxa3xx_nand: add a default chunk size
> > mtd: pxa3xx_nand: add helpers to setup the timings
> > mtd: pxa3xx_nand: rework flash detection and timing setup
> > mtd: pxa3xx_nand: clean up the pxa3xx timings
>
> Next one just adds support for a new device. It has little to do with
> the series. I already replied about using JEDEC ID; once we solve that,
> I suggest that you submit this patch alone.
>
> > mtd: nand: add Samsung K9GBG08U0A-M to nand_ids table
>
> And finally, once we solve the timing rework, we can go through Berlin
> support.
>
> > mtd: pxa3xx_nand: add support for the Marvell Berlin nand controller
> > Documentation: bindings: add the Berlin nand controller compatible
> > mtd: nand: let Marvell Berlin SoCs select the pxa3xx driver
> > ARM: berlin: add BG2Q node for the nand
> > ARM: berlin: enable flash on the BG2Q DMP
> >
>
> What do you think?

So we'd have 3 series:
- Timing rework.
- Berlin stuff.
- Samsung K9GBG08U0A-M support.

I'm OK with splitting the series to ease the review process :)

Antoine

--
Antoine Ténart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


\
 
 \ /
  Last update: 2015-05-20 16:41    [W:0.167 / U:0.164 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site