[RFC PATCH 0/4] Replace VIDEO_COMMAND with VIDIOC_DECODER_CMD

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



During the 2011 workshop we discussed replacing the decoder commands in
include/linux/dvb/video.h and audio.h by a proper V4L2 API.

This patch series is the first phase of that. It adds new VIDIOC_(TRY_)DECODER_CMD
ioctls to the V4L2 API. These are identical to the VIDEO_(TRY_)COMMAND from
dvb/video.h, but the names of the fields and defines now conform to the V4L2
API conventions.

Documentation has been added and ivtv (the only V4L2 driver that used VIDEO_COMMAND)
has been adapted to support the new V4L2 API.

I do have one question for Mauro: what do you want to do with video.h? Should it be
removed altogether eventually?

Some of the commands defined there aren't used by any driver (e.g. VIDEO_GET_NAVI),
some are specific to the av7110 driver (e.g. VIDEO_STILLPICTURE), some are specific
to ivtv (VIDEO_COMMAND) and some are used by both ivtv and av7110 (e.g. VIDEO_PLAY).

My proposal would be to:

1) remove anything that is not used by any driver from audio.h and video.h
2) move av7110 specific stuff to a new linux/av7110.h header
3) move ivtv specific stuff to the linux/ivtv.h header
4) shared code should be moved to the new linux/av7110.h header and also copied
   to linux/ivtv.h. The ivtv version will rename the names (e.g. VIDEO_ becomes
   IVTV_) but is otherwise unchanged to preserve the ABI. Comments are added
   on how to convert the legacy ioctls to standard V4L2 API in applications.
   Perhaps these legacy ioctls in ivtv can even be removed in a few years time.
5) remove linux/dvb/audio.h and video.h.

What do you think, Mauro?

Regards,

	Hans

--
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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux