From: Roger Quadros <rogerq@ti.com> To: Tony Lindgren <tony@atomide.com>, Ladislav Michl <ladis@linux-mips.org> Cc: Boris Brezillon <boris.brezillon@free-electrons.com>, Kyungmin Park <kyungmin.park@samsung.com>, linux-omap@vger.kernel.org, linux-mtd@lists.infradead.org Subject: Re: [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support Date: Fri, 10 Nov 2017 10:12:22 +0200 [thread overview] Message-ID: <b59f9975-bc59-97c4-ddb2-57637de12e2c@ti.com> (raw) In-Reply-To: <20171109184828.GF28152@atomide.com> On 09/11/17 20:48, Tony Lindgren wrote: > * Ladislav Michl <ladis@linux-mips.org> [171109 18:35]: >> On Thu, Nov 09, 2017 at 10:26:45AM -0800, Tony Lindgren wrote: >>> Well we should have the dependencies merged first to avoid breaking >> >> Yes, that's what cover letter says :-) >> Also, I still count on your suggestion to merge it via mtd tree. > > Well only after it has been tested :) At this point I'd say > we're best off waiting for v4.16 merge window on the clean > up too as the merge window is about to open. > >>> git bisect. After applying this and the first patch I see: >>> >>> omap-gpmc 6e000000.gpmc: /ocp@68000000/gpmc@6e000000/onenand@0,0 has no 'bank-width' property >>> omap-gpmc 6e000000.gpmc: failed to probe DT child 'onenand': -22 >>> >>> So seems like more dts changes are needed to test this. >> >> Argh... You are right, I should add this into serie: >> https://patchwork.kernel.org/patch/10043259/ You don't need to resend that patch as it has been already reviewed and in my v4.16 queue. Just indicate in the cover letter that this series depends on that patch. > > Well don't we still need the related dts changes posted > and merged first> > Regards, > > Tony > -- cheers, -roger Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/
WARNING: multiple messages have this Message-ID (diff)
From: Roger Quadros <rogerq@ti.com> To: Tony Lindgren <tony@atomide.com>, Ladislav Michl <ladis@linux-mips.org> Cc: <linux-mtd@lists.infradead.org>, <linux-omap@vger.kernel.org>, Boris Brezillon <boris.brezillon@free-electrons.com>, Kyungmin Park <kyungmin.park@samsung.com> Subject: Re: [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support Date: Fri, 10 Nov 2017 10:12:22 +0200 [thread overview] Message-ID: <b59f9975-bc59-97c4-ddb2-57637de12e2c@ti.com> (raw) In-Reply-To: <20171109184828.GF28152@atomide.com> On 09/11/17 20:48, Tony Lindgren wrote: > * Ladislav Michl <ladis@linux-mips.org> [171109 18:35]: >> On Thu, Nov 09, 2017 at 10:26:45AM -0800, Tony Lindgren wrote: >>> Well we should have the dependencies merged first to avoid breaking >> >> Yes, that's what cover letter says :-) >> Also, I still count on your suggestion to merge it via mtd tree. > > Well only after it has been tested :) At this point I'd say > we're best off waiting for v4.16 merge window on the clean > up too as the merge window is about to open. > >>> git bisect. After applying this and the first patch I see: >>> >>> omap-gpmc 6e000000.gpmc: /ocp@68000000/gpmc@6e000000/onenand@0,0 has no 'bank-width' property >>> omap-gpmc 6e000000.gpmc: failed to probe DT child 'onenand': -22 >>> >>> So seems like more dts changes are needed to test this. >> >> Argh... You are right, I should add this into serie: >> https://patchwork.kernel.org/patch/10043259/ You don't need to resend that patch as it has been already reviewed and in my v4.16 queue. Just indicate in the cover letter that this series depends on that patch. > > Well don't we still need the related dts changes posted > and merged first> > Regards, > > Tony > -- cheers, -roger Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
next prev parent reply other threads:[~2017-11-10 8:12 UTC|newest] Thread overview: 92+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-11-09 9:11 [PATCH v3 0/7] OMAP2+ OneNAND driver update Ladislav Michl 2017-11-09 9:11 ` Ladislav Michl 2017-11-09 9:12 ` [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support Ladislav Michl 2017-11-09 9:12 ` Ladislav Michl 2017-11-09 17:56 ` Tony Lindgren 2017-11-09 17:56 ` Tony Lindgren 2017-11-09 18:10 ` Ladislav Michl 2017-11-09 18:10 ` Ladislav Michl 2017-11-09 18:26 ` Tony Lindgren 2017-11-09 18:26 ` Tony Lindgren 2017-11-09 18:34 ` Ladislav Michl 2017-11-09 18:34 ` Ladislav Michl 2017-11-09 18:48 ` Tony Lindgren 2017-11-09 18:48 ` Tony Lindgren 2017-11-09 19:10 ` Ladislav Michl 2017-11-09 19:10 ` Ladislav Michl 2017-11-09 21:59 ` Tony Lindgren 2017-11-09 21:59 ` Tony Lindgren 2017-11-09 22:26 ` Ladislav Michl 2017-11-09 22:26 ` Ladislav Michl 2017-11-10 8:12 ` Roger Quadros [this message] 2017-11-10 8:12 ` Roger Quadros 2017-11-09 9:13 ` [PATCH v3 2/7] mtd: onenand: omap2: Remove regulator support Ladislav Michl 2017-11-09 9:13 ` Ladislav Michl 2017-11-10 8:17 ` Roger Quadros 2017-11-10 8:17 ` Roger Quadros 2017-11-09 9:14 ` [PATCH v3 3/7] mtd: onenand: omap2: Remove skip initial unlocking support Ladislav Michl 2017-11-09 9:14 ` Ladislav Michl 2017-11-10 8:18 ` Roger Quadros 2017-11-10 8:18 ` Roger Quadros 2017-11-09 9:14 ` [PATCH v3 4/7] mtd: onenand: omap2: Remove partitioning support from platform data Ladislav Michl 2017-11-09 9:14 ` Ladislav Michl 2017-11-10 8:19 ` Roger Quadros 2017-11-10 8:19 ` Roger Quadros 2017-11-10 9:48 ` Ladislav Michl 2017-11-10 9:48 ` Ladislav Michl 2017-11-09 9:15 ` [PATCH v3 5/7] mtd: onenand: omap2: Unify OMAP2 and OMAP3 DMA implementation Ladislav Michl 2017-11-09 9:15 ` Ladislav Michl 2017-11-10 8:21 ` Roger Quadros 2017-11-10 8:21 ` Roger Quadros 2017-11-10 9:51 ` Ladislav Michl 2017-11-10 9:51 ` Ladislav Michl 2017-11-10 15:26 ` Tony Lindgren 2017-11-10 15:26 ` Tony Lindgren 2017-11-10 18:19 ` Ladislav Michl 2017-11-10 18:19 ` Ladislav Michl 2017-11-10 18:29 ` Tony Lindgren 2017-11-10 18:29 ` Tony Lindgren 2017-11-11 12:50 ` Ladislav Michl 2017-11-11 12:50 ` Ladislav Michl 2017-11-13 20:10 ` Peter Ujfalusi 2017-11-13 20:10 ` Peter Ujfalusi 2017-11-14 14:47 ` Roger Quadros 2017-11-14 14:47 ` Roger Quadros 2017-11-14 15:03 ` Ladislav Michl 2017-11-14 15:03 ` Ladislav Michl 2017-11-10 8:25 ` Peter Ujfalusi 2017-11-10 8:25 ` Peter Ujfalusi 2017-11-10 10:04 ` Ladislav Michl 2017-11-10 10:04 ` Ladislav Michl 2017-11-10 15:24 ` Tony Lindgren 2017-11-10 15:24 ` Tony Lindgren 2017-11-10 18:26 ` Ladislav Michl 2017-11-10 18:26 ` Ladislav Michl 2017-11-10 18:48 ` Tony Lindgren 2017-11-10 18:48 ` Tony Lindgren 2017-11-10 21:39 ` Ladislav Michl 2017-11-10 21:39 ` Ladislav Michl 2017-11-14 21:53 ` Tony Lindgren 2017-11-14 21:53 ` Tony Lindgren 2017-11-14 22:32 ` Ladislav Michl 2017-11-14 22:32 ` Ladislav Michl 2017-11-15 2:11 ` Tony Lindgren 2017-11-15 2:11 ` Tony Lindgren 2017-11-13 8:22 ` Peter Ujfalusi 2017-11-13 8:22 ` Peter Ujfalusi 2017-11-13 12:15 ` Ladislav Michl 2017-11-13 12:15 ` Ladislav Michl 2017-11-13 14:36 ` Peter Ujfalusi 2017-11-13 14:36 ` Peter Ujfalusi 2017-11-13 15:23 ` Tony Lindgren 2017-11-13 15:23 ` Tony Lindgren 2017-11-13 15:27 ` Ladislav Michl 2017-11-13 15:27 ` Ladislav Michl 2017-11-14 15:05 ` Roger Quadros 2017-11-14 15:05 ` Roger Quadros 2017-11-14 15:22 ` Ladislav Michl 2017-11-14 15:22 ` Ladislav Michl 2017-11-09 9:16 ` [PATCH v3 6/7] mtd: onenand: omap2: Do not make delay for GPIO OMAP3 specific Ladislav Michl 2017-11-09 9:16 ` Ladislav Michl 2017-11-09 9:17 ` [PATCH v3 7/7] mtd: onenand: omap2: Configure driver from DT Ladislav Michl 2017-11-09 9:17 ` Ladislav Michl
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=b59f9975-bc59-97c4-ddb2-57637de12e2c@ti.com \ --to=rogerq@ti.com \ --cc=boris.brezillon@free-electrons.com \ --cc=kyungmin.park@samsung.com \ --cc=ladis@linux-mips.org \ --cc=linux-mtd@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=tony@atomide.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.