On Thu, Nov 09, 2017 at 10:48:28AM -0800, Tony Lindgren wrote: > * Ladislav Michl <ladis@xxxxxxxxxxxxxx> [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/ > > Well don't we still need the related dts changes posted > and merged first? Well, except Roger's feedback, there were complete silence so far. Note that simple fix was posted in February, this was rejected as we should aim towards clean DT only driver. And here we are with some doubts. As I have no hardware nor any special knowledge going prior initial driver commit it is rather hard to send "related dts changes" given simple fact, that I'm unsure whenever we need to distinguish between OMAP2 and OMAP3. Based on that we need either one or two compatible strings. Having two is safe, but u-boot will be unable to bring onenand equipped igep up. ladis > Regards, > > Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
