Pierre-Louis Bossart
ff40e0d41a
ALSA: usb: update old-style static const declaration
...
GCC reports the following warning with W=1
sound/usb/mixer_quirks.c: In function ‘snd_microii_controls_create’:
sound/usb/mixer_quirks.c:1694:2: warning: ‘static’ is not at beginning
of declaration [-Wold-style-declaration]
1694 | const static usb_mixer_elem_resume_func_t resume_funcs[] = {
| ^~~~~
Move static to the beginning of declaration
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com >
Link: https://lore.kernel.org/r/20200111214736.3002-3-pierre-louis.bossart@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-12 09:45:17 +01:00
Pierre-Louis Bossart
8a71821f12
ALSA: hda: patch_realtek: fix empty macro usage in if block
...
GCC reports the following warning with W=1
sound/pci/hda/patch_realtek.c: In function ‘alc269_suspend’:
sound/pci/hda/patch_realtek.c:3616:29: warning: suggest braces around
empty body in an ‘if’ statement [-Wempty-body]
3616 | alc5505_dsp_suspend(codec);
| ^
sound/pci/hda/patch_realtek.c: In function ‘alc269_resume’:
sound/pci/hda/patch_realtek.c:3651:28: warning: suggest braces around empty body in an ‘if’ statement [-Wempty-body]
3651 | alc5505_dsp_resume(codec);
| ^
This is a classic macro problem and can indeed lead to bad program
flows.
Fix by using the usual "do { } while (0)" pattern
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com >
Link: https://lore.kernel.org/r/20200111214736.3002-2-pierre-louis.bossart@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-12 09:45:09 +01:00
Pierre-Louis Bossart
f999348021
ALSA: timer: fix nsec/sec initialization confusion
...
GCC reports a warning with W=1:
sound/core/timer.c: In function ‘snd_timer_user_read’:
sound/core/timer.c:2219:19: warning: initialized field overwritten
[-Woverride-init]
2219 | .tstamp_sec = tread->tstamp_nsec,
| ^~~~~
sound/core/timer.c:2219:19: note: (near initialization for
‘(anonymous).tstamp_sec’)
Assigning nsec values to sec fields is problematic in general, even
more so when the initial goal was to survive the 2030 timer
armageddon.
Fix by using the proper field in the initialization
Cc: Baolin Wang <baolin.wang@linaro.org >
Cc: Arnd Bergmann <arnd@arndb.de >
Fixes: 07094ae6f9 ("ALSA: Avoid using timespec for struct snd_timer_tread")
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com >
Acked-by: Arnd Bergmann <arnd@arndb.de >
Link: https://lore.kernel.org/r/20200111203325.20498-1-pierre-louis.bossart@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-12 09:08:22 +01:00
Takashi Iwai
3cdca6d62b
Merge branch 'for-linus' into for-next
...
Sync 5.5-devel branch once again for applying the HD-audio fixes.
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-11 09:03:40 +01:00
Takashi Iwai
7fba6aea44
ALSA: hda: Rename back to dmic_detect option
...
We've got quite a few bug reports showing the SOF driver being loaded
unintentionally recently, and the reason seems to be that users didn't
know the module option change: with the recent kernel, a new option
dsp_driver=1 has to be passed to a new module snd-intel-dspcfg
instead of snd_hda_intel.dmic_detect=0 option.
That is, actually there are two tricky things here:
- We changed the whole detection in another module and another
option semantics.
- The existing option for skipping the DSP probe was also renamed.
For avoiding the confusion and giving user more hint, this patch
reverts the renamed option dsp_driver back to dmic_detect for
snd-hda-intel module, and show the warning about the module option
change when the non-default value is passed.
Fixes: 82d9d54a6c ("ALSA: hda: add Intel DSP configuration / probe code")
Link: https://lore.kernel.org/r/20200109082000.26729-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-11 09:02:23 +01:00
Kai Vehmanen
8e85def572
ALSA: hda: enable regmap internal locking
...
This reverts commit 42ec336f1f ("ALSA: hda: Disable regmap
internal locking").
Without regmap locking, there is a race between snd_hda_codec_amp_init()
and PM callbacks issuing regcache_sync(). This was caught by
following kernel warning trace:
<4> [358.080081] WARNING: CPU: 2 PID: 4157 at drivers/base/regmap/regcache.c:498 regcache_cache_only+0xf5/0x130
[...]
<4> [358.080148] Call Trace:
<4> [358.080158] snd_hda_codec_amp_init+0x4e/0x100 [snd_hda_codec]
<4> [358.080169] snd_hda_codec_amp_init_stereo+0x40/0x80 [snd_hda_codec]
Suggested-by: Takashi Iwai <tiwai@suse.de >
BugLink: https://gitlab.freedesktop.org/drm/intel/issues/592
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com >
Link: https://lore.kernel.org/r/20200108180856.5194-1-kai.vehmanen@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-08 20:10:08 +01:00
Takashi Iwai
1a0986c1bc
ALSA: hda: Fix a typo in comments
...
Link: https://lore.kernel.org/r/20200108153430.31456-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-08 16:35:00 +01:00
YueHaibing
55c203a308
ALSA: pci: echoaudio: remove set but not used variable 'chip'
...
sound/pci/echoaudio/echoaudio.c: In function snd_echo_mixer_info:
sound/pci/echoaudio/echoaudio.c:1233:20: warning: variable chip set but not used [-Wunused-but-set-variable]
sound/pci/echoaudio/echoaudio.c: In function 'snd_echo_vmixer_info':
sound/pci/echoaudio/echoaudio.c:1300:20: warning: variable 'chip' set but not used [-Wunused-but-set-variable]
commit e67c3f0fd4 ("ALSA: pci: echoaudio: remove usage
of dimen menber of elem_value structure") left behind this
unused variable.
Reported-by: Hulk Robot <hulkci@huawei.com >
Signed-off-by: YueHaibing <yuehaibing@huawei.com >
Link: https://lore.kernel.org/r/20200108125803.45584-1-yuehaibing@huawei.com
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-08 15:07:24 +01:00
Kailang Yang
54a6a7dc10
ALSA: hda/realtek - Add quirk for the bass speaker on Lenovo Yoga X1 7th gen
...
Add quirk to ALC285_FIXUP_SPEAKER2_TO_DAC1, which is the same fixup
applied for X1 Carbon 7th gen in commit d2cd795c4e ("ALSA: hda -
fixup for the bass speaker on Lenovo Carbon X1 7th gen").
Signed-off-by: Kailang Yang <kailang@realtek.com >
Reviewed-by: Jaroslav Kysela <perex@perex.cz >
Cc: <stable@vger.kernel.org >
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-08 10:46:33 +01:00
Kailang Yang
9194a1ebbc
ALSA: hda/realtek - Set EAPD control to default for ALC222
...
Set EAPD control to verb control.
Signed-off-by: Kailang Yang <kailang@realtek.com >
Cc: <stable@vger.kernel.org >
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-08 09:48:36 +01:00
Dan Carpenter
3b2549a374
ALSA: control: potential uninitialized return value
...
Smatch complains that "ret" might be uninitialized.
Fixes: fbd3eb7f66 ("ALSA: control: Add verification for kctl accesses")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com >
Link: https://lore.kernel.org/r/20200108053706.h3hcnvmnf62wkjac@kili.mountain
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-08 07:20:28 +01:00
Takashi Iwai
fbca72c154
Merge tag 'asoc-fix-v5.5-rc5' of https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound into for-linus
...
ASoC: Fixes for v5.5
More fixes that have been collected, nothing super remarkable here - the
few core fixes are mainly error handling related as are many of the
driver fixes.
2020-01-06 17:39:15 +01:00
Takashi Iwai
f1dd4795b1
ALSA: sh: Fix compile warning wrt const
...
A long-standing compile warning was seen during build test:
sound/sh/aica.c: In function 'load_aica_firmware':
sound/sh/aica.c:521:25: warning: passing argument 2 of 'spu_memload' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
Fixes: 198de43d75 ("[ALSA] Add ALSA support for the SEGA Dreamcast PCM device")
Link: https://lore.kernel.org/r/20200105144823.29547-69-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:14 +01:00
Takashi Iwai
cabc046807
ALSA: spi: More constification
...
Apply const prefix to the static table for registers.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-68-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:13 +01:00
Takashi Iwai
6ab9eabe19
ALSA: pdaudiocf: More constification
...
Apply const prefix to the static table for verbs.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-67-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:13 +01:00
Takashi Iwai
a05c0737d3
ALSA: via82xx: More constification
...
Apply const prefix to the static table for the chip models.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-66-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:12 +01:00
Takashi Iwai
8045d0fc95
ALSA: fm801: More constification
...
Apply const prefix to the static register table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-65-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:12 +01:00
Takashi Iwai
7103e4a70c
ALSA: ens137x: More constification
...
Apply const prefix to the quirk entry, forgotten in the previous fix.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-64-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:11 +01:00
Takashi Iwai
9bca090793
ALSA: echoaudio: More constification
...
Apply const prefix to the static channel list table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-63-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:10 +01:00
Takashi Iwai
e157f0cd05
ALSA: cs4281: More constification
...
Apply const prefix to the static register table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-62-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:10 +01:00
Takashi Iwai
83fdb6fbcc
ALSA: azt3328: More constification
...
Apply const prefix to the static initial register table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-61-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:09 +01:00
Takashi Iwai
24ce8056a9
ALSA: ak4531: More constification
...
Apply const prefix to the initial register map.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-60-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:09 +01:00
Takashi Iwai
6c8454888f
ALSA: serial-u16550: More constification
...
Apply const prefix to the string array.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-59-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:08 +01:00
Takashi Iwai
1d99500a1d
ALSA: opl4: More constification
...
Apply const prefix to the static volume table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-58-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:08 +01:00
Takashi Iwai
7840d8a103
ALSA: arm: More constification
...
Apply const prefix to the static channel list table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-57-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:07 +01:00
Takashi Iwai
2f200ce291
ALSA: lx6464es: More constifications
...
Apply const prefix to the static tables for the DSP command and the
peak map.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-56-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:07 +01:00
Takashi Iwai
26170691ff
ALSA: aw2: More constifications
...
Apply const prefix to the static parameter tables.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-55-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:06 +01:00
Takashi Iwai
616986985d
ALSA: mts64: More constifications
...
Apply const prefix to the static mapping tables.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-54-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:06 +01:00
Takashi Iwai
8fc179162d
ALSA: sc6000: More constification
...
Apply const prefix to the static resource tables.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-53-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:05 +01:00
Takashi Iwai
429bca4d76
ALSA: wss: More constifications
...
Apply const prefix to the static tables for parameters.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-52-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:05 +01:00
Takashi Iwai
80e0a7c095
ALSA: opti9xx: More constifications
...
Apply const prefix to the static resource tables, the mc size tables
and the string arrays.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-51-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:04 +01:00
Takashi Iwai
15a1af95d6
ALSA: gus: More constifications
...
Apply const prefix to each possible place: the static resource tables,
the volume table, etc.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-50-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:03 +01:00
Takashi Iwai
fce6709730
ALSA: es18xx: More constifications
...
Apply const prefix to the static map tables and the static resource
tables.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-49-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:03 +01:00
Takashi Iwai
748f518121
ALSA: es1688: More constifications
...
Apply const prefix to the static resource tables and init tables.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-48-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:02 +01:00
Takashi Iwai
350f613f37
ALSA: cmi8330: More constifications
...
Apply const prefix to the static tables for the register, controls and
callbacks.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-47-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:01 +01:00
Takashi Iwai
2a076d0af4
ALSA: sb: More constifications
...
Apply const prefix to each possible place: the static tables for the
resources, controls, registers, values and parameters.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-46-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:00 +01:00
Takashi Iwai
58b5b36370
ALSA: cs423x: More constification
...
Apply const prefix to the static mapping table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-45-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:15:00 +01:00
Takashi Iwai
f3c0916946
ALSA: cmi8328: More constifications
...
Apply const prefix to the static resource tables.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-44-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:59 +01:00
Takashi Iwai
e3de2a406e
ALSA: opl3: More constifications
...
Apply const prefix to the static tables for drum, volume and notes.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-43-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:59 +01:00
Takashi Iwai
edb87ed5f9
ALSA: rawmidi: More constification
...
Apply const prefix to the static flag info table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-42-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:58 +01:00
Takashi Iwai
b40fe4bbc7
ALSA: oss: More constifications
...
Apply const prefix to the static tables for strings and formats.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-41-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:58 +01:00
Takashi Iwai
f98903fce9
ALSA: jack: More constification
...
Apply const prefix to the static jack switch table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-40-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:57 +01:00
Takashi Iwai
51d7847a2d
ALSA: info: More constifications
...
Apply const prefix to the string array and its callers.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-39-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:57 +01:00
Takashi Iwai
121f46be2c
ALSA: sparc: More constifications
...
Apply const prefix to the static tables for rates, bits and strings.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-38-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:56 +01:00
Takashi Iwai
5f3aca1065
ALSA: cmipci: More constifications
...
Apply const prefix to the static tables for rates, ports and
registers.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-37-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:55 +01:00
Takashi Iwai
096da809f6
ALSA: es1968: More constifications
...
Apply const prefix to the quirk white/black lists.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-36-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:55 +01:00
Takashi Iwai
96d5ebf25e
ALSA: es1938: More constifications
...
Apply const prefix to the static register table and its callers.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-35-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:54 +01:00
Takashi Iwai
066c044b05
ALSA: atiixp: More constifications
...
Apply const prefix to the static register tables.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-34-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:54 +01:00
Takashi Iwai
9ca7a0c979
ALSA: asihpi: More constifications
...
Apply const prefix to each possible place: the static tables for
formats, parameters, etc.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-33-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:53 +01:00
Takashi Iwai
6e0e75d94e
ALSA: ctxfi: More constifications
...
Apply const prefix to each possible place: the DAIO tables and the
register offset table.
Just for minor optimization and no functional changes.
Link: https://lore.kernel.org/r/20200105144823.29547-32-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de >
2020-01-05 16:14:52 +01:00