On Sat, 13 Aug 2011 23:22:14 -0400, Hai Lan <hai....@intel.com> wrote:

>       case 32:
>               if (fb->depth == 24)
>                       dspcntr |= DISPPLANE_32BPP_NO_ALPHA;
> +             else if (fb->depth == 32)
> +                     dspcntr |= DISPPLANE_32BPP_NO_ALPHA;
>               else if (fb->depth == 30)
>                       dspcntr |= DISPPLANE_32BPP_30BIT_NO_ALPHA;
>               else

There is no hardware support for depth 32 output -- the hardware can
only do 24-bits (8-8-8-8) XRGB or 30-bits (2-10-10-10) XRGB. Passing
depth 32 to this function is in fact an error which the kernel now
correctly reports.

This patch simply assumes that the application programmer created a
depth 24 frame buffer and then mistakenly told the kernel that it was
depth 32.

-- 
keith.pack...@intel.com

Attachment: pgpSxwHSVAhTM.pgp
Description: PGP signature

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to