On Thu, Apr 30, 2015 at 10:43:39AM -0300, Paulo Zanoni wrote: > 2015-04-15 8:06 GMT-03:00 Ander Conselvan De Oliveira <conselvan2@xxxxxxxxx>: > > Reviewed-by: Ander Conselvan de Oliveira <conselvan2@xxxxxxxxx> > > > > On Fri, 2015-04-10 at 16:22 +0200, Daniel Vetter wrote: > >> Since universal planes the primary plane might not be around, and it's > >> kinda silly to restrict the pipe bpp to the primary plane if we might > >> end up displaying a 10bpc video overlay. And with atomic we might very > >> well enable a pipe without a primary plane. So just use the platform > >> max as a starting point and then restrict appropriately. > >> > >> Of course this is all still a bit moot as long as we artificially > >> compress everything to max 8bpc because we don't use the hi-bpc gamma > >> tables. > >> > >> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > > This commit makes the HDMI colors look very funny on my BDW machine. I > just boot it with HDMI (no eDP), and I get this: > http://people.freedesktop.org/~pzanoni/color-regression.jpg (the > original wallpaper only has gradients of blue). Although I like the > new wallpaper color scheme, I don't think most users will. > > Reverting this commit (and also "drm/i915: Drop unecessary fb > arguments from function signatures" in order to make things compile > again) fixes the problem for me. > > Since this is a major issue for me, I'd be happy to test patches. We dump pipe_bpp and dither settings into dmesg. Do you see any differences in there between working and broken kernel? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
