diff options
Diffstat (limited to 'sound/oss/soundcard.c')
| -rw-r--r-- | sound/oss/soundcard.c | 76 | 
1 files changed, 27 insertions, 49 deletions
diff --git a/sound/oss/soundcard.c b/sound/oss/soundcard.c index 46c0d03dbec..b70c7c8f9c5 100644 --- a/sound/oss/soundcard.c +++ b/sound/oss/soundcard.c @@ -87,7 +87,7 @@ int *load_mixer_volumes(char *name, int *levels, int present)  	int             i, n;  	for (i = 0; i < num_mixer_volumes; i++) { -		if (strcmp(name, mixer_vols[i].name) == 0) { +		if (strncmp(name, mixer_vols[i].name, 32) == 0) {  			if (present)  				mixer_vols[i].num = i;  			return mixer_vols[i].levels; @@ -99,7 +99,7 @@ int *load_mixer_volumes(char *name, int *levels, int present)  	}  	n = num_mixer_volumes++; -	strcpy(mixer_vols[n].name, name); +	strncpy(mixer_vols[n].name, name, 32);  	if (present)  		mixer_vols[n].num = n; @@ -143,7 +143,7 @@ static int get_mixer_levels(void __user * arg)  static ssize_t sound_read(struct file *file, char __user *buf, size_t count, loff_t *ppos)  { -	int dev = iminor(file->f_path.dentry->d_inode); +	int dev = iminor(file_inode(file));  	int ret = -EINVAL;  	/* @@ -154,7 +154,6 @@ static ssize_t sound_read(struct file *file, char __user *buf, size_t count, lof  	mutex_lock(&soundcard_mutex); -	DEB(printk("sound_read(dev=%d, count=%d)\n", dev, count));  	switch (dev & 0x0f) {  	case SND_DEV_DSP:  	case SND_DEV_DSP16: @@ -176,11 +175,10 @@ static ssize_t sound_read(struct file *file, char __user *buf, size_t count, lof  static ssize_t sound_write(struct file *file, const char __user *buf, size_t count, loff_t *ppos)  { -	int dev = iminor(file->f_path.dentry->d_inode); +	int dev = iminor(file_inode(file));  	int ret = -EINVAL;  	mutex_lock(&soundcard_mutex); -	DEB(printk("sound_write(dev=%d, count=%d)\n", dev, count));  	switch (dev & 0x0f) {  	case SND_DEV_SEQ:  	case SND_DEV_SEQ2: @@ -206,7 +204,6 @@ static int sound_open(struct inode *inode, struct file *file)  	int dev = iminor(inode);  	int retval; -	DEB(printk("sound_open(dev=%d)\n", dev));  	if ((dev >= SND_NDEVS) || (dev < 0)) {  		printk(KERN_ERR "Invalid minor device %d\n", dev);  		return -ENXIO; @@ -257,7 +254,6 @@ static int sound_release(struct inode *inode, struct file *file)  	int dev = iminor(inode);  	mutex_lock(&soundcard_mutex); -	DEB(printk("sound_release(dev=%d)\n", dev));  	switch (dev & 0x0f) {  	case SND_DEV_CTL:  		module_put(mixer_devs[dev >> 4]->owner); @@ -333,7 +329,7 @@ static int sound_mixer_ioctl(int mixdev, unsigned int cmd, void __user *arg)  static long sound_ioctl(struct file *file, unsigned int cmd, unsigned long arg)  {  	int len = 0, dtype; -	int dev = iminor(file->f_dentry->d_inode); +	int dev = iminor(file_inode(file));  	long ret = -EINVAL;  	void __user *p = (void __user *)arg; @@ -351,7 +347,6 @@ static long sound_ioctl(struct file *file, unsigned int cmd, unsigned long arg)  			if (!access_ok(VERIFY_WRITE, p, len))  				return -EFAULT;  	} -	DEB(printk("sound_ioctl(dev=%d, cmd=0x%x, arg=0x%x)\n", dev, cmd, arg));  	if (cmd == OSS_GETVERSION)  		return __put_user(SOUND_VERSION, (int __user *)p); @@ -406,10 +401,9 @@ static long sound_ioctl(struct file *file, unsigned int cmd, unsigned long arg)  static unsigned int sound_poll(struct file *file, poll_table * wait)  { -	struct inode *inode = file->f_path.dentry->d_inode; +	struct inode *inode = file_inode(file);  	int dev = iminor(inode); -	DEB(printk("sound_poll(dev=%d)\n", dev));  	switch (dev & 0x0f) {  	case SND_DEV_SEQ:  	case SND_DEV_SEQ2: @@ -431,7 +425,7 @@ static int sound_mmap(struct file *file, struct vm_area_struct *vma)  	int dev_class;  	unsigned long size;  	struct dma_buffparms *dmap = NULL; -	int dev = iminor(file->f_path.dentry->d_inode); +	int dev = iminor(file_inode(file));  	dev_class = dev & 0x0f;  	dev >>= 4; @@ -526,31 +520,21 @@ bad:  } -/* These device names follow the official Linux device list, - * Documentation/devices.txt.  Let us know if there are other - * common names we should support for compatibility. - * Only those devices not created by the generic code in sound_core.c are - * registered here. - */ -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}, -}; -  static int dmabuf;  static int dmabug;  module_param(dmabuf, int, 0444);  module_param(dmabug, int, 0444); +/* additional minors for compatibility */ +struct oss_minor_dev { +	unsigned short minor; +	unsigned int enabled; +} dev_list[] = { +	{ SND_DEV_DSP16 }, +	{ SND_DEV_AUDIO }, +}; +  static int __init oss_init(void)  {  	int             err; @@ -571,18 +555,12 @@ 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), NULL, -			      "%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)), -				      NULL, "%s%d", dev_list[i].name, j); +		j = 0; +		do { +			unsigned short minor = dev_list[i].minor + j * 0x10; +			if (!register_sound_special(&oss_sound_fops, minor)) +				dev_list[i].enabled = (1 << j); +		} while (++j < num_audiodevs);  	}  	if (sound_nblocks >= MAX_MEM_BLOCKS - 1) @@ -596,11 +574,11 @@ 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))); +		j = 0; +		do { +			if (dev_list[i].enabled & (1 << j)) +				unregister_sound_special(dev_list[i].minor); +		} while (++j < num_audiodevs);  	}  	unregister_sound_special(1);  | 
