Hi, On Fri, Nov 06, 2015 at 06:53:52AM -0800, Tony Lindgren wrote: > * Felipe Balbi <balbi@xxxxxx> [151106 06:41]: > > Belisko Marek <marek.belisko@xxxxxxxxx> writes: > > > On Fri, Nov 6, 2015 at 3:24 PM, Felipe Balbi <balbi@xxxxxx> wrote: > > >> Felipe Balbi <balbi@xxxxxx> writes: > > >>> your commit af19161aaed7 ("ARM: dts: twl4030: Add iio properties for bci > > >>> subnode") breaks build on current linus/master (which current sits in > > >> > > >> this commit cannot be found in next. How come it's in linus/master ? > > > I did post fix but Tony seems doesn't merge it: > > > https://lkml.org/lkml/2015/10/13/816 > > > > looking at that thread, I have no idea how come the old version was > > merged in the first place. Tony was clear that it broke build and yet > > this patch has made its way to mainline and it didn't even go through > > linux-next, which makes the problem worse. > > > > Now we have a bisection point where the tree (well, some DTS files) > > won't even build. This is quite messy. > > Yes didn't I drop the patch and pointed out it breaks the build? > > Guys, please stop doing this. Do not merge driver code that has > not been sitting in linux next at least a week. And for the dts > changes, please make sure you have proper acks. > > And now we have yet another merge window where things unexpectedly > break during the merge window because of untested driver changes. > > Please repost the fix and I'll ack it and you guys send a new pull > request to fix it ASAP explaining how it happened. mh for some reason I accidently added it to my next branch. I'm not sure why, since I did not intend to queue it at all. Sorry for the mess. I wonder why it has not been in next, though. Checking my next tree, it should have been there since end of September. -- Sebastian
Attachment:
signature.asc
Description: PGP signature