public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: LKML <linux-kernel@vger.kernel.org>,
	Lee Revell <rlrevell@joe-job.com>, Takashi Iwai <tiwai@suse.de>,
	alsa devel <alsa-devel@lists.sourceforge.net>,
	Andrew Morton <akpm@osdl.org>, Linus Torvalds <torvalds@osdl.org>
Cc: Johannes Stezenbach <js@linuxtv.org>,
	Linux and Kernel Video <video4linux-list@redhat.com>,
	Manu Abraham <abraham.manu@gmail.com>
Subject: [RFC] Moving snd-bt87x and btaudio to drivers/media
Date: Wed, 18 Jan 2006 11:29:28 -0200	[thread overview]
Message-ID: <1137590968.32449.46.camel@localhost> (raw)

Lee/Andrew/Linus/Takashi and others,

	Currently, we have some audio modules for multimedia devices under
drivers/media and others under sound. They also appear under different
points at Kconfig menus. 

	So, for BTTV support, current structure at linus -git is (from Kconfig
perspective):

	Sound/ALSA/PCI Devices/Bt87x Audio capture
		sound/snd-bt87x.ko - ALSA audio for bttv boards
	Sound/OSS/TV Card (bt848) mixer support
		drivers/media/video/tvmixer.ko

	Multimedia/V4L/BT848 V4L
		drivers/media/video/bttv.ko
	Multimedia/V4L/BT848 V4L/DVB-ATSC support
		drivers/media/dvb/bt8xx/bt878.ko
		drivers/media/dvb/bt8xx/dvb-bt8xx.o

	This I couldn't found at any Kconfig (but module exists, and also an
entry at Makefile):
sound/oss/Makefile:obj-$(CONFIG_SOUND_BT878)    += btaudio.o

	For SAA7134 and CX88, all are under Multimedia/V4L. All
OSS/ALSA/DVB/MPEG options are under the driver name.

	IMHO, from users perspective, it makes much more sense if all BTTV
moules (even sound ones) being under bttv video driver. Current module
allows using audio driver without video, but this doesn't make sense,
since audio will be only available, in practice, after selecting a
video/audio input or tuning a channel. These functionalities are
provided by bttv. So, we could even disable audio modules if bttv were
not compiled.

	Also, bug 5995 showed a problem when user have a bttv card and dvb is
also probed. dvb also handles audio, so, currently, it is mutually
exclusive with snd-bt87x audio.

	So, my proposal is to move sound/snd-bt87x.ko to drivers/media, moving
also its menu, and moving tvmixer menu also. After it, should move some
related code at bttv dvb modules, to reduce or eliminate mutually
exclusiveness between the two. 

	We might keep supporting btaudio, but I think this is already obsoleted
by alsa one, so, IMHO, we can just drop it. We intend to do the same
with saa7134-oss after some time (kernel 2.6.15 is the first with this
module, so we may remove it on 2.6.18 to give some time for testing).

Cheers, 
Mauro.


             reply	other threads:[~2006-01-18 13:29 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-18 13:29 Mauro Carvalho Chehab [this message]
2006-01-18 16:25 ` [RFC] Moving snd-bt87x and btaudio to drivers/media Takashi Iwai
2006-01-18 17:51   ` Mauro Carvalho Chehab
2006-01-18 19:04   ` Manu Abraham
2006-01-18 19:41 ` Adrian Bunk
2006-01-18 19:53   ` 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=1137590968.32449.46.camel@localhost \
    --to=mchehab@infradead.org \
    --cc=abraham.manu@gmail.com \
    --cc=akpm@osdl.org \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=js@linuxtv.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rlrevell@joe-job.com \
    --cc=tiwai@suse.de \
    --cc=torvalds@osdl.org \
    --cc=video4linux-list@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox