BMC/Intel-BMC/linux.git/sound/isa/wavefront, branch dev

sound: Deparenthesize negative error returns

2015-03-24T10:41:37+00:00

Make the returns a bit more kernel standard style. Signed-off-by: Joe Perches Signed-off-by: Takashi Iwai

ALSA: Include linux/io.h instead of asm/io.h

2015-01-28T15:49:33+00:00

Nowadays it's recommended. Replace all in a shot. Signed-off-by: Takashi Iwai

ALSA: wavefront: Use setup_timer() and mod_timer()

2015-01-19T10:32:28+00:00

No functional change, refactoring with the standard helpers. Signed-off-by: Takashi Iwai

ALSA: wss: Remove (almost) always NULL parameters

2015-01-02T15:30:08+00:00

Most callers of snd_wss_pcm(), snd_wss_timer() and snd_cs4236_pcm() pass NULL as the last parameter, some callers pass a pointer but never use it after the function has been called and only a few callers pass a pointer and actually use it. The later is only the case for snd_wss_pcm() for snd_cs4236_pcm() and it is possible to get the same PCM object by accessing the pcm field of the snd_wss struct that was passed as the first parameter. This function removes the last parameters from the functions mentioned above and updates the callers which used it to use chip->pcm instead. This allows us to slightly simplify the functions since they don't have to check and set the last parameter anymore which makes the code slightly shorter and cleaner. Signed-off-by: Lars-Peter Clausen Signed-off-by: Takashi Iwai

ALSA: isa: Convert to snd_card_new() with a device pointer

2014-02-12T09:58:21+00:00

Also remove superfluous snd_card_set_dev() calls. Signed-off-by: Takashi Iwai

ALSA: isa: not allocating enough space

2013-11-13T08:28:31+00:00

We allocate only 4 bytes here "sizeof(int)" instead of WF_MSAMPLE_BYTES (259) which was intended. Signed-off-by: Dan Carpenter Signed-off-by: Takashi Iwai

ALSA: ISA: Remove superfluous *_set_drvdata(NULL) calls

2013-05-29T10:50:28+00:00

Similarly like the previous commit for PCI drivers, remove dev_set_drvdata(NULL) and pnp_set_drvdata(NULL) calls in ISA drivers now. Signed-off-by: Takashi Iwai

ALSA: isa: remove __dev* attributes

2012-12-07T06:32:17+00:00

CONFIG_HOTPLUG is going away as an option. As result the __dev* markings will be going away. Remove use of __devinit, __devexit_p, __devinitdata, __devinitconst, and __devexit. Signed-off-by: Bill Pemberton Signed-off-by: Takashi Iwai

sound: Remove unnecessary semicolon

2012-10-06T14:47:53+00:00

A simplified version of the semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r1@ statement S; position p,p1; @@ S@p1;@p @script:python r2@ p << r1.p; p1 << r1.p1; @@ if p[0].line != p1[0].line_end: cocci.include_match(False) @@ position r1.p; @@ -;@p // Signed-off-by: Peter Senna Tschudin Signed-off-by: Takashi Iwai

ALSA: module_param: make bool parameters really bool

2011-12-19T09:34:41+00:00

module_param(bool) used to counter-intuitively take an int. In fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy trick. It's time to remove the int/unsigned int option. For this version it'll simply give a warning, but it'll break next kernel version. Signed-off-by: Rusty Russell Signed-off-by: Takashi Iwai