All of lore.kernel.org
 help / color / mirror / Atom feed
From: Clemens Ladisch <clemens@ladisch.de>
To: Takashi Iwai <tiwai@suse.de>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	Wu Fengguang <fengguang.wu@intel.com>
Subject: Re: [PATCH 1/5] allow up to 32 PCM devices
Date: Wed, 12 May 2010 12:20:33 +0200	[thread overview]
Message-ID: <4BEA80F1.7040600@ladisch.de> (raw)
In-Reply-To: <s5hbpclsdbu.wl%tiwai@suse.de>

Takashi Iwai wrote:
> Wu Fengguang wrote:
> > > Jaroslav Kysela wrote:
> > > > I don't agree to have only 4 slots for soundcards in the static minor 
> > > > numbering. Maybe the driver should be converted to use subdevices or we 
> > > > might drop the static minor number allocation at all (it might have only 
> > > > impact for old distros).
> > 
> > Jaroslav, will there be so many sound cards in one system?
> 
> In the old time, yes.  Now we have less and less PCI slots.
> In theory, we may have lots of USB audio devices, though :)

I implemented CONFIG_SND_DYNAMIC_MINORS because people had been asking
for more than eight cards.  (And by now I have lots of cards too,
although my computer probably isn't very typical.)

Anyway, static numbering is needed only for systems without udev/devfs,
and there we shouldn't change it for backwards compatibility.  The HDA
driver already requires kernels >= 2.6, so I don't see a problem with
requiring CONFIG_SND_DYNAMIC_MINORS to get all HDMI outputs.

> Another possible solution would be to change the minor number
> assignment to a really dynamic one.  So far, due to legacy /dev/aload
> and co, we have some static restriction per card basis.

What restriction would that be?  With CONFIG_SND_DYNAMIC_MINORS, we
don't allocate minors that would be used by /dev/aload*, but there are
no restrictions on the number of cards or devices.


-- 
Regards,
Clemens


cat /proc/asound/cards
 0 [SB             ]: HDA-Intel - HDA ATI SB
                      HDA ATI SB at 0xfe7f4000 irq 16
 1 [HDMI           ]: HDA-Intel - HDA ATI HDMI
                      HDA ATI HDMI at 0xfe9e8000 irq 19
 2 [D2             ]: AV200 - Xonar D2
                      Asus Virtuoso 200 (rev 2) at 0xe800, irq 20
 3 [YMF754         ]: YMF754 - Yamaha DS-1E (YMF754)
                      Yamaha DS-1E (YMF754) at 0xfebf8000, irq 22
 4 [UA1A           ]: USB-Audio - EDIROL UA-1A
                      Roland EDIROL UA-1A at usb-0000:00:12.2-4.3, full speed
 5 [SC8820         ]: USB-Audio - SC-8820
                      Roland SC-8820 at usb-0000:00:13.0-2, full speed
 6 [UM2            ]: USB-Audio - UM-2
                      EDIROL UM-2 at usb-0000:00:12.0-2, full speed
 7 [NX             ]: USB-Audio - SB Audigy 2 NX
                      Creative Technology Ltd SB Audigy 2 NX at usb-0000:00:12.2-3, high speed
 8 [UART16550A     ]: Serial - Serial MIDI (UART16550A)
                      Serial MIDI (UART16550A) [Soundcanvas] at 0x3f8, irq 4
 9 [VirMIDI        ]: VirMIDI - VirMIDI
                      Virtual MIDI Card 1
10 [Dummy          ]: Dummy - Dummy
                      Dummy 1
11 [Live           ]: EMU10K1 - SB Live! Value [CT4670]
                      SB Live! Value [CT4670] (rev.5, serial:0x201102) at 0xec00, irq 21
12 [pcsp           ]: PC-Speaker - pcsp
                      Internal PC-Speaker at port 0x61
13 [AudioFire2     ]: FireWorks - AudioFire2
                      Echo Digital Audio AudioFire2 (v3.2) at fw1.0, S400

  parent reply	other threads:[~2010-05-12 10:20 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-12  1:30 [PATCH 0/5] updates for Intel SandyBridge/CougarPoint HDMI codec Wu Fengguang
2010-05-12  1:30 ` [PATCH 1/5] allow up to 32 PCM devices Wu Fengguang
2010-05-12  7:29   ` Jaroslav Kysela
2010-05-12  8:03     ` Takashi Iwai
2010-05-12  8:39       ` Wu Fengguang
2010-05-12  9:01         ` Takashi Iwai
2010-05-12 10:06           ` Wu Fengguang
2010-05-13  0:05             ` Eliot Blennerhassett
2010-05-12 10:20           ` Clemens Ladisch [this message]
2010-05-12 10:55             ` Takashi Iwai
2010-05-13  2:21               ` Wu Fengguang
2010-05-14  8:21                 ` Takashi Iwai
2010-05-14  8:32                   ` Wu Fengguang
2010-05-12  9:49       ` Jaroslav Kysela
2010-05-12  1:30 ` [PATCH 2/5] hda - allow up to 10 Azalia codecs Wu Fengguang
2010-05-12 14:35   ` Takashi Iwai
2010-05-13  3:03     ` Wu Fengguang
2010-05-12  1:30 ` [PATCH 3/5] intelhdmi - user friendly codec name Wu Fengguang
2010-05-12  1:30 ` [PATCH 4/5] intelhdmi - add id for the CougarPoint chipset Wu Fengguang
2010-05-12  1:30 ` [PATCH 5/5] hdmi - dont fail on extra nodes Wu Fengguang

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=4BEA80F1.7040600@ladisch.de \
    --to=clemens@ladisch.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=fengguang.wu@intel.com \
    --cc=tiwai@suse.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 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.