[alsa-devel] [PATCH 1/2] ALSA: soundcore: coding style fixes
Daniel Mack
zonque at gmail.com
Thu Jun 5 14:12:25 CEST 2014
sound/sounc_core.c has a larger number of style issues. Fix them to make the
code more readable. No functional change.
Signed-off-by: Daniel Mack <zonque at gmail.com>
---
sound/sound_core.c | 192 ++++++++++++++++++++++++-----------------------------
1 file changed, 85 insertions(+), 107 deletions(-)
diff --git a/sound/sound_core.c b/sound/sound_core.c
index 11e953a..bf9e41e 100644
--- a/sound/sound_core.c
+++ b/sound/sound_core.c
@@ -68,21 +68,16 @@ module_exit(cleanup_soundcore);
#ifdef CONFIG_SOUND_OSS_CORE
/*
* OSS sound core handling. Breaks out sound functions to submodules
- *
- * Author: Alan Cox <alan at lxorguk.ukuu.org.uk>
- *
- * Fixes:
*
+ * Author: Alan Cox <alan at lxorguk.ukuu.org.uk>
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version
* 2 of the License, or (at your option) any later version.
*
- * --------------------
- *
* Top level handler for the sound subsystem. Various devices can
- * plug into this. The fact they don't all go via OSS doesn't mean
+ * plug into this. The fact they don't all go via OSS doesn't mean
* they don't have to implement the OSS API. There is a lot of logic
* to keeping much of the OSS weight out of the code in a compatibility
* module, but it's up to the driver to rember to load it...
@@ -111,8 +106,7 @@ module_exit(cleanup_soundcore);
#define SOUND_STEP 16
-struct sound_unit
-{
+struct sound_unit {
int unit_minor;
const struct file_operations *unit_fops;
struct sound_unit *next;
@@ -158,8 +152,7 @@ module_param(preclaim_oss, int, 0444);
static int soundcore_open(struct inode *, struct file *);
-static const struct file_operations soundcore_fops =
-{
+static const struct file_operations soundcore_fops = {
/* We must have an owner or the module locking fails */
.owner = THIS_MODULE,
.open = soundcore_open,
@@ -171,71 +164,66 @@ static const struct file_operations soundcore_fops =
* join into it. Called with the lock asserted
*/
-static int __sound_insert_unit(struct sound_unit * s, struct sound_unit **list, const struct file_operations *fops, int index, int low, int top)
+static int __sound_insert_unit(struct sound_unit *s, struct sound_unit **list,
+ const struct file_operations *fops,
+ int index, int low, int top)
{
- int n=low;
+ int n = low;
if (index < 0) { /* first free */
+ while (*list && (*list)->unit_minor < n)
+ list = &((*list)->next);
- while (*list && (*list)->unit_minor<n)
- list=&((*list)->next);
-
- while(n<top)
- {
+ while (n < top) {
/* Found a hole ? */
- if(*list==NULL || (*list)->unit_minor>n)
+ if (*list == NULL || (*list)->unit_minor > n)
break;
- list=&((*list)->next);
- n+=SOUND_STEP;
+
+ list = &((*list)->next);
+ n += SOUND_STEP;
}
- if(n>=top)
+ if (n >= top)
return -ENOENT;
} else {
- n = low+(index*16);
+ n = low + (index * 16);
while (*list) {
- if ((*list)->unit_minor==n)
+ if ((*list)->unit_minor == n)
return -EBUSY;
- if ((*list)->unit_minor>n)
+ if ((*list)->unit_minor > n)
break;
- list=&((*list)->next);
+ list = &((*list)->next);
}
- }
-
- /*
- * Fill it in
- */
-
- s->unit_minor=n;
- s->unit_fops=fops;
-
- /*
- * Link it
- */
-
- s->next=*list;
- *list=s;
-
-
+ }
+
+ /* Fill it in */
+ s->unit_minor = n;
+ s->unit_fops = fops;
+
+ /* Link it */
+ s->next = *list;
+ *list = s;
+
return n;
}
/*
* Remove a node from the chain. Called with the lock asserted
*/
-
-static struct sound_unit *__sound_remove_unit(struct sound_unit **list, int unit)
+static struct sound_unit *__sound_remove_unit(struct sound_unit **list,
+ int unit)
{
- while(*list)
- {
- struct sound_unit *p=*list;
- if(p->unit_minor==unit)
- {
- *list=p->next;
+ while (*list) {
+ struct sound_unit *p = *list;
+
+ if (p->unit_minor == unit) {
+ *list = p->next;
return p;
}
- list=&(p->next);
+
+ list = &p->next;
}
+
printk(KERN_ERR "Sound device %d went missing!\n", unit);
return NULL;
}
@@ -250,12 +238,15 @@ static DEFINE_SPINLOCK(sound_loader_lock);
* Allocate the controlling structure and add it to the sound driver
* list. Acquires locks as needed
*/
-
-static int sound_insert_unit(struct sound_unit **list, const struct file_operations *fops, int index, int low, int top, const char *name, umode_t mode, struct device *dev)
+static int sound_insert_unit(struct sound_unit **list,
+ const struct file_operations *fops,
+ int index, int low, int top, const char *name,
+ umode_t mode, struct device *dev)
{
- struct sound_unit *s = kmalloc(sizeof(*s), GFP_KERNEL);
+ struct sound_unit *s;
int r;
+ s = kmalloc(sizeof(*s), GFP_KERNEL);
if (!s)
return -ENOMEM;
@@ -263,7 +254,7 @@ static int sound_insert_unit(struct sound_unit **list, const struct file_operati
retry:
r = __sound_insert_unit(s, list, fops, index, low, top);
spin_unlock(&sound_loader_lock);
-
+
if (r < 0)
goto fail;
else if (r < SOUND_STEP)
@@ -292,7 +283,8 @@ retry:
}
device_create(sound_class, dev, MKDEV(SOUND_MAJOR, s->unit_minor),
- NULL, "%s", s->name+6);
+ NULL, "%s", s->name + 6);
+
return s->unit_minor;
fail:
@@ -305,7 +297,6 @@ fail:
* completed the removal before their file operations become
* invalid.
*/
-
static void sound_remove_unit(struct sound_unit **list, int unit)
{
struct sound_unit *p;
@@ -357,7 +348,6 @@ static struct sound_unit *chains[SOUND_STEP];
* Return: The allocated number is returned on success. On failure,
* a negative error code is returned.
*/
-
int register_sound_special_device(const struct file_operations *fops, int unit,
struct device *dev)
{
@@ -367,69 +357,66 @@ int register_sound_special_device(const struct file_operations *fops, int unit,
char _name[16];
switch (chain) {
- case 0:
+ case 0:
name = "mixer";
break;
- case 1:
+ case 1:
name = "sequencer";
if (unit >= SOUND_STEP)
goto __unknown;
max_unit = unit + 1;
break;
- case 2:
+ case 2:
name = "midi";
break;
- case 3:
+ case 3:
name = "dsp";
break;
- case 4:
+ case 4:
name = "audio";
break;
- case 5:
+ case 5:
name = "dspW";
break;
- case 8:
+ case 8:
name = "sequencer2";
if (unit >= SOUND_STEP)
goto __unknown;
max_unit = unit + 1;
break;
- case 9:
+ case 9:
name = "dmmidi";
break;
- case 10:
+ case 10:
name = "dmfm";
break;
- case 12:
+ case 12:
name = "adsp";
break;
- case 13:
+ case 13:
name = "amidi";
break;
- case 14:
+ case 14:
name = "admmidi";
break;
- default:
- {
- __unknown:
- sprintf(_name, "unknown%d", chain);
- if (unit >= SOUND_STEP)
- strcat(_name, "-");
- name = _name;
- }
+ default:
+__unknown:
+ sprintf(_name, "unknown%d", chain);
+ if (unit >= SOUND_STEP)
+ strcat(_name, "-");
+ name = _name;
break;
}
+
return sound_insert_unit(&chains[chain], fops, -1, unit, max_unit,
name, S_IRUSR | S_IWUSR, dev);
}
-
EXPORT_SYMBOL(register_sound_special_device);
int register_sound_special(const struct file_operations *fops, int unit)
{
return register_sound_special_device(fops, unit, NULL);
}
-
EXPORT_SYMBOL(register_sound_special);
/**
@@ -443,13 +430,11 @@ EXPORT_SYMBOL(register_sound_special);
* Return: On success, the allocated number is returned. On failure,
* a negative error code is returned.
*/
-
int register_sound_mixer(const struct file_operations *fops, int dev)
{
return sound_insert_unit(&chains[0], fops, dev, 0, 128,
"mixer", S_IRUSR | S_IWUSR, NULL);
}
-
EXPORT_SYMBOL(register_sound_mixer);
/**
@@ -469,14 +454,13 @@ int register_sound_midi(const struct file_operations *fops, int dev)
return sound_insert_unit(&chains[2], fops, dev, 2, 130,
"midi", S_IRUSR | S_IWUSR, NULL);
}
-
EXPORT_SYMBOL(register_sound_midi);
/*
* DSP's are registered as a triple. Register only one and cheat
* in open - see below.
*/
-
+
/**
* register_sound_dsp - register a DSP device
* @fops: File operations for the driver
@@ -497,7 +481,6 @@ int register_sound_dsp(const struct file_operations *fops, int dev)
return sound_insert_unit(&chains[3], fops, dev, 3, 131,
"dsp", S_IWUSR | S_IRUSR, NULL);
}
-
EXPORT_SYMBOL(register_sound_dsp);
/**
@@ -514,7 +497,6 @@ void unregister_sound_special(int unit)
{
sound_remove_unit(&chains[unit % SOUND_STEP], unit);
}
-
EXPORT_SYMBOL(unregister_sound_special);
/**
@@ -529,7 +511,6 @@ void unregister_sound_mixer(int unit)
{
sound_remove_unit(&chains[0], unit);
}
-
EXPORT_SYMBOL(unregister_sound_mixer);
/**
@@ -544,7 +525,6 @@ void unregister_sound_midi(int unit)
{
sound_remove_unit(&chains[2], unit);
}
-
EXPORT_SYMBOL(unregister_sound_midi);
/**
@@ -561,21 +541,19 @@ void unregister_sound_dsp(int unit)
{
sound_remove_unit(&chains[3], unit);
}
-
-
EXPORT_SYMBOL(unregister_sound_dsp);
static struct sound_unit *__look_for_unit(int chain, int unit)
{
struct sound_unit *s;
-
- s=chains[chain];
- while(s && s->unit_minor <= unit)
- {
- if(s->unit_minor==unit)
+
+ s = chains[chain];
+ while (s && s->unit_minor <= unit) {
+ if (s->unit_minor == unit)
return s;
- s=s->next;
+ s = s->next;
}
+
return NULL;
}
@@ -586,14 +564,13 @@ static int soundcore_open(struct inode *inode, struct file *file)
struct sound_unit *s;
const struct file_operations *new_fops = NULL;
- chain=unit&0x0F;
- if(chain==4 || chain==5) /* dsp/audio/dsp16 */
- {
- unit&=0xF0;
- unit|=3;
- chain=3;
+ chain = unit & 0x0F;
+ if (chain == 4 || chain == 5) { /* dsp/audio/dsp16 */
+ unit &= 0xF0;
+ unit |= 3;
+ chain = 3;
}
-
+
spin_lock(&sound_loader_lock);
s = __look_for_unit(chain, unit);
if (s)
@@ -608,8 +585,8 @@ static int soundcore_open(struct inode *inode, struct file *file)
* ALSA toplevel modules for soundcards, thus we need
* load them at first. [Jaroslav Kysela <perex at jcu.cz>]
*/
- request_module("sound-slot-%i", unit>>4);
- request_module("sound-service-%i-%i", unit>>4, chain);
+ request_module("sound-slot-%i", unit >> 4);
+ request_module("sound-service-%i-%i", unit >> 4, chain);
/*
* sound-slot/service-* module aliases are scheduled
@@ -636,10 +613,11 @@ static int soundcore_open(struct inode *inode, struct file *file)
replace_fops(file, new_fops);
if (file->f_op->open)
- err = file->f_op->open(inode,file);
+ err = file->f_op->open(inode, file);
return err;
}
+
return -ENODEV;
}
--
1.9.3
More information about the Alsa-devel
mailing list