Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-11-11 Thread Brian Norris
On Thu, Nov 05, 2015 at 09:49:30AM +0100, Boris Brezillon wrote: > On Fri, 30 Oct 2015 20:33:28 -0700 > Brian Norris wrote: > > > --- a/drivers/mtd/maps/lantiq-flash.c > > +++ b/drivers/mtd/maps/lantiq-flash.c > > @@ -110,7 +110,6 @@ ltq_copy_to(struct map_info *map, unsigned long to, > >

Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-11-11 Thread Brian Norris
On Thu, Nov 05, 2015 at 09:49:30AM +0100, Boris Brezillon wrote: > On Fri, 30 Oct 2015 20:33:28 -0700 > Brian Norris wrote: > > > --- a/drivers/mtd/maps/lantiq-flash.c > > +++ b/drivers/mtd/maps/lantiq-flash.c > > @@ -110,7 +110,6 @@ ltq_copy_to(struct map_info *map,

Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-11-05 Thread Boris Brezillon
On Fri, 30 Oct 2015 20:33:28 -0700 Brian Norris wrote: > --- a/drivers/mtd/maps/lantiq-flash.c > +++ b/drivers/mtd/maps/lantiq-flash.c > @@ -110,7 +110,6 @@ ltq_copy_to(struct map_info *map, unsigned long to, > static int > ltq_mtd_probe(struct platform_device *pdev) > { > - struct

Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-11-05 Thread Boris Brezillon
On Fri, 30 Oct 2015 20:33:28 -0700 Brian Norris wrote: > --- a/drivers/mtd/maps/lantiq-flash.c > +++ b/drivers/mtd/maps/lantiq-flash.c > @@ -110,7 +110,6 @@ ltq_copy_to(struct map_info *map, unsigned long to, > static int > ltq_mtd_probe(struct platform_device

Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-10-31 Thread Brian Norris
On Sat, Oct 31, 2015 at 04:26:32PM +0100, Marek Vasut wrote: > On Saturday, October 31, 2015 at 04:33:28 AM, Brian Norris wrote: > > - mtd_device_parse_register(info->cmtd, part_probe_types, , > > + mtd_device_parse_register(info->cmtd, part_probe_types, NULL, > > Did you miss this one ?

Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-10-31 Thread Marek Vasut
On Saturday, October 31, 2015 at 04:33:28 AM, Brian Norris wrote: > We should assign the MTD dev.of_node instead of the parser data field. > This gets us the equivalent partition parser behavior with fewer special > fields and parameter passing. > > Also convert several of these to

Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-10-31 Thread Marek Vasut
On Saturday, October 31, 2015 at 04:33:28 AM, Brian Norris wrote: > We should assign the MTD dev.of_node instead of the parser data field. > This gets us the equivalent partition parser behavior with fewer special > fields and parameter passing. > > Also convert several of these to

Re: [PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-10-31 Thread Brian Norris
On Sat, Oct 31, 2015 at 04:26:32PM +0100, Marek Vasut wrote: > On Saturday, October 31, 2015 at 04:33:28 AM, Brian Norris wrote: > > - mtd_device_parse_register(info->cmtd, part_probe_types, , > > + mtd_device_parse_register(info->cmtd, part_probe_types, NULL, > > Did you miss this one ?

[PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-10-30 Thread Brian Norris
We should assign the MTD dev.of_node instead of the parser data field. This gets us the equivalent partition parser behavior with fewer special fields and parameter passing. Also convert several of these to mtd_device_register(), since we don't need the 2nd and 3rd parameters anymore.

[PATCH v2 09/11] mtd: drop unnecessary partition parser data

2015-10-30 Thread Brian Norris
We should assign the MTD dev.of_node instead of the parser data field. This gets us the equivalent partition parser behavior with fewer special fields and parameter passing. Also convert several of these to mtd_device_register(), since we don't need the 2nd and 3rd parameters anymore.