From: Klaus Schmidinger <Klaus.Schmidinger@tvdr.de>
To: linux-media@vger.kernel.org
Subject: Re: [RFCv2 PATCH 10/12] av7110: replace audio.h, video.h and osd.h by av7110.h.
Date: Fri, 25 Nov 2011 16:35:59 +0100 [thread overview]
Message-ID: <4ECFB5DF.1040806@tvdr.de> (raw)
In-Reply-To: <5276295e57ca56ed2a27148d918b63b00dd05b34.1322141686.git.hans.verkuil@cisco.com>
On 24.11.2011 14:39, Hans Verkuil wrote:
> From: Hans Verkuil<hans.verkuil@cisco.com>
>
> Create a new public header, av7110.h, that contains all the av7110
> specific audio, video and osd APIs that used to be defined in dvb/audio.h,
> dvb/video.h and dvb/osd.h. These APIs are no longer part of DVBv5 but are
> now av7110-specific.
>
> This decision was taken during the 2011 Prague V4L-DVB workshop.
>
> Ideally av7110 would be converted to use the replacement V4L2 MPEG
> decoder API, but that's a huge job for such an old driver.
>
> Signed-off-by: Hans Verkuil<hans.verkuil@cisco.com>
This would break applications, especially VDR.
I therefore strongly oppose this!
You may introduce new APIs as you like, but don't break the
existing ones that have worked for many years.
Nacked-by: Klaus Schmidinger <Klaus.Schmidinger@tvdr.de>
Klaus
next prev parent reply other threads:[~2011-11-25 15:44 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-24 13:38 Remove audio and video DVBv5 API Hans Verkuil
2011-11-24 13:38 ` [RFCv2 PATCH 01/12] v4l2: add VIDIOC_(TRY_)DECODER_CMD Hans Verkuil
2011-11-24 13:38 ` [RFCv2 PATCH 02/12] v4l spec: document VIDIOC_(TRY_)DECODER_CMD Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 03/12] ivtv: implement new decoder command ioctls Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 04/12] v4l2-ctrls: add new controls for MPEG decoder devices Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 05/12] Document decoder controls Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 06/12] ivtv: implement new " Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 07/12] cx18/ddbridge: remove unused headers Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 08/12] ivtv: extend ivtv.h with structs and ioctls from dvb/audio.h and video.h Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 09/12] ivtv: use the new ivtv-specific ioctls from ivtv.h Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 10/12] av7110: replace audio.h, video.h and osd.h by av7110.h Hans Verkuil
2011-11-24 23:24 ` Oliver Endriss
2011-11-25 15:35 ` Klaus Schmidinger [this message]
2011-11-24 13:39 ` [RFCv2 PATCH 11/12] Replace audio.xml and video.xml with av.xml Hans Verkuil
2011-11-24 13:39 ` [RFCv2 PATCH 12/12] Remove audio.h, video.h and osd.h Hans Verkuil
2011-11-24 17:08 ` Andreas Oberritter
2011-11-24 17:37 ` Mauro Carvalho Chehab
2011-11-24 17:59 ` Manu Abraham
2011-11-24 18:01 ` Andreas Oberritter
2011-11-24 23:32 ` Oliver Endriss
2011-11-24 17:44 ` Hans Verkuil
2011-11-24 17:51 ` Andreas Oberritter
2011-11-24 17:58 ` Mauro Carvalho Chehab
2011-11-24 18:07 ` Andreas Oberritter
2011-11-24 18:25 ` Mauro Carvalho Chehab
2011-11-24 18:34 ` Manu Abraham
2011-11-25 1:09 ` Andreas Oberritter
2011-11-25 2:44 ` Mauro Carvalho Chehab
2011-11-25 12:55 ` Andreas Oberritter
2011-11-25 15:18 ` Mauro Carvalho Chehab
2011-11-25 15:25 ` Hans Verkuil
2011-11-25 16:00 ` Mauro Carvalho Chehab
2011-11-24 18:01 ` Manu Abraham
2011-11-24 18:08 ` Mauro Carvalho Chehab
2011-11-24 18:13 ` Manu Abraham
2011-11-24 18:47 ` Mauro Carvalho Chehab
2011-11-24 18:51 ` Manu Abraham
2011-11-24 19:05 ` Manu Abraham
2011-11-25 12:00 ` Andreas Oberritter
2011-11-25 13:48 ` Mauro Carvalho Chehab
2011-11-25 13:59 ` Manu Abraham
2011-11-25 14:41 ` Andreas Oberritter
2011-11-25 15:38 ` Mauro Carvalho Chehab
2011-11-25 16:03 ` Andreas Oberritter
2011-11-25 16:26 ` Mauro Carvalho Chehab
2011-11-25 16:51 ` Manu Abraham
2011-11-25 22:06 ` Andreas Oberritter
2011-11-26 5:55 ` Oliver Endriss
2011-11-26 6:25 ` Manu Abraham
2011-11-26 11:32 ` Mauro Carvalho Chehab
2011-11-26 11:59 ` Mauro Carvalho Chehab
2011-11-26 12:46 ` Oliver Endriss
2011-11-26 11:49 ` Hans Verkuil
2011-11-26 20:27 ` Andreas Oberritter
2011-11-27 18:28 ` Mauro Carvalho Chehab
2011-11-26 21:58 ` Manu Abraham
2011-11-27 19:03 ` Mauro Carvalho Chehab
2011-11-27 19:27 ` Manu Abraham
2011-11-27 21:39 ` Mauro Carvalho Chehab
2011-11-27 22:24 ` Manu Abraham
2011-11-27 22:50 ` Mauro Carvalho Chehab
2011-11-26 22:11 ` Manu Abraham
2011-11-25 15:22 ` Hans Verkuil
2011-11-25 15:52 ` Mauro Carvalho Chehab
2011-11-26 10:44 ` Hans Verkuil
2011-11-25 15:58 ` Manu Abraham
2011-11-25 16:03 ` Mauro Carvalho Chehab
2011-11-25 16:11 ` Manu Abraham
2011-11-24 23:25 ` Oliver Endriss
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4ECFB5DF.1040806@tvdr.de \
--to=klaus.schmidinger@tvdr.de \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.