[PATCH] mdt nand: omap2+ use platform options

2011-11-29 Thread Jan Weitzel
Options from struct omap_nand_platform_data are not used. Apply options after nand_scan_ident to avoid overwrite due to NAND_CHIPOPTIONS_MSK. So you can pass options from platformcode Signed-off-by: Jan Weitzel --- drivers/mtd/nand/omap2.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions

Re: [PATCH] mdt nand: omap2+ use platform options

2011-11-30 Thread Brian Norris
On Tue, Nov 29, 2011 at 1:00 AM, Jan Weitzel wrote: > Options from struct omap_nand_platform_data are not used. > Apply options after nand_scan_ident to avoid overwrite due to > NAND_CHIPOPTIONS_MSK. > So you can pass options from platformcode Just to clarify, were the pdata->options ever used be

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-01 Thread Jan Weitzel
Am Mittwoch, den 30.11.2011, 12:31 -0800 schrieb Brian Norris: > On Tue, Nov 29, 2011 at 1:00 AM, Jan Weitzel wrote: > > Options from struct omap_nand_platform_data are not used. > > Apply options after nand_scan_ident to avoid overwrite due to > > NAND_CHIPOPTIONS_MSK. > > So you can pass options

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-01 Thread Artem Bityutskiy
On Tue, 2011-11-29 at 10:00 +0100, Jan Weitzel wrote: > Options from struct omap_nand_platform_data are not used. > Apply options after nand_scan_ident to avoid overwrite due to > NAND_CHIPOPTIONS_MSK. > So you can pass options from platformcode > > Signed-off-by: Jan Weitzel Pushed to l2-mtd-2.

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-02 Thread Grazvydas Ignotas
On Thu, Dec 1, 2011 at 10:42 AM, Artem Bityutskiy wrote: > On Tue, 2011-11-29 at 10:00 +0100, Jan Weitzel wrote: >> Signed-off-by: Jan Weitzel > > Pushed to l2-mtd-2.6.git, thank you! This breaks build here, did you really test it, Jan? drivers/mtd/nand/omap2.c: In function 'omap_nand_probe': d

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-02 Thread Artem Bityutskiy
On Fri, 2011-12-02 at 12:20 +0200, Grazvydas Ignotas wrote: > On Thu, Dec 1, 2011 at 10:42 AM, Artem Bityutskiy wrote: > > On Tue, 2011-11-29 at 10:00 +0100, Jan Weitzel wrote: > >> Signed-off-by: Jan Weitzel > > > > Pushed to l2-mtd-2.6.git, thank you! > > This breaks build here, did you really

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-02 Thread Brian Norris
On Fri, Dec 2, 2011 at 2:20 AM, Grazvydas Ignotas wrote: > On Thu, Dec 1, 2011 at 10:42 AM, Artem Bityutskiy wrote: >> On Tue, 2011-11-29 at 10:00 +0100, Jan Weitzel wrote: >>> Signed-off-by: Jan Weitzel >> >> Pushed to l2-mtd-2.6.git, thank you! > > This breaks build here, did you really test i

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-04 Thread Artem Bityutskiy
On Fri, 2011-12-02 at 09:28 -0800, Brian Norris wrote: > On Fri, Dec 2, 2011 at 2:20 AM, Grazvydas Ignotas wrote: > > On Thu, Dec 1, 2011 at 10:42 AM, Artem Bityutskiy > > wrote: > >> On Tue, 2011-11-29 at 10:00 +0100, Jan Weitzel wrote: > >>> Signed-off-by: Jan Weitzel > >> > >> Pushed to l2-m

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-05 Thread Jan Weitzel
Am Freitag, den 02.12.2011, 12:20 +0200 schrieb Grazvydas Ignotas: > On Thu, Dec 1, 2011 at 10:42 AM, Artem Bityutskiy wrote: > > On Tue, 2011-11-29 at 10:00 +0100, Jan Weitzel wrote: > >> Signed-off-by: Jan Weitzel > > > > Pushed to l2-mtd-2.6.git, thank you! > > This breaks build here, did you

Re: [PATCH] mdt nand: omap2+ use platform options

2011-12-05 Thread Grazvydas Ignotas
On Mon, Dec 5, 2011 at 10:11 AM, Jan Weitzel wrote: > Sorry, I wasn't carefully enough. I missed 7a559c78 where you removed > options from struct omap_nand_platform_data. Did you think getting we > should reintroduce it? I am looking for a good way to get > NAND_NO_SUBPAGE_WRITE from boardfile to