At Mon, 29 Oct 2007 08:30:50 +0100, I wrote:
At Fri, 26 Oct 2007 22:45:38 +0530, Kamalesh Babulal wrote:
Takashi Iwai wrote:
At Fri, 26 Oct 2007 08:52:02 -0700, Greg KH wrote:
On Fri, Oct 26, 2007 at 02:47:22PM +0530, Kamalesh Babulal wrote:
I get similar warning, while loading the alsa driver
This is a different problem, not a rename issue like the network one was.
[ 191.933548] Advanced Linux Sound Architecture Driver Version 1.0.15 (Tue Oct 23 06:09:18 2007 UTC). [ 191.996323] ALSA sound/core/seq/oss/seq_oss.c:232: can't register device seq [ 192.046004] sysfs: duplicate filename 'audio' can not be created [ 192.086048] WARNING: at fs/sysfs/dir.c:424 sysfs_add_one()
This is an error in the alsa core, perhaps the alsa developers can help figure out why they are registering the same file twice?
ALSA core shouldn't do so. Maybe a built-in OSS driver conflict with ALSA? We'd need to check .config, then.
Takashi
Hi Takashi,
Attaching the .config file,
You're using allyesconfig, and it's a missing runtime configuration to avoid the duplication. Some ALSA and OSS drivers can be loaded without real hardwares. With allyesconfig, ALSA and OSS drivers may be registered to the same device unless explicitly set up.
Meanwhile, it's not sexy to have such warnings from sysfs core, not from the caller side.
The (untested) patch below may fix the problem. Now the standard register_sound_*() is used for old OSS code. Please give it a try.
Takashi
[PATCH] Use register_sound_*() in OSS core code
Use the standard register_sound_*() functions instead of calling device_create() directly in the old OSS core code.
Signed-off-by: Takashi Iwai tiwai@suse.de
---
diff --git a/sound/oss/soundcard.c b/sound/oss/soundcard.c index a9c23b2..2c20e70 100644 --- a/sound/oss/soundcard.c +++ b/sound/oss/soundcard.c @@ -523,15 +523,11 @@ bad: */ static const struct { unsigned short minor; - char *name; - umode_t mode; int *num; } dev_list[] = { /* list of minor devices */ /* seems to be some confusion here -- this device is not in the device list */ - {SND_DEV_DSP16, "dspW", S_IWUGO | S_IRUSR | S_IRGRP, - &num_audiodevs}, - {SND_DEV_AUDIO, "audio", S_IWUGO | S_IRUSR | S_IRGRP, - &num_audiodevs}, + {SND_DEV_DSP16, &num_audiodevs}, + {SND_DEV_AUDIO, &num_audiodevs}, };
static int dmabuf; @@ -560,17 +556,11 @@ static int __init oss_init(void) sound_dmap_flag = (dmabuf > 0 ? 1 : 0);
for (i = 0; i < ARRAY_SIZE(dev_list); i++) { - device_create(sound_class, NULL, - MKDEV(SOUND_MAJOR, dev_list[i].minor), - "%s", dev_list[i].name); - if (!dev_list[i].num) continue; - - for (j = 1; j < *dev_list[i].num; j++) - device_create(sound_class, NULL, - MKDEV(SOUND_MAJOR, dev_list[i].minor + (j*0x10)), - "%s%d", dev_list[i].name, j); + for (j = 0; j < *dev_list[i].num; j++) + register_sound_special(&oss_sound_fops, + dev_list[i].minor + (j * 0x10)); }
if (sound_nblocks >= 1024) @@ -584,11 +574,10 @@ static void __exit oss_cleanup(void) int i, j;
for (i = 0; i < ARRAY_SIZE(dev_list); i++) { - device_destroy(sound_class, MKDEV(SOUND_MAJOR, dev_list[i].minor)); if (!dev_list[i].num) continue; - for (j = 1; j < *dev_list[i].num; j++) - device_destroy(sound_class, MKDEV(SOUND_MAJOR, dev_list[i].minor + (j*0x10))); + for (j = 0; j < *dev_list[i].num; j++) + unregister_sound_special(dev_list[i].minor + (j * 0x10)); } unregister_sound_special(1); diff --git a/sound/sound_core.c b/sound/sound_core.c index 46daca1..3a22848 100644 --- a/sound/sound_core.c +++ b/sound/sound_core.c @@ -259,6 +259,9 @@ int register_sound_special_device(const struct file_operations *fops, int unit, case 4: name = "audio"; break; + case 5: + name = "dspW"; + break; case 8: name = "sequencer2"; if (unit >= SOUND_STEP)