Hi, no matter what that workshop discussed: *** It is not acceptable to change the DVB kernel <-> user-space API! *** The av7110 driver is working for years and still in use. I hereby NACK any attempt to remove dvb/audio.h or dvb/video.h. Nacked-by: Oliver Endriss <o.endriss@xxxxxx> On Wednesday 23 November 2011 12:12:32 Hans Verkuil wrote: > 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 -- ---------------------------------------------------------------- VDR Remote Plugin 0.4.0: http://www.escape-edv.de/endriss/vdr/ 4 MByte Mod: http://www.escape-edv.de/endriss/dvb-mem-mod/ Full-TS Mod: http://www.escape-edv.de/endriss/dvb-full-ts-mod/ ---------------------------------------------------------------- -- 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
