On Sat, 16 Apr 2011 06:42:44 +1000
Dave Airlie <airl...@gmail.com> wrote:
> On Sat, Apr 16, 2011 at 6:39 AM, Jesse Barnes <jbar...@virtuousgeek.org>
> wrote:
> > On Sat, 16 Apr 2011 06:10:07 +1000
> > Dave Airlie <airl...@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?
>
> Its unsigned long, its a different size on 32 and 64-bit, not
> something I want to fall
> over when you add the 33rd bit field.
I hope we don't get to more than 32, but sure I'll change it to u32 to
match some of the other flags.
--
Jesse Barnes, Intel Open Source Technology Center
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
