[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