From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Oliver Endriss <o.endriss@gmx.de>
Cc: David Howells <dhowells@redhat.com>,
linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
Hans Verkuil <hverkuil@xs4all.nl>
Subject: Re: [GIT PULL] Disintegrate UAPI for media
Date: Thu, 11 Oct 2012 09:37:03 -0300 [thread overview]
Message-ID: <20121011093703.24b60232@infradead.org> (raw)
In-Reply-To: <201210111023.51311.o.endriss@gmx.de>
Em Thu, 11 Oct 2012 10:23:50 +0200
Oliver Endriss <o.endriss@gmx.de> escreveu:
> Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
> > Em Tue, 09 Oct 2012 14:30:24 +0100
> > David Howells <dhowells@redhat.com> escreveu:
> >
> > > Can you merge the following branch into the media tree please.
> > >
> > > This is to complete part of the Userspace API (UAPI) disintegration for which
> > > the preparatory patches were pulled recently. After these patches, userspace
> > > headers will be segregated into:
> > >
> > > include/uapi/linux/.../foo.h
> > >
> > > for the userspace interface stuff, and:
> > >
> > > include/linux/.../foo.h
> > >
> > > for the strictly kernel internal stuff.
> > >
> > > ---
> > > The following changes since commit 9e2d8656f5e8aa214e66b462680cf86b210b74a8:
> > >
> > > Merge branch 'akpm' (Andrew's patch-bomb) (2012-10-09 16:23:15 +0900)
> > >
> > > are available in the git repository at:
> > >
> > >
> > > git://git.infradead.org/users/dhowells/linux-headers.git tags/disintegrate-media-20121009
> > >
> > > for you to fetch changes up to 1c436decd49665be131887b08d172a7989cdceee:
> > >
> > > UAPI: (Scripted) Disintegrate include/linux/dvb (2012-10-09 09:48:42 +0100)
> > >
> > > ----------------------------------------------------------------
> > > UAPI Disintegration 2012-10-09
> > >
> > > ----------------------------------------------------------------
> > > David Howells (1):
> > > UAPI: (Scripted) Disintegrate include/linux/dvb
> > >
> > > include/linux/dvb/Kbuild | 8 -
> > > include/linux/dvb/dmx.h | 130 +--------------
> > > include/linux/dvb/video.h | 249 +----------------------------
> > > include/uapi/linux/dvb/Kbuild | 8 +
> > > include/{ => uapi}/linux/dvb/audio.h | 0
> > > include/{ => uapi}/linux/dvb/ca.h | 0
> > > include/uapi/linux/dvb/dmx.h | 155 ++++++++++++++++++
> > > include/{ => uapi}/linux/dvb/frontend.h | 0
> > > include/{ => uapi}/linux/dvb/net.h | 0
> > > include/{ => uapi}/linux/dvb/osd.h | 0
> > > include/{ => uapi}/linux/dvb/version.h | 0
> > > include/uapi/linux/dvb/video.h | 274 ++++++++++++++++++++++++++++++++
> > > 12 files changed, 439 insertions(+), 385 deletions(-)
> > > rename include/{ => uapi}/linux/dvb/audio.h (100%)
> > > rename include/{ => uapi}/linux/dvb/ca.h (100%)
> > > create mode 100644 include/uapi/linux/dvb/dmx.h
> > > rename include/{ => uapi}/linux/dvb/frontend.h (100%)
> > > rename include/{ => uapi}/linux/dvb/net.h (100%)
> > > rename include/{ => uapi}/linux/dvb/osd.h (100%)
> > > rename include/{ => uapi}/linux/dvb/version.h (100%)
> > > create mode 100644 include/uapi/linux/dvb/video.h
> >
> > Hmm... last year, it was decided that we would be putting the DVB av7110-only
> > API files on a separate place, as the API there conflicts with V4L/alsa APIs
>
> Wrong! Hans Verkuil and you tried to do it, without caring that it would
> break userspace, and it was NAKed.
My understanding here is that, as the file location will change
with this series, your original concern is now void, as userspace
will require patches to use the new location. So, if we're willing
to do it, let's put this one-driver-only obsolete API on a separate
place.
Cheers,
Mauro
next prev parent reply other threads:[~2012-10-11 12:37 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-09 13:30 [GIT PULL] Disintegrate UAPI for media David Howells
2012-10-09 21:39 ` Mauro Carvalho Chehab
2012-10-10 11:26 ` David Howells
2012-10-11 8:23 ` Oliver Endriss
2012-10-11 8:41 ` Manu Abraham
2012-10-11 12:37 ` Mauro Carvalho Chehab [this message]
2012-10-11 14:05 ` David Howells
2012-10-11 14:57 ` Mauro Carvalho Chehab
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=20121011093703.24b60232@infradead.org \
--to=mchehab@infradead.org \
--cc=dhowells@redhat.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=o.endriss@gmx.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox