Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-06-09 Thread Brian Norris
Hi Boris, On Fri, Jun 09, 2017 at 09:16:43AM +0200, Boris Brezillon wrote: > On Thu, 8 Jun 2017 19:32:51 -0700 > Brian Norris wrote: > > On Tue, Apr 18, 2017 at 11:35:56AM +0200, Boris Brezillon wrote: > > > On Tue, 18 Apr 2017 10:58:02 +0200 > > > Andrea Adami wrote: > > > > On Mon, Apr 17, 2

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-06-09 Thread Boris Brezillon
Hi Brian, On Thu, 8 Jun 2017 19:32:51 -0700 Brian Norris wrote: > Hi Boris and Andrea, > > Sorry I didn't thoroughly read through this earlier discussion before > reviewing the later versions. I also don't want to rehash old > disagreements. But I had a few questions. > > On Tue, Apr 18, 2017

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-06-08 Thread Brian Norris
Hi Boris and Andrea, Sorry I didn't thoroughly read through this earlier discussion before reviewing the later versions. I also don't want to rehash old disagreements. But I had a few questions. On Tue, Apr 18, 2017 at 11:35:56AM +0200, Boris Brezillon wrote: > On Tue, 18 Apr 2017 10:58:02 +0200

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-18 Thread Dmitry Eremin-Solenikov
2017-04-17 17:44 GMT+03:00 Boris Brezillon : > Marek, Andrea, > > Before we even start discussing minor improvements (like coding style), > I'd like to discuss the sharp FTL and partition table format, and > decide whether we want to have such an old FTL included in the kernel. > > Actually, that's

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-18 Thread Boris Brezillon
Hi Andrea, You know what, I give up, since even if I review and find a problem in the FTL/partition-table format/approach, you'll keep arguing that it should be supported in the kernel. I think I have enough things on my plate to not spend extra time on this. I'll let others review and take the f

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-18 Thread Andrea Adami
On Tue, Apr 18, 2017 at 11:35 AM, Boris Brezillon wrote: > Hi Andrea, > > On Tue, 18 Apr 2017 10:58:02 +0200 > Andrea Adami wrote: > >> Boris, >> thanks for having read it. >> >> On Mon, Apr 17, 2017 at 4:44 PM, Boris Brezillon < >> boris.brezil...@free-electrons.com> wrote: >> > Marek, Andrea, >

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-18 Thread Boris Brezillon
Hi Andrea, On Tue, 18 Apr 2017 10:58:02 +0200 Andrea Adami wrote: > Boris, > thanks for having read it. > > On Mon, Apr 17, 2017 at 4:44 PM, Boris Brezillon < > boris.brezil...@free-electrons.com> wrote: > > Marek, Andrea, > > > > Before we even start discussing minor improvements (like coding

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-18 Thread Andrea Adami
RESEND (sorry for the HTML) Boris, thanks for having read it. On Mon, Apr 17, 2017 at 4:44 PM, Boris Brezillon wrote: > Marek, Andrea, > > Before we even start discussing minor improvements (like coding style), > I'd like to discuss the sharp FTL and partition table format, and > decide whether

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-18 Thread Andrea Adami
On Sun, Apr 16, 2017 at 6:07 PM, Marek Vasut wrote: > On 04/15/2017 10:11 PM, Andrea Adami wrote: >> The Sharp SL Series (Zaurus) PXA handhelds have 16/64/128M of NAND flash >> and share the same layout of the first 7M partition, managed by Sharp FTL. >> >> The purpose of this self-contained patch

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-17 Thread Boris Brezillon
Marek, Andrea, Before we even start discussing minor improvements (like coding style), I'd like to discuss the sharp FTL and partition table format, and decide whether we want to have such an old FTL included in the kernel. Actually, that's the very reason I asked Andrea to post his series as soo

Re: [PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-16 Thread Marek Vasut
On 04/15/2017 10:11 PM, Andrea Adami wrote: > The Sharp SL Series (Zaurus) PXA handhelds have 16/64/128M of NAND flash > and share the same layout of the first 7M partition, managed by Sharp FTL. > > The purpose of this self-contained patch is to add a common parser and > remove the hardcoded size

[PATCH 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-15 Thread Andrea Adami
The Sharp SL Series (Zaurus) PXA handhelds have 16/64/128M of NAND flash and share the same layout of the first 7M partition, managed by Sharp FTL. The purpose of this self-contained patch is to add a common parser and remove the hardcoded sizes in the board files (these devices are not yet conver