Hi, On Mon, May 22, 2017 at 11:50:04AM -0500, Rob Herring wrote: > [...] > >> >> Thanks, I'll take it then. Not sure why Sebastian was suggested to > >> >> submit this patch via your tree in the first place. > >> >> > >> >> https://patchwork.kernel.org/patch/9713645/ > >> > > >> > Thanks. The idea was to get into early 4.12-rc to avoid merge > >> > conflicts in the droid 4 *.dts during 4.13 cycle. This strategy > >> > obviously failed :) > >> > >> First, I'm not sure why you combined everything. A maintainer can just > >> as easily take a series as a single patch and we prefer binding doc, > >> dts and driver changes all separate. > >> > >> Second, the dts changes could go thru arm-soc and the driver change > >> thru netdev. The binding doc can be thru either. There's no bisecting > >> dependency and things shouldn't break. It just won't all work until > >> you have both branches. > > > > This is only true for new devices. WLAN for droid4 works at the > > moment using incorrect compatible string. If *.dts is updated and > > driver is not yet updated WLAN does not work. IMHO that is a > > bisecting dependency. > > True. That's also breaking compatibility if a new kernel doesn't work > with an old DT. This way around works. It's the other way around, that does not work (new DT with old kernel). > Is it just a compatible string change? If so, then just keep the > old string as a fallback. That should work. -- Sebastian
Attachment:
signature.asc
Description: PGP signature
