Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-20 Thread Brian Norris
Hi, On Tue, Jun 20, 2017 at 10:52:44AM +0200, Andrea Adami wrote: > Brian, > thanks for the review and for the unvaluable advices. > > I have almost fixed the problems you spotted and I am almost ready to > send the new v4 of the patchset. Great! > Some doubts are still present: I'll comment

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-20 Thread Brian Norris
Hi, On Tue, Jun 20, 2017 at 10:52:44AM +0200, Andrea Adami wrote: > Brian, > thanks for the review and for the unvaluable advices. > > I have almost fixed the problems you spotted and I am almost ready to > send the new v4 of the patchset. Great! > Some doubts are still present: I'll comment

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-20 Thread Andrea Adami
On Fri, Jun 9, 2017 at 3:42 AM, Brian Norris wrote: > Oh, one more thing: > > On Thu, Jun 01, 2017 at 12:40:50AM +0200, Andrea Adami wrote: >> diff --git a/drivers/mtd/sharpslpart.c b/drivers/mtd/sharpslpart.c >> new file mode 100644 >> index 000..40aebe9 >> ---

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-20 Thread Andrea Adami
On Fri, Jun 9, 2017 at 3:42 AM, Brian Norris wrote: > Oh, one more thing: > > On Thu, Jun 01, 2017 at 12:40:50AM +0200, Andrea Adami wrote: >> diff --git a/drivers/mtd/sharpslpart.c b/drivers/mtd/sharpslpart.c >> new file mode 100644 >> index 000..40aebe9 >> --- /dev/null >> +++

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-20 Thread Andrea Adami
Brian, thanks for the review and for the unvaluable advices. I have almost fixed the problems you spotted and I am almost ready to send the new v4 of the patchset. Some doubts are still present: I'll comment under your remarks: On Fri, Jun 9, 2017 at 3:30 AM, Brian Norris

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-20 Thread Andrea Adami
Brian, thanks for the review and for the unvaluable advices. I have almost fixed the problems you spotted and I am almost ready to send the new v4 of the patchset. Some doubts are still present: I'll comment under your remarks: On Fri, Jun 9, 2017 at 3:30 AM, Brian Norris wrote: > Hi, > > On

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-08 Thread Brian Norris
Oh, one more thing: On Thu, Jun 01, 2017 at 12:40:50AM +0200, Andrea Adami wrote: > diff --git a/drivers/mtd/sharpslpart.c b/drivers/mtd/sharpslpart.c > new file mode 100644 > index 000..40aebe9 > --- /dev/null > +++ b/drivers/mtd/sharpslpart.c > @@ -0,0 +1,132 @@ > +/* > + * MTD partition

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-08 Thread Brian Norris
Oh, one more thing: On Thu, Jun 01, 2017 at 12:40:50AM +0200, Andrea Adami wrote: > diff --git a/drivers/mtd/sharpslpart.c b/drivers/mtd/sharpslpart.c > new file mode 100644 > index 000..40aebe9 > --- /dev/null > +++ b/drivers/mtd/sharpslpart.c > @@ -0,0 +1,132 @@ > +/* > + * MTD partition

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-08 Thread Brian Norris
Hi, On Thu, Jun 01, 2017 at 12:40:50AM +0200, 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 >

Re: [PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-06-08 Thread Brian Norris
Hi, On Thu, Jun 01, 2017 at 12:40:50AM +0200, 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 >

[PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-05-31 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

[PATCH v3 1/9] mtd: sharpslpart: add sharpslpart MTD partition parser

2017-05-31 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