RE: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-09-02 Thread Hu Mingkai-B21284
; David Woodhouse Subject: Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions On Tue, Aug 10, 2010 at 08:56:42AM -0600, Grant Likely wrote: [...] The suggestion has been raised to use something like bus notifiers to get a hook onto

RE: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-09-02 Thread Hu Mingkai-B21284
- gene...@lists.sourceforge.net Subject: Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions On Tue, Aug 10, 2010 at 11:23 AM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: glik...@secretlab.ca wrote on 2010/08/10 16:56:42: On Tue

Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-10 Thread Grant Likely
On Mon, Aug 2, 2010 at 1:52 AM, Mingkai Hu mingkai...@freescale.com wrote: Signed-off-by: Mingkai Hu mingkai...@freescale.com --- v2:  - Move the flash partition function from of_spi.c to MTD driver  drivers/mtd/devices/m25p80.c |   29 +  1 files changed, 29

Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-10 Thread Grant Likely
On Tue, Aug 10, 2010 at 1:14 AM, David Brownell davi...@pacbell.net wrote: --- On Mon, 8/9/10, Grant Likely grant.lik...@secretlab.ca wrote: +       nr_parts = of_mtd_parse_partitions(spi-dev, np, parts); Let's keep OF-specific logic out of drivers like this one ...  intended to work

Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-10 Thread David Brownell
--- On Mon, 8/9/10, Grant Likely grant.lik...@secretlab.ca wrote: +       nr_parts = of_mtd_parse_partitions(spi-dev, np, parts); Let's keep OF-specific logic out of drivers like this one ... intended to work without OF. NAK on adding dependencies like OF to drivers and other

Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-10 Thread Joakim Tjernlund
On Tue, Aug 10, 2010 at 1:14 AM, David Brownell davi...@pacbell.net wrote: --- On Mon, 8/9/10, Grant Likely grant.lik...@secretlab.ca wrote: +       nr_parts = of_mtd_parse_partitions(spi-dev, np, parts); Let's keep OF-specific logic out of drivers like this one ...  intended

Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-10 Thread Grant Likely
On Tue, Aug 10, 2010 at 2:29 AM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: On Tue, Aug 10, 2010 at 1:14 AM, David Brownell davi...@pacbell.net wrote: --- On Mon, 8/9/10, Grant Likely grant.lik...@secretlab.ca wrote: +       nr_parts = of_mtd_parse_partitions(spi-dev,

Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-10 Thread Anton Vorontsov
On Tue, Aug 10, 2010 at 08:56:42AM -0600, Grant Likely wrote: [...] The suggestion has been raised to use something like bus notifiers to get a hook onto the device registration before the driver is probed so that the platform_data can be translated in an separate chuck of code, but down that

Re: [spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-10 Thread Grant Likely
On Tue, Aug 10, 2010 at 11:23 AM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: glik...@secretlab.ca wrote on 2010/08/10 16:56:42: On Tue, Aug 10, 2010 at 2:29 AM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: On Tue, Aug 10, 2010 at 1:14 AM, David Brownell

[spi-devel-general] [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions

2010-08-02 Thread Mingkai Hu
Signed-off-by: Mingkai Hu mingkai...@freescale.com --- v2: - Move the flash partition function from of_spi.c to MTD driver drivers/mtd/devices/m25p80.c | 29 + 1 files changed, 29 insertions(+), 0 deletions(-) diff --git a/drivers/mtd/devices/m25p80.c