On Mon, 25 Apr 2011 16:52:58 -0700, Jesse Barnes jbarnes@virtuousgeek.org wrote:
struct drm_mode_surface { enum components; /* YUV, VUY, RGB, BGR, ARGB, ... */ int depth; enum packing; /* some list of packing types? */ ... };
Might just make a uint32_t 'type', predefine some obvious types and allow drivers to provide more, perhaps with a magic 'vendor' bit so it's easy to add more standard types. And, these are scanout buffers, I think it would be nice to use that in the name rather than the all-too-overused 'surface'.