[PATCH 2/2] drm: add an fb creation ioctl that takes a pixel format

[Jesse Barnes]

On Fri, 11 Nov 2011 10:50:00 +0000
Dave Airlie <airlied at gmail.com> wrote:

> On Fri, Nov 11, 2011 at 10:47 AM, Dave Airlie <airlied at gmail.com> wrote:
> > On Tue, Nov 8, 2011 at 9:18 PM, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> >> To properly support the various plane formats supported by different
> >> hardware, the kernel must know the pixel format of a framebuffer object.
> >> So add a new ioctl taking a format argument corresponding to a fourcc
> >> name from videodev2.h.  Implement the fb creation hooks in terms of the
> >> new mode_fb_cmd2 using helpers where the old bpp/depth values are
> >> needed.
> >
> > I'm nearly sure I said this before, but please move the FOURCC defines
> > somewhere generic,
> > then use those, otherwise this is going to encourage crappy userspace
> > coding where
> > they drag in libv4l2 just to use a drm feature.
> >
> > I know they are generic, but userspace writes will start to include
> > v4l headers to get at
> > the same defines you use in the kernel.
> >
> 
> Oh and I'll take it as a follow-up patch. as I'd rather merge the base
> code now before it gets too late.

So just a separate header would be enough for you?
include/linux/fourcc or something?  Do you care if they continue to use
the v4l prefixes or not?  My preference would be to preserve that;
anything else is just churn for churn's sake.

Thanks,
-- 
Jesse Barnes, Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20111111/64e4424a/attachment-0001.pgp>