From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fllnx210.ext.ti.com ([198.47.19.17]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1eD4Qs-0001pG-Kl for linux-mtd@lists.infradead.org; Fri, 10 Nov 2017 08:13:00 +0000 Subject: Re: [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support To: Tony Lindgren , Ladislav Michl CC: , , Boris Brezillon , Kyungmin Park References: <20171109091155.6a6azfvjarwvlfh2@lenoch> <20171109091253.lvrzi5kbaykkxlcb@lenoch> <20171109175626.GD28152@atomide.com> <20171109181004.g55vx4iveo5sulpt@lenoch> <20171109182645.GE28152@atomide.com> <20171109183408.xbc4bacawv7li54l@lenoch> <20171109184828.GF28152@atomide.com> From: Roger Quadros Message-ID: Date: Fri, 10 Nov 2017 10:12:22 +0200 MIME-Version: 1.0 In-Reply-To: <20171109184828.GF28152@atomide.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-GB Content-Transfer-Encoding: 7bit List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 09/11/17 20:48, Tony Lindgren wrote: > * Ladislav Michl [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