On Sun, Mar 17, 2013 at 09:10:42PM +0100, Bjørn Mork wrote: > Russell King - ARM Linux <li...@arm.linux.org.uk> writes: > > > Sigh. > > > > At times, I really don't know why I bother wasting my time commenting on > > patches if they just go and agree with them, and promptly ignore them. > > I pointed out what this should be: > > > > http://marc.info/?l=linux-kernel&m=135972750522902&w=2 > > > > And it was agreed: > > > > http://marc.info/?l=linux-kernel&m=135993546317541&w=2 > > > > But for some reason the wrong patch got merged: > > > > http://marc.info/?l=git-commits-head&m=136147889001219&w=2 > > > > Maybe because it was "simpler" ? Don't know. But... well, we now know > > what the outcome of it was. Let's try a different technique. I'm going > > to say nothing of any real help and hopefully it will get fixed properly. > > Maybe that'll work. > > To be fair, looking at the commitdate of that commit it was already in > Greg's tree when you commented on it. So the discussion you had would > have to be about a followup-fix to this, and not about that particular > patch.
Yes, I thought someone was going to send me a follow-on patch to fix this, that is what Minchan said, after I had committed it. Anyway, it's a staging driver, if it doesn't build, that's not a big deal. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
