All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andreas Oberritter <obi@linuxtv.org>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Manu Abraham <abraham.manu@gmail.com>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	linux-media@vger.kernel.org,
	Hans Verkuil <hans.verkuil@cisco.com>
Subject: Re: [RFCv2 PATCH 12/12] Remove audio.h, video.h and osd.h.
Date: Fri, 25 Nov 2011 15:41:43 +0100	[thread overview]
Message-ID: <4ECFA927.10108@linuxtv.org> (raw)
In-Reply-To: <4ECF9C92.2040607@redhat.com>

On 25.11.2011 14:48, Mauro Carvalho Chehab wrote:
> If your complain is about the removal of audio.h, video.h

We're back on topic, thank you!

> and osd.h, then my proposal is
> to keep it there, writing a text that they are part of a deprecated API,

That's exactly what I proposed. Well, you shouldn't write "deprecated",
because it's not. Just explain - inside this text - when V4L2 should be
preferred over DVB.

> but keeping
> the rest of the patches

Which ones?

> and not accepting anymore any submission using them

Why? First you complain about missing users and then don't want to allow
any new ones.

>, removing
> the ioctl's that aren't used by av7110 from them.

That's just stupid. I can easily provide a list of used and valuable
ioctls, which need to remain present in order to not break userspace
applications.

Btw.: It's not easy to submit a driver for a SoC. Even if you are
legally allowed to do it, you have to first merge and maintain the board
support code before even thinking about multimedia.

Regards,
Andreas

  parent reply	other threads:[~2011-11-25 14:41 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
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 [this message]
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=4ECFA927.10108@linuxtv.org \
    --to=obi@linuxtv.org \
    --cc=abraham.manu@gmail.com \
    --cc=hans.verkuil@cisco.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    /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.