[FFmpeg-devel] [PATCH] avdevice: add fbdev output device

Stefano Sabatini stefasab at gmail.com
Sat Oct 19 00:39:27 CEST 2013


On date Friday 2013-10-18 23:11:12 +0200, Lukasz Marek encoded:
> Signed-off-by: Lukasz Marek <lukasz.m.luki at gmail.com>
> ---
>  Changelog                |    1 +
>  configure                |    1 +
>  doc/outdevs.texi         |   31 ++++++
>  libavdevice/Makefile     |    1 +
>  libavdevice/alldevices.c |    2 +-
>  libavdevice/fbdev_enc.c  |  256 ++++++++++++++++++++++++++++++++++++++++++++++
>  libavdevice/version.h    |    2 +-
>  7 files changed, 292 insertions(+), 2 deletions(-)
>  create mode 100644 libavdevice/fbdev_enc.c

You're welcome to add an entry in MAINTAINERS if you want to work on
it.

> 
> diff --git a/Changelog b/Changelog
> index ee58336..e196d8e 100644
> --- a/Changelog
> +++ b/Changelog
> @@ -40,6 +40,7 @@ version <next>
>  - dpx parser
>  - max_error_rate parameter in ffmpeg
>  - PulseAudio output device
> +- Linux framebuffer output device
>  
>  
>  version 2.0:
> diff --git a/configure b/configure
> index 101954e..0eb62b3 100755
> --- a/configure
> +++ b/configure
> @@ -2125,6 +2125,7 @@ dshow_indev_extralibs="-lpsapi -lole32 -lstrmiids -luuid"
>  dv1394_indev_deps="dv1394"
>  dv1394_indev_select="dv_demuxer"
>  fbdev_indev_deps="linux_fb_h"
> +fbdev_outdev_deps="linux_fb_h"
>  iec61883_indev_deps="libiec61883"
>  jack_indev_deps="jack_jack_h sem_timedwait"
>  lavfi_indev_deps="avfilter"
> diff --git a/doc/outdevs.texi b/doc/outdevs.texi
> index 55c9110..569b402 100644
> --- a/doc/outdevs.texi
> +++ b/doc/outdevs.texi
> @@ -104,6 +104,37 @@ ffmpeg -i INPUT -pix_fmt rgb24 -f caca -list_dither colors -
>  @end example
>  @end itemize
>  
> + at section fbdev
> +
> +Linux framebuffer output device.
> +
> +The Linux framebuffer is a graphic hardware-independent abstraction
> +layer to show graphics on a computer monitor, typically on the
> +console. It is accessed through a file device node, usually
> + at file{/dev/fb0}.
> +
> +For more detailed information read the file
> +Documentation/fb/framebuffer.txt included in the Linux source tree.
> +
> + at subsection Options
> + at table @option
> +

> + at item xoffset
> +Set x coordinate of top left corner. Default is 0.
> +
> + at item yoffset
> +Set y coordinate of top left corner. Default is 0.

You can merge these descriptions:

@item xoffset
@item yoffset
Set x/y coordinate of top left corner. Default is 0.

> + at end table
> +
> + at subsection Examples
> +Play a file on framebuffer device @file{/dev/fb0}.
> +Required pixel format depends on current framebuffer settings.
> + at example
> +ffmpeg -re -i INPUT -vcodec rawvideo -pix_fmt bgra -f fbdev /dev/fb0
> + at end example
> +
> +See also @url{http://linux-fbdev.sourceforge.net/}, and fbset(1).
> +
>  @section oss
>  
>  OSS (Open Sound System) output device.
> diff --git a/libavdevice/Makefile b/libavdevice/Makefile
> index ad959ee..f6f05e5 100644
> --- a/libavdevice/Makefile
> +++ b/libavdevice/Makefile
> @@ -24,6 +24,7 @@ OBJS-$(CONFIG_DSHOW_INDEV)               += dshow.o dshow_enummediatypes.o \
>                                              dshow_pin.o dshow_common.o
>  OBJS-$(CONFIG_DV1394_INDEV)              += dv1394.o
>  OBJS-$(CONFIG_FBDEV_INDEV)               += fbdev.o
> +OBJS-$(CONFIG_FBDEV_OUTDEV)              += fbdev_enc.o
>  OBJS-$(CONFIG_IEC61883_INDEV)            += iec61883.o
>  OBJS-$(CONFIG_JACK_INDEV)                += jack_audio.o timefilter.o
>  OBJS-$(CONFIG_LAVFI_INDEV)               += lavfi.o
> diff --git a/libavdevice/alldevices.c b/libavdevice/alldevices.c
> index 33ce155..5178f30 100644
> --- a/libavdevice/alldevices.c
> +++ b/libavdevice/alldevices.c
> @@ -51,7 +51,7 @@ void avdevice_register_all(void)
>      REGISTER_OUTDEV  (CACA,             caca);
>      REGISTER_INDEV   (DSHOW,            dshow);
>      REGISTER_INDEV   (DV1394,           dv1394);
> -    REGISTER_INDEV   (FBDEV,            fbdev);
> +    REGISTER_INOUTDEV(FBDEV,            fbdev);
>      REGISTER_INDEV   (IEC61883,         iec61883);
>      REGISTER_INDEV   (JACK,             jack);
>      REGISTER_INDEV   (LAVFI,            lavfi);
> diff --git a/libavdevice/fbdev_enc.c b/libavdevice/fbdev_enc.c
> new file mode 100644
> index 0000000..d92a6d1
> --- /dev/null
> +++ b/libavdevice/fbdev_enc.c
> @@ -0,0 +1,256 @@
> +/*
> + * Copyright (c) 2013 Lukasz Marek
> + *
> + * This file is part of FFmpeg.
> + *
> + * FFmpeg is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU Lesser General Public
> + * License as published by the Free Software Foundation; either
> + * version 2.1 of the License, or (at your option) any later version.
> + *
> + * FFmpeg is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with FFmpeg; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> + */
> +
> +#include <unistd.h>
> +#include <fcntl.h>
> +#include <sys/ioctl.h>
> +#include <sys/mman.h>
> +#include <linux/fb.h>
> +#include "libavutil/pixdesc.h"
> +#include "libavutil/log.h"
> +#include "libavutil/mem.h"
> +#include "libavutil/opt.h"
> +#include "libavformat/avformat.h"
> +
> +typedef struct {
> +    AVClass *class;                   ///< class for private options
> +    int xoffset;                      ///< x coordinate of top left corner
> +    int yoffset;                      ///< y coordinate of top left corner
> +    struct fb_var_screeninfo varinfo; ///< variable info;
> +    struct fb_fix_screeninfo fixinfo; ///< fixed info;
> +    int fd;                           ///< framebuffer device file descriptor

> +    int stream_index;                 ///< Index of a video stream

nit++++: index

> +    uint8_t *data;                    ///< framebuffer data
> +    int width;
> +    int height;

> +    int bpp;

nit: bytes_per_pixels or add a comment, since the name "bpp" has been
historically confusing.

> +    enum AVPixelFormat pix_fmt;
> +} FBDevContext;
> +

> +struct rgb_pixfmt_map_entry {
> +    int bits_per_pixel;
> +    int red_offset, green_offset, blue_offset, alpha_offset;
> +    enum AVPixelFormat pixfmt;
> +};
> +
> +static const struct rgb_pixfmt_map_entry rgb_pixfmt_map[] = {
> +    // bpp, red_offset,  green_offset, blue_offset, alpha_offset, pixfmt
> +    {  32,       0,           8,          16,           24,   AV_PIX_FMT_RGBA  },
> +    {  32,      16,           8,           0,           24,   AV_PIX_FMT_BGRA  },
> +    {  32,       8,          16,          24,            0,   AV_PIX_FMT_ARGB  },
> +    {  32,       3,           2,           8,            0,   AV_PIX_FMT_ABGR  },
> +    {  24,       0,           8,          16,            0,   AV_PIX_FMT_RGB24 },
> +    {  24,      16,           8,           0,            0,   AV_PIX_FMT_BGR24 },
> +    {  16,      11,           5,           0,           16,   AV_PIX_FMT_RGB565 },
> +};
> +
> +static enum AVPixelFormat get_pixfmt_from_fb_varinfo(struct fb_var_screeninfo *varinfo)
> +{
> +    int i;
> +
> +    for (i = 0; i < FF_ARRAY_ELEMS(rgb_pixfmt_map); i++) {
> +        const struct rgb_pixfmt_map_entry *entry = &rgb_pixfmt_map[i];
> +        if (entry->bits_per_pixel == varinfo->bits_per_pixel &&
> +            entry->red_offset     == varinfo->red.offset     &&
> +            entry->green_offset   == varinfo->green.offset   &&
> +            entry->blue_offset    == varinfo->blue.offset)
> +            return entry->pixfmt;
> +    }
> +
> +    return AV_PIX_FMT_NONE;
> +}

assuming these are the same as in fbdev.c, they should be shared in a
common file. Feel free to rename fbdev.c -> fbdev-dec.c.

> +
> +static av_cold int fbdev_write_header(AVFormatContext *h)
> +{
> +    FBDevContext *s = h->priv_data;
> +    enum AVPixelFormat pix_fmt;
> +    AVStream *st = NULL;
> +    int ret, flags = O_RDWR;
> +
> +    for (int i = 0; i < h->nb_streams; i++) {
> +        if (h->streams[i]->codec->codec_type == AVMEDIA_TYPE_VIDEO) {
> +            s->stream_index = i;
> +            st = h->streams[i];
> +            break;
> +        }
> +    }
> +    if (!st) {
> +        av_log(h, AV_LOG_ERROR, "No video stream found.\n");
> +        return AVERROR(EINVAL);
> +    }

You could add a check and an error or warning in case more than one
video stream is sent to the device.

> +
> +    if ((s->fd = avpriv_open(h->filename, flags)) == -1) {
> +        ret = AVERROR(errno);
> +        av_log(h, AV_LOG_ERROR,
> +               "Could not open framebuffer device '%s': %s\n",
> +               h->filename, strerror(ret));

av_err2str(ret), as is it is wrong since ret was negated (see my
recent commit on fbdev.c)

> +        return ret;
> +    }
> +
> +    if (ioctl(s->fd, FBIOGET_VSCREENINFO, &s->varinfo) < 0) {
> +        ret = AVERROR(errno);
> +        av_log(h, AV_LOG_ERROR, "FBIOGET_VSCREENINFO: %s\n", strerror(errno));
> +        goto fail;
> +    }
> +
> +    if (ioctl(s->fd, FBIOGET_FSCREENINFO, &s->fixinfo) < 0) {
> +        ret = AVERROR(errno);
> +        av_log(h, AV_LOG_ERROR, "FBIOGET_FSCREENINFO: %s\n", strerror(errno));
> +        goto fail;
> +    }
> +
> +    pix_fmt = get_pixfmt_from_fb_varinfo(&s->varinfo);
> +    if (pix_fmt == AV_PIX_FMT_NONE) {
> +        ret = AVERROR(EINVAL);

> +        av_log(h, AV_LOG_ERROR, "Framebuffer pixel format not supported.\n");
> +        goto fail;
> +    }
> +
> +    s->data = mmap(NULL, s->fixinfo.smem_len, PROT_WRITE, MAP_SHARED, s->fd, 0);
> +    if (s->data == MAP_FAILED) {
> +        ret = AVERROR(errno);
> +        av_log(h, AV_LOG_ERROR, "Error in mmap(): %s\n", strerror(errno));
> +        goto fail;
> +    }

strerror(errno) -> av_err2str(ret)
same below

> +
> +    s->pix_fmt = st->codec->pix_fmt;
> +    s->width = st->codec->width;
> +    s->height = st->codec->height;
> +    s->bpp = ((st->codec->bits_per_coded_sample + 7) >> 3);
> +
> +    return 0;
> +  fail:
> +    close(s->fd);
> +    return ret;
> +}
> +

> +static int fbdev_write_packet(AVFormatContext *h, AVPacket *pkt)
> +{
> +    FBDevContext *s = h->priv_data;

nit: I'd prefer a more meaningful name (e.g. "fbdev" or "fb" if you
feel like you don't like to type)

> +    uint8_t *pin, *pout;
> +    enum AVPixelFormat pix_fmt;
> +    int disp_height;
> +    int bytes_to_copy;
> +    int src_line_size = s->width * s->bpp;
> +
> +    if (s->stream_index != pkt->stream_index)
> +        return 0;
> +
> +    if (ioctl(s->fd, FBIOGET_VSCREENINFO, &s->varinfo) < 0)
> +        av_log(h, AV_LOG_WARNING,
> +               "Error refreshing variable info: %s\n", strerror(errno));
> +
> +    pix_fmt = get_pixfmt_from_fb_varinfo(&s->varinfo);
> +

> +    //TODO: add mapping between formats

what do you exactly mean (this is useful for the archive)?

> +    if (pix_fmt != s->pix_fmt) {
> +        av_log(h, AV_LOG_ERROR, "Not supported pixel format, use %s\n",
> +               av_get_pix_fmt_name(pix_fmt));
> +        return AVERROR(EINVAL);
> +    }

Better:

"Pixel format %s is not supported, use %s.\n"

[...]
-- 
FFmpeg = Fancy Faithful Moronic Proud Experimenting Geek


More information about the ffmpeg-devel mailing list