01 Mar, 2016
3 commits
-
snd-hdsp driver accesses enum item values (int) instead of boolean
values (long) wrongly for some ctl elements. This patch fixes them.Cc:
Signed-off-by: Takashi Iwai -
HDSPM driver contains a code issuing zero-division potentially in
system sample rate ctl code. This patch fixes it by not processing
a zero or invalid rate value as a divisor, as well as excluding the
invalid value to be passed via the given ctl element.Cc:
Signed-off-by: Takashi Iwai -
snd-hdspm driver accesses enum item values (int) instead of boolean
values (long) wrongly for some ctl elements. This patch fixes them.Cc:
Signed-off-by: Takashi Iwai
23 Oct, 2015
1 commit
21 Oct, 2015
1 commit
-
Use the new snd_pcm_hw_constraint_single() helper function instead of
calling snd_pcm_hw_constraint_minmax() with the same value for min and max
to install a constraint that limits the possible configuration values to a
single value. Using snd_pcm_hw_constraint_single() makes the indented
result clearer.Signed-off-by: Lars-Peter Clausen
Signed-off-by: Takashi Iwai
11 Sep, 2015
1 commit
-
If the size of the firmware is less than expected size then we are
exiting with the error code but we missed releasing the firmware.Signed-off-by: Sudip Mukherjee
Signed-off-by: Takashi Iwai
22 Aug, 2015
1 commit
-
card->shortname is a 32 char string so the sprintf() can theoretically
overflow. snd_rawmidi_new() can accept strings up to 64 bytes long.I have made the temporay buf[] array 40 bytes long and changed the
sprintf() to snprintf().Signed-off-by: Dan Carpenter
Signed-off-by: Takashi Iwai
21 Aug, 2015
1 commit
-
I believe this probably cannot happen, as the code suggests. There
would have to be an kcontrol->index.id which was zero, otherwise this
would be prevented in snd_ctl_find_id(). But snd_BUG_ON() is just a
WARN() or a no-op so static checkers complain that we keep on going with
a negative offset. Let's just handle the error as well as printing
a warning.Signed-off-by: Dan Carpenter
Signed-off-by: Takashi Iwai
17 Feb, 2015
3 commits
-
This commit removes code duplication between
snd_hdspm_{capture,playback}_ops. No semantic changes intended, this is
purely cosmetic.Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai -
This commit removes code duplication between
snd_hdspm_{capture,playback}_release. No semantic changes intended, this
is purely cosmetic.Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai -
This commit removes code duplication between
snd_hdspm_{capture,playback}_open. No semantic changes intended, this is
purely cosmetic.Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai
14 Feb, 2015
1 commit
-
RME RayDAT and AIO use a fixed buffer size of 16384 samples. With period
sizes of 32-4096, this translates to 4-512 periods.The older RME cards have a variable buffer size but require exactly two
periods.This patch enforces nperiods=2 on those cards.
Signed-off-by: Adrian Knoth
Cc: # 2.6.39+
Signed-off-by: Takashi Iwai
28 Jan, 2015
1 commit
-
Nowadays it's recommended. Replace all in a shot.
Signed-off-by: Takashi Iwai
19 Jan, 2015
2 commits
-
No functional change, refactoring with the standard helpers.
Signed-off-by: Takashi Iwai
-
No functional change, refactoring with the standard helpers.
Signed-off-by: Takashi Iwai
04 Jan, 2015
1 commit
-
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
17 Nov, 2014
1 commit
-
The release_firmware() function tests whether its argument is NULL and then
return immediately. 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
11 Nov, 2014
1 commit
-
They are no real kerneldoc comments, so drop such markers.
Signed-off-by: Takashi Iwai
29 Oct, 2014
1 commit
-
removed the unused variables. These variables were only being
assigned some value, but the values were never being used.it has been build tested after removing the variables.
Signed-off-by: Sudip Mukherjee
Signed-off-by: Takashi Iwai
21 Oct, 2014
3 commits
-
... and reduce the open codes. Also add missing const to text arrays.
Signed-off-by: Takashi Iwai
-
... and reduce the open codes.
Signed-off-by: Takashi Iwai
-
... and reduce the open codes. Also add missing const to text arrays.
Signed-off-by: Takashi Iwai
13 Aug, 2014
1 commit
-
We should prefer `struct pci_device_id` over `DEFINE_PCI_DEVICE_TABLE` to
meet kernel coding style guidelines. This issue was reported by checkpatch.A simplified version of the semantic patch that makes this change is as
follows (http://coccinelle.lip6.fr/)://
@@
identifier i;
declarer name DEFINE_PCI_DEVICE_TABLE;
initializer z;
@@- DEFINE_PCI_DEVICE_TABLE(i)
+ const struct pci_device_id i[]
= z;//
[bhelgaas: add semantic patch]
Signed-off-by: Benoit Taine
Signed-off-by: Bjorn Helgaas
26 Feb, 2014
3 commits
-
Convert with dev_err() and co from snd_printk(), etc.
Signed-off-by: Takashi Iwai
-
Convert with dev_err() and co from snd_printk(), etc.
Signed-off-by: Takashi Iwai
-
Convert with dev_err() and co from snd_printk(), etc.
Signed-off-by: Takashi Iwai
12 Feb, 2014
1 commit
-
Also remove superfluous snd_card_set_dev() calls.
Signed-off-by: Takashi Iwai
09 Jan, 2014
1 commit
-
Nowadays we have CMA for obtaining the contiguous memory pages
efficiently. Let's kill the old kludge for reserving the memory pages
for large buffers. It was rarely useful (only for preserving pages
among module reloading or a little help by an early boot scripting),
used only by a couple of drivers, and yet it gives too much ugliness
than its benefit.Signed-off-by: Takashi Iwai
05 Jan, 2014
1 commit
-
The lack of comma leads to the wrong channel for an SPDIF channel.
Unfortunately this wasn't caught by compiler because it's still a
valid expression.Reported-by: Alexander Aristov
Cc:
Signed-off-by: Takashi Iwai
29 Oct, 2013
1 commit
-
We tend to make stupid mistakes with strncpy(). Let's take a safer
one, strlcpy().Signed-off-by: Takashi Iwai
25 Oct, 2013
1 commit
-
ASoC: Updates for v3.13
- Further work on the dmaengine helpers, including support for
configuring the parameters for DMA by reading the capabilities of the
DMA controller which removes some guesswork and magic numbers fromm
drivers.
- A refresh of the documentation.
- Conversions of many drivers to direct regmap API usage in order to
allow the ASoC level register I/O code to be removed, this will
hopefully be completed by v3.14.
- Support for using async register I/O in DAPM, reducing the time taken
to implement power transitions on systems that support it.
16 Oct, 2013
1 commit
-
In GCC the sizeof(hdsp_version) is 8 because there is a 2 byte hole at
the end of the struct after ->firmware_rev.Signed-off-by: Dan Carpenter
Signed-off-by: Takashi Iwai
13 Sep, 2013
1 commit
-
'break' after return statement is not necessary.
Signed-off-by: Sachin Kamat
Signed-off-by: Takashi Iwai
20 Aug, 2013
2 commits
-
This patch doesn't change functionality, it only improves readability
and fixes a copy&paste error in a comment.Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai -
Use enum hdspm_ltc_format's fps_30 (corresponds to 4) instead of 30,
Other case branches return 1, 2 or 3 respectively, so 30 obviously is
wrong.Since SNDRV_HDSPM_IOCTL_GET_LTC had never been working due to a
copy&paste error in hdspm.h, this change doesn't break userspace.Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai
15 Jul, 2013
1 commit
-
sound/pci/rme9652/hdspm.c:1110:2-3: Unneeded semicolon
Generated by: coccinelle/misc/semicolon.cocci
Reported-by: Fengguang Wu
Signed-off-by: Takashi Iwai
05 Jul, 2013
4 commits
-
Signed-off-by: Martin Dausel
Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai -
Also use snd_ctl_enum_info() to fill the autosync text fields on AES32
and MADI cards (only users of snd_hdspm_info_autosync_ref).Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai -
Also use snd_ctl_enum_info() to fill the autosync enumerated controls.
Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai -
Use snd_ctl_enum_info() to fill most of the enumerated controls. More
non-trivial occurrences will follow in separate commits.Signed-off-by: Adrian Knoth
Signed-off-by: Takashi Iwai