On Sun, May 13, 2012 at 09:15:51PM +0100, Chris Wilson wrote: > On Sun, 13 May 2012 22:08:10 +0200, Daniel Vetter <daniel at ffwll.ch> wrote: > > On Sun, May 13, 2012 at 08:16:12PM +0100, Chris Wilson wrote: > > > Turn a fatal lockup into a merely blank display with lots of shouty > > > messages. > > > > > > v2: Whilst in the area, convert the other BUG_ON into less fatal errors. > > > In particular, note that we may be called on a PCH platform not using > > > PLLs, such as Haswell, and so we do not always want to BUG_ON(!pll) > > > > > > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk> > > > > Ok, I guess you want me to drop v1 of this again. I'll wait for a bit of > > feedback on these two then (and the tested-by from qa on the 2nd one). > > Up to you, I thought that the original patch was overshadowed by the > your change request and little reason for it to be split. It would be > good to get QA's tested-by on the second patch at any rate. Ok, I've picked up v2 of patch 1 and the 2nd patch. Looks like we still have an issue somewhere in that code that makes ilk unhappy :( -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48