On Wed, May 02, 2012 at 05:22:19PM -0700, Jesse Barnes wrote: > On Wed, 2 May 2012 13:18:56 -0700 > Ben Widawsky <ben at bwidawsk.net> wrote: > > > On Wed, 2 May 2012 11:55:48 -0700 > > Jesse Barnes <jbarnes at virtuousgeek.org> wrote: > > > > > On Wed, 2 May 2012 08:44:27 -0700 > > > Ben Widawsky <ben at bwidawsk.net> wrote: > > > > > > > On Wed, 2 May 2012 11:06:23 +0200 > > > > Daniel Vetter <daniel.vetter at ffwll.ch> wrote: > > > > > > > > > Unfortunately it looks like further vlv patches are still stalled due > > > > > to fried hw, and too many people are a bit annoyed about the unused > > > > > function warning. > > > > > > > > It is my preference that you never merge code which you know will > > > > produce a warning. I still stand by adding a __used for intermediate > > > > patches. > > > > > > Well he could have just merged the other patches too, but nooo... > > > > > > > Twasn't a shot at you... shot was fired at Daniel (although if you *did* > > but in __used I wouldn't have had to shoot at Daniel). > > Meh ok you can kill this function. Vijay will be re-posting the code > that uses it (and adds it again) once he gets his hardware, unless I > beat him to it (still waiting on a new CPU). I guess that counts as an ack ... patch merged for -next. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48
