[PATCH 2/2] drm: parse color format support for digital displays

Jesse Barnes jbarnes at virtuousgeek.org
Fri Apr 15 13:39:36 PDT 2011


On Sat, 16 Apr 2011 06:10:07 +1000
Dave Airlie <airlied at gmail.com> wrote:

> > -
> > +#define DRM_COLOR_FORMAT_RGB444                (1<<0)
> > +#define DRM_COLOR_FORMAT_YCRCB444      (1<<1)
> > +#define DRM_COLOR_FORMAT_YCRCB422      (1<<2)
> >  /*
> >  * Describes a given display (e.g. CRT or flat panel) and its limitations.
> >  */
> > @@ -201,6 +203,7 @@ struct drm_display_info {
> >        unsigned int bpc;
> >
> >        enum subpixel_order subpixel_order;
> > +       unsigned long color_formats;
> 
> ^ wtf?
> 
> unsigned long? its 2011.

That doesn't tell me much about what you'd prefer...  I figured a
bitfield would be fairly extensible if new surface formats were added.
Maybe you're thinking it's not enough to support all the misc ones out
there though?

-- 
Jesse Barnes, Intel Open Source Technology Center


More information about the dri-devel mailing list