linux-toradex.git/sound/isa/msnd, branch v4.4.23

ALSA: msnd: add some missing curly braces

2015-03-05T12:02:10+00:00

There were some curly braces intended here.

Signed-off-by: Dan Carpenter 
Signed-off-by: Takashi Iwai

ALSA: Deletion of checks before the function call "iounmap"

2015-01-04T14:13:45+00:00

The iounmap() function performs also input parameter validation.
Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring 
Signed-off-by: Takashi Iwai

ALSA: msnd: Remove always NULL parameter

2015-01-02T15:28:04+00:00

snd_msnd_pcm() takes a pointer to a pointer of a PCM where if this parameter
is provided the newly allocated PCM is stored. All callers pass NULL though,
so remove the parameter. This makes the code a bit cleaner and shorter.

Signed-off-by: Lars-Peter Clausen 
Signed-off-by: Takashi Iwai

ALSA: msnd: Use snd_ctl_enum_info()

2014-10-21T06:33:35+00:00

... and reduce the open codes.  Also add missing const to the text array.

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: msnd: Avoid duplicated driver name

2013-11-12T07:06:20+00:00

msnd_pinnacle.c is used for both snd-msnd-pinnacle and
snd-msnd-classic drivers, and both should have different driver
names.  Using the same driver name results in the sysfs warning for
duplicated entries like
 kobject: 'msnd-pinnacle.7' (cec33408): kobject_release, parent   (null) (delayed)
 kobject: 'msnd-pinnacle' (cecd4980): kobject_release, parent cf3ad9b0 (delayed)
 ------------[ cut here ]------------
 WARNING: CPU: 0 PID: 1 at fs/sysfs/dir.c:486 sysfs_warn_dup+0x7d/0xa0()
 sysfs: cannot create duplicate filename '/bus/isa/drivers/msnd-pinnacle'
 ......

Reported-by: Fengguang Wu 
Cc: 
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

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

sound: Add export.h for THIS_MODULE/EXPORT_SYMBOL where needed

2011-10-31T23:31:22+00:00

These aren't modules, but they do make use of these macros, so
they will need export.h to get that definition.  Previously,
they got it via the implicit module.h inclusion.

Signed-off-by: Paul Gortmaker