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). -- Jesse Barnes, Intel Open Source Technology Center