ALSA: ctl: allow TLV read operation for callback type of element in locked case
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Mon, 23 Dec 2019 09:33:47 +0000 (18:33 +0900)
committerTakashi Iwai <tiwai@suse.de>
Tue, 24 Dec 2019 14:29:12 +0000 (15:29 +0100)
commitd61fe22c2ae42d9fd76c34ef4224064cca4b04b0
tree022701192179027eedcffa1f605bc7baf124790f
parentbd3eb4e87eb399a9fe15ef1b59db78faf9c20359
ALSA: ctl: allow TLV read operation for callback type of element in locked case

A design of ALSA control core allows applications to execute three
operations for TLV feature; read, write and command. Furthermore, it
allows driver developers to process the operations by two ways; allocated
array or callback function. In the former, read operation is just allowed,
thus developers uses the latter when device driver supports variety of
models or the target model is expected to dynamically change information
stored in TLV container.

The core also allows applications to lock any element so that the other
applications can't perform write operation to the element for element
value and TLV information. When the element is locked, write and command
operation for TLV information are prohibited as well as element value.
Any read operation should be allowed in the case.

At present, when an element has callback function for TLV information,
TLV read operation returns EPERM if the element is locked. On the
other hand, the read operation is success when an element has allocated
array for TLV information. In both cases, read operation is success for
element value expectedly.

This commit fixes the bug. This change can be backported to v4.14
kernel or later.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Reviewed-by: Jaroslav Kysela <perex@perex.cz>
Link: https://lore.kernel.org/r/20191223093347.15279-1-o-takashi@sakamocchi.jp
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/core/control.c