Hi Laurent, On 08/31/2011 11:18 AM, Laurent Pinchart wrote: > This API will be used to support YUV frame buffer formats in a standard > way. looks like the union is causing problems. With this patch applied I get errors like this: CC [M] drivers/auxdisplay/cfag12864bfb.o drivers/auxdisplay/cfag12864bfb.c:57: error: unknown field ‘red’ specified in initializer drivers/auxdisplay/cfag12864bfb.c:57: warning: missing braces around initializer drivers/auxdisplay/cfag12864bfb.c:57: warning: (near initialization for ‘cfag12864bfb_var.<anonymous>.<anonymous>’) drivers/auxdisplay/cfag12864bfb.c:58: error: unknown field ‘green’ specified in initializer drivers/auxdisplay/cfag12864bfb.c:58: warning: braces around scalar initializer drivers/auxdisplay/cfag12864bfb.c:58: warning: (near initialization for ‘cfag12864bfb_var.nonstd’) drivers/auxdisplay/cfag12864bfb.c:58: warning: excess elements in scalar initializer drivers/auxdisplay/cfag12864bfb.c:58: warning: (near initialization for ‘cfag12864bfb_var.nonstd’) drivers/auxdisplay/cfag12864bfb.c:58: warning: excess elements in scalar initializer drivers/auxdisplay/cfag12864bfb.c:58: warning: (near initialization for ‘cfag12864bfb_var.nonstd’) drivers/auxdisplay/cfag12864bfb.c:59: error: unknown field ‘blue’ specified in initializer drivers/auxdisplay/cfag12864bfb.c:59: warning: braces around scalar initializer drivers/auxdisplay/cfag12864bfb.c:59: warning: (near initialization for ‘cfag12864bfb_var.activate’) drivers/auxdisplay/cfag12864bfb.c:59: warning: excess elements in scalar initializer drivers/auxdisplay/cfag12864bfb.c:59: warning: (near initialization for ‘cfag12864bfb_var.activate’) drivers/auxdisplay/cfag12864bfb.c:59: warning: excess elements in scalar initializer drivers/auxdisplay/cfag12864bfb.c:59: warning: (near initialization for ‘cfag12864bfb_var.activate’) make[2]: *** [drivers/auxdisplay/cfag12864bfb.o] Error 1 Any idea? Best regards, Florian Tobias Schandinat > > Last but not least, create a much needed fbdev API documentation and > document the format setting APIs. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > Documentation/fb/api.txt | 317 ++++++++++++++++++++++++++++++++++++++++++++++ > include/linux/fb.h | 28 +++- > 2 files changed, 339 insertions(+), 6 deletions(-) > create mode 100644 Documentation/fb/api.txt > > diff --git a/Documentation/fb/api.txt b/Documentation/fb/api.txt > new file mode 100644 > index 0000000..d842534 > --- /dev/null > +++ b/Documentation/fb/api.txt > @@ -0,0 +1,317 @@ > + The Frame Buffer Device API > + --------------------------- > + > +Last revised: June 21, 2011 > + > + > +0. Introduction > +--------------- > + > +This document describes the frame buffer API used by applications to interact > +with frame buffer devices. In-kernel APIs between device drivers and the frame > +buffer core are not described. > + > +Due to a lack of documentation in the original frame buffer API, drivers > +behaviours differ in subtle (and not so subtle) ways. This document describes > +the recommended API implementation, but applications should be prepared to > +deal with different behaviours. > + > + > +1. Capabilities > +--------------- > + > +Device and driver capabilities are reported in the fixed screen information > +capabilities field. > + > +struct fb_fix_screeninfo { > + ... > + __u16 capabilities; /* see FB_CAP_* */ > + ... > +}; > + > +Application should use those capabilities to find out what features they can > +expect from the device and driver. > + > +- FB_CAP_FOURCC > + > +The driver supports the four character code (FOURCC) based format setting API. > +When supported, formats are configured using a FOURCC instead of manually > +specifying color components layout. > + > + > +2. Types and visuals > +-------------------- > + > +Pixels are stored in memory in hardware-dependent formats. Applications need > +to be aware of the pixel storage format in order to write image data to the > +frame buffer memory in the format expected by the hardware. > + > +Formats are described by frame buffer types and visuals. Some visuals require > +additional information, which are stored in the variable screen information > +bits_per_pixel, grayscale, fourcc, red, green, blue and transp fields. > + > +Visuals describe how color information is encoded and assembled to create > +macropixels. Types describe how macropixels are stored in memory. The following > +types and visuals are supported. > + > +- FB_TYPE_PACKED_PIXELS > + > +Macropixels are stored contiguously in a single plane. If the number of bits > +per macropixel is not a multiple of 8, whether macropixels are padded to the > +next multiple of 8 bits or packed together into bytes depends on the visual. > + > +Padding at end of lines may be present and is then reported through the fixed > +screen information line_length field. > + > +- FB_TYPE_PLANES > + > +Macropixels are split across multiple planes. The number of planes is equal to > +the number of bits per macropixel, with plane i'th storing i'th bit from all > +macropixels. > + > +Planes are located contiguously in memory. > + > +- FB_TYPE_INTERLEAVED_PLANES > + > +Macropixels are split across multiple planes. The number of planes is equal to > +the number of bits per macropixel, with plane i'th storing i'th bit from all > +macropixels. > + > +Planes are interleaved in memory. The interleave factor, defined as the > +distance in bytes between the beginning of two consecutive interleaved blocks > +belonging to different planes, is stored in the fixed screen information > +type_aux field. > + > +- FB_TYPE_FOURCC > + > +Macropixels are stored in memory as described by the format FOURCC identifier > +stored in the variable screen information fourcc field. > + > +- FB_VISUAL_MONO01 > + > +Pixels are black or white and stored on a number of bits (typically one) > +specified by the variable screen information bpp field. > + > +Black pixels are represented by all bits set to 1 and white pixels by all bits > +set to 0. When the number of bits per pixel is smaller than 8, several pixels > +are packed together in a byte. > + > +FB_VISUAL_MONO01 is currently used with FB_TYPE_PACKED_PIXELS only. > + > +- FB_VISUAL_MONO10 > + > +Pixels are black or white and stored on a number of bits (typically one) > +specified by the variable screen information bpp field. > + > +Black pixels are represented by all bits set to 0 and white pixels by all bits > +set to 1. When the number of bits per pixel is smaller than 8, several pixels > +are packed together in a byte. > + > +FB_VISUAL_MONO01 is currently used with FB_TYPE_PACKED_PIXELS only. > + > +- FB_VISUAL_TRUECOLOR > + > +Pixels are broken into red, green and blue components, and each component > +indexes a read-only lookup table for the corresponding value. Lookup tables > +are device-dependent, and provide linear or non-linear ramps. > + > +Each component is stored in a macropixel according to the variable screen > +information red, green, blue and transp fields. > + > +- FB_VISUAL_PSEUDOCOLOR and FB_VISUAL_STATIC_PSEUDOCOLOR > + > +Pixel values are encoded as indices into a colormap that stores red, green and > +blue components. The colormap is read-only for FB_VISUAL_STATIC_PSEUDOCOLOR > +and read-write for FB_VISUAL_PSEUDOCOLOR. > + > +Each pixel value is stored in the number of bits reported by the variable > +screen information bits_per_pixel field. > + > +- FB_VISUAL_DIRECTCOLOR > + > +Pixels are broken into red, green and blue components, and each component > +indexes a programmable lookup table for the corresponding value. > + > +Each component is stored in a macropixel according to the variable screen > +information red, green, blue and transp fields. > + > +- FB_VISUAL_FOURCC > + > +Pixels are encoded and interpreted as described by the format FOURCC > +identifier stored in the variable screen information fourcc field. > + > + > +3. Screen information > +--------------------- > + > +Screen information are queried by applications using the FBIOGET_FSCREENINFO > +and FBIOGET_VSCREENINFO ioctls. Those ioctls take a pointer to a > +fb_fix_screeninfo and fb_var_screeninfo structure respectively. > + > +struct fb_fix_screeninfo stores device independent unchangeable information > +about the frame buffer device and the current format. Those information can't > +be directly modified by applications, but can be changed by the driver when an > +application modifies the format. > + > +struct fb_fix_screeninfo { > + char id[16]; /* identification string eg "TT Builtin" */ > + unsigned long smem_start; /* Start of frame buffer mem */ > + /* (physical address) */ > + __u32 smem_len; /* Length of frame buffer mem */ > + __u32 type; /* see FB_TYPE_* */ > + __u32 type_aux; /* Interleave for interleaved Planes */ > + __u32 visual; /* see FB_VISUAL_* */ > + __u16 xpanstep; /* zero if no hardware panning */ > + __u16 ypanstep; /* zero if no hardware panning */ > + __u16 ywrapstep; /* zero if no hardware ywrap */ > + __u32 line_length; /* length of a line in bytes */ > + unsigned long mmio_start; /* Start of Memory Mapped I/O */ > + /* (physical address) */ > + __u32 mmio_len; /* Length of Memory Mapped I/O */ > + __u32 accel; /* Indicate to driver which */ > + /* specific chip/card we have */ > + __u16 capabilities; /* see FB_CAP_* */ > + __u16 reserved[2]; /* Reserved for future compatibility */ > +}; > + > +struct fb_var_screeninfo stores device independent changeable information > +about a frame buffer device, its current format and video mode, as well as > +other miscellaneous parameters. > + > +struct fb_var_screeninfo { > + __u32 xres; /* visible resolution */ > + __u32 yres; > + __u32 xres_virtual; /* virtual resolution */ > + __u32 yres_virtual; > + __u32 xoffset; /* offset from virtual to visible */ > + __u32 yoffset; /* resolution */ > + > + __u32 bits_per_pixel; /* guess what */ > + union { > + struct { /* Legacy format API */ > + __u32 grayscale; /* 0 = color, 1 = grayscale */ > + /* bitfields in fb mem if true color, else only */ > + /* length is significant */ > + struct fb_bitfield red; > + struct fb_bitfield green; > + struct fb_bitfield blue; > + struct fb_bitfield transp; /* transparency */ > + }; > + struct { /* FOURCC-based format API */ > + __u32 fourcc; /* FOURCC format */ > + __u32 colorspace; > + __u32 reserved[11]; > + } fourcc; > + }; > + > + __u32 nonstd; /* != 0 Non standard pixel format */ > + > + __u32 activate; /* see FB_ACTIVATE_* */ > + > + __u32 height; /* height of picture in mm */ > + __u32 width; /* width of picture in mm */ > + > + __u32 accel_flags; /* (OBSOLETE) see fb_info.flags */ > + > + /* Timing: All values in pixclocks, except pixclock (of course) */ > + __u32 pixclock; /* pixel clock in ps (pico seconds) */ > + __u32 left_margin; /* time from sync to picture */ > + __u32 right_margin; /* time from picture to sync */ > + __u32 upper_margin; /* time from sync to picture */ > + __u32 lower_margin; > + __u32 hsync_len; /* length of horizontal sync */ > + __u32 vsync_len; /* length of vertical sync */ > + __u32 sync; /* see FB_SYNC_* */ > + __u32 vmode; /* see FB_VMODE_* */ > + __u32 rotate; /* angle we rotate counter clockwise */ > + __u32 reserved[5]; /* Reserved for future compatibility */ > +}; > + > +To modify variable information, applications call the FBIOPUT_VSCREENINFO > +ioctl with a pointer to a fb_var_screeninfo structure. If the call is > +successful, the driver will update the fixed screen information accordingly. > + > +Instead of filling the complete fb_var_screeninfo structure manually, > +applications should call the FBIOGET_VSCREENINFO ioctl and modify only the > +fields they care about. > + > + > +4. Format configuration > +----------------------- > + > +Frame buffer devices offer two ways to configure the frame buffer format: the > +legacy API and the FOURCC-based API. > + > + > +The legacy API has been the only frame buffer format configuration API for a > +long time and is thus widely used by application. It is the recommended API > +for applications when using RGB and grayscale formats, as well as legacy > +non-standard formats. > + > +To select a format, applications set the fb_var_screeninfo bits_per_pixel field > +to the desired frame buffer depth. Values up to 8 will usually map to > +monochrome, grayscale or pseudocolor visuals, although this is not required. > + > +- For grayscale formats, applications set the grayscale field to one. The red, > + blue, green and transp fields must be set to 0 by applications and ignored by > + drivers. Drivers must fill the red, blue and green offsets to 0 and lengths > + to the bits_per_pixel value. > + > +- For pseudocolor formats, applications set the grayscale field to zero. The > + red, blue, green and transp fields must be set to 0 by applications and > + ignored by drivers. Drivers must fill the red, blue and green offsets to 0 > + and lengths to the bits_per_pixel value. > + > +- For truecolor and directcolor formats, applications set the grayscale field > + to zero, and the red, blue, green and transp fields to describe the layout of > + color components in memory. > + > +struct fb_bitfield { > + __u32 offset; /* beginning of bitfield */ > + __u32 length; /* length of bitfield */ > + __u32 msb_right; /* != 0 : Most significant bit is */ > + /* right */ > +}; > + > + Pixel values are bits_per_pixel wide and are split in non-overlapping red, > + green, blue and alpha (transparency) components. Location and size of each > + component in the pixel value are described by the fb_bitfield offset and > + length fields. Offset are computed from the right. > + > + Pixels are always stored in an integer number of bytes. If the number of > + bits per pixel is not a multiple of 8, pixel values are padded to the next > + multiple of 8 bits. > + > +Upon successful format configuration, drivers update the fb_fix_screeninfo > +type, visual and line_length fields depending on the selected format. > + > + > +The FOURCC-based API replaces format descriptions by four character codes > +(FOURCC). FOURCCs are abstract identifiers that uniquely define a format > +without explicitly describing it. This is the only API that supports YUV > +formats. Drivers are also encouraged to implement the FOURCC-based API for RGB > +and grayscale formats. > + > +Drivers that support the FOURCC-based API report this capability by setting > +the FB_CAP_FOURCC bit in the fb_fix_screeninfo capabilities field. > + > +FOURCC definitions are located in the linux/videodev2.h header. However, and > +despite starting with the V4L2_PIX_FMT_prefix, they are not restricted to V4L2 > +and don't require usage of the V4L2 subsystem. FOURCC documentation is > +available in Documentation/DocBook/v4l/pixfmt.xml. > + > +To select a format, applications set the fourcc.fourcc field to the desired > +FOURCC. For YUV formats, they should also select the appropriate colorspace by > +setting the fourcc.colorspace field to one of the colorspaces listed in > +linux/videodev2.h and documented in Documentation/DocBook/v4l/colorspaces.xml. > + > +For forward compatibility reasons applications must zero the fourcc reserved > +fields by zeroing the whole fourcc structure before filling it. The reserved > +fields must be ignored by drivers. Values other than 0 may get a meaning in > +future extensions. Note that the grayscale, red, green, blue and transp field > +share memory with the fourcc field. Application must thus not touch those > +fields when using the FOURCC-based API. > + > +Upon successful format configuration, drivers update the fb_fix_screeninfo > +type, visual and line_length fields depending on the selected format. The type > +and visual fields are set to FB_TYPE_FOURCC and FB_VISUAL_FOURCC respectively. > diff --git a/include/linux/fb.h b/include/linux/fb.h > index 1d6836c..98b23e3 100644 > --- a/include/linux/fb.h > +++ b/include/linux/fb.h > @@ -45,6 +45,7 @@ > #define FB_TYPE_INTERLEAVED_PLANES 2 /* Interleaved planes */ > #define FB_TYPE_TEXT 3 /* Text/attributes */ > #define FB_TYPE_VGA_PLANES 4 /* EGA/VGA planes */ > +#define FB_TYPE_FOURCC 5 /* Type identified by a V4L2 FOURCC */ > > #define FB_AUX_TEXT_MDA 0 /* Monochrome text */ > #define FB_AUX_TEXT_CGA 1 /* CGA/EGA/VGA Color text */ > @@ -69,6 +70,7 @@ > #define FB_VISUAL_PSEUDOCOLOR 3 /* Pseudo color (like atari) */ > #define FB_VISUAL_DIRECTCOLOR 4 /* Direct color */ > #define FB_VISUAL_STATIC_PSEUDOCOLOR 5 /* Pseudo color readonly */ > +#define FB_VISUAL_FOURCC 6 /* Visual identified by a V4L2 FOURCC */ > > #define FB_ACCEL_NONE 0 /* no hardware accelerator */ > #define FB_ACCEL_ATARIBLITT 1 /* Atari Blitter */ > @@ -154,6 +156,8 @@ > > #define FB_ACCEL_PUV3_UNIGFX 0xa0 /* PKUnity-v3 Unigfx */ > > +#define FB_CAP_FOURCC 1 /* Device supports FOURCC-based formats */ > + > struct fb_fix_screeninfo { > char id[16]; /* identification string eg "TT Builtin" */ > unsigned long smem_start; /* Start of frame buffer mem */ > @@ -171,7 +175,8 @@ struct fb_fix_screeninfo { > __u32 mmio_len; /* Length of Memory Mapped I/O */ > __u32 accel; /* Indicate to driver which */ > /* specific chip/card we have */ > - __u16 reserved[3]; /* Reserved for future compatibility */ > + __u16 capabilities; /* see FB_CAP_* */ > + __u16 reserved[2]; /* Reserved for future compatibility */ > }; > > /* Interpretation of offset for color fields: All offsets are from the right, > @@ -246,12 +251,23 @@ struct fb_var_screeninfo { > __u32 yoffset; /* resolution */ > > __u32 bits_per_pixel; /* guess what */ > - __u32 grayscale; /* != 0 Graylevels instead of colors */ > > - struct fb_bitfield red; /* bitfield in fb mem if true color, */ > - struct fb_bitfield green; /* else only length is significant */ > - struct fb_bitfield blue; > - struct fb_bitfield transp; /* transparency */ > + union { > + struct { /* Legacy format API */ > + __u32 grayscale; /* 0 = color, 1 = grayscale */ > + /* bitfields in fb mem if true color, else only */ > + /* length is significant */ > + struct fb_bitfield red; > + struct fb_bitfield green; > + struct fb_bitfield blue; > + struct fb_bitfield transp; /* transparency */ > + }; > + struct { /* FOURCC-based format API */ > + __u32 fourcc; /* FOURCC format */ > + __u32 colorspace; > + __u32 reserved[11]; > + } fourcc; > + }; > > __u32 nonstd; /* != 0 Non standard pixel format */ > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html