On Mon, 2017-01-02 at 14:42 -0500, Adam Jackson wrote: > On Thu, 2016-12-22 at 15:41 +0100, Stefan Agner wrote: > > When setting DefaultDepth to 16 in the Screen section, the current > > code requests a 32 bpp framebuffer, however the X-Server seems to > > assumes 16 bpp. > > > > Fixes commit 21217d02168d ("modesetting: Implement 32->24 bpp > > conversion in shadow update") > > > > Signed-off-by: Stefan Agner <ste...@agner.ch> > > --- > > Observed this issue using X-Server/modesetting 1.18.4 and the FSL > > DCU DRM driver, but I am pretty sure it is driver independent. Not > > sure if the change is properly taking care of the corner case which > > was initially targetted, but it makes specifiying DefaultDepth with > > values of 16/24/32 and not specifiying it work for me. > > Without this patch, what path do you end up taking through > drmmode_get_default_bpp() ? I'm a little suspicious of the error > handling in that function.
Enh, not worth caring that much. This is correct, merged: remote: remote: I: patch #129230 updated using rev 1c7f34e99ff9750979a03ae20c6be1f2b42c284c. remote: remote: I: 1 patch(es) updated to state Accepted. remote: To ssh://kemper.freedesktop.org/git/xorg/xserver remote: d95a1310e..1c7f34e99 1c7f34e99ff9750979a03ae20c6be1f2b42c284c -> master To ssh://gitlab.freedesktop.org/xorg/xserver d95a1310ef..1c7f34e99f master -> master - ajax _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel