August 2024 - Alsa-devel - mailman.alsa-project.org

Use PyBytes and not Unicode strings for IEC958 values
by GitHub pull_request - opened 16 Aug '24

16 Aug '24

alsa-project/alsa-python pull request #14 was opened from tormodvolden: Suggested fix for #13. - the first commit is not strictly related but also needed to avoid hctltest1.py crashing with my audio devices, where some have info.count = 0. Let me know if more information is needed. - I went for PyBytes instead of PyByteArray - for setting values, if the provided PyBytes value is shorter than the value, it will be zero-padded. - if it is too long, it will be silently truncated - following the existing code style, no empty lines were inserted, but this could help readability Request URL : https://github.com/alsa-project/alsa-python/pull/14 Patch URL : https://github.com/alsa-project/alsa-python/pull/14.patch Repository URL: https://github.com/alsa-project/alsa-python

1 0

[PATCH] ASoC: codecs: lpass-va-macro: set the default codec version for sm8250
by srinivas.kandagatla＠linaro.org 16 Aug '24

16 Aug '24

From: Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> sm8250 and sc7280 have lpass codec version 1.0, as these are very old platforms, they do not have a reliable way to get the codec version from core_id registers. Add the version info into of_data, so that it does not need to use core_id registers to get version number. Fixes: 378918d59181 ("ASoC: codecs: lpass-macro: add helpers to get codec version") Fixes: dbacef05898d ("ASoC: codec: lpass-rx-macro: prepare driver to accomdate new codec versions") Fixes: 727de4fbc546 ("ASoC: codecs: lpass-wsa-macro: Correct support for newer v2.5 version") Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> --- sound/soc/codecs/lpass-va-macro.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/sound/soc/codecs/lpass-va-macro.c b/sound/soc/codecs/lpass-va-macro.c index 8454193ed22a..e95d1f29ef18 100644 --- a/sound/soc/codecs/lpass-va-macro.c +++ b/sound/soc/codecs/lpass-va-macro.c @@ -228,11 +228,13 @@ struct va_macro { struct va_macro_data { bool has_swr_master; bool has_npl_clk; + int version; }; static const struct va_macro_data sm8250_va_data = { .has_swr_master = false, .has_npl_clk = false, + .version = LPASS_CODEC_VERSION_1_0, }; static const struct va_macro_data sm8450_va_data = { @@ -1587,7 +1589,14 @@ static int va_macro_probe(struct platform_device *pdev) goto err_npl; } - va_macro_set_lpass_codec_version(va); + /** + * old version of codecs do not have a reliable way to determine the + * version from registers, get them from soc specific data + */ + if (data->version) + lpass_macro_set_codec_version(data->version); + else /* read version from register */ + va_macro_set_lpass_codec_version(va); if (va->has_swr_master) { /* Set default CLK div to 1 */ -- 2.25.1

3 4

[PATCH v1] ASoC: codecs: ES8326: Adjust buttons
by zhangyi＠everest-semi.com 16 Aug '24

16 Aug '24

The content of this message was lost. It was probably cross-posted to multiple lists and previously handled on another list.

2 1

[PATCH] ALSA: aoa: Use helper function for_each_child_of_node()
by Zhang Zekun 16 Aug '24

16 Aug '24

for_each_child_of_node can help to iterate through the device_node, and we don't need to use while loop. No functional change with this conversion. Signed-off-by: Zhang Zekun <zhangzekun11(a)huawei.com> --- sound/aoa/soundbus/i2sbus/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/aoa/soundbus/i2sbus/core.c b/sound/aoa/soundbus/i2sbus/core.c index 5431d2c49421..ce84288168e4 100644 --- a/sound/aoa/soundbus/i2sbus/core.c +++ b/sound/aoa/soundbus/i2sbus/core.c @@ -335,7 +335,7 @@ static int i2sbus_add_dev(struct macio_dev *macio, static int i2sbus_probe(struct macio_dev* dev, const struct of_device_id *match) { - struct device_node *np = NULL; + struct device_node *np; int got = 0, err; struct i2sbus_control *control = NULL; @@ -347,7 +347,7 @@ static int i2sbus_probe(struct macio_dev* dev, const struct of_device_id *match) return -ENODEV; } - while ((np = of_get_next_child(dev->ofdev.dev.of_node, np))) { + for_each_child_of_node(dev->ofdev.dev.of_node, np) { if (of_device_is_compatible(np, "i2sbus") || of_device_is_compatible(np, "i2s-modem")) { got += i2sbus_add_dev(dev, control, np); -- 2.17.1

1 0

[PATCH v2 0/6] ASoC: codecs: wsa88xx: add support for static port mapping.
by srinivas.kandagatla＠linaro.org 15 Aug '24

15 Aug '24

Existing way of allocating soundwire master ports on Qualcommm platforms is dynamic, and in linear order starting from 1 to MAX_PORTS. This will work as long as soundwire device ports are 1:1 mapped linearly. However on most Qcom SoCs like SM8550, SM8650, x1e80100, these are NOT mapped in that order. The result of this is that only one speaker among the pair of speakers is always silent, With recent changes for WSA codec to support codec versions and along with these patches we are able to get all speakers working on these SoCs. Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> --- Changes in v2: - used dev_dbg instead of dev_info - Link to v1: https://lore.kernel.org/r/20240626-port-map-v1-0-bd8987d2b332@linaro.org --- Manikantan R (1): ASoC: dt-bindings: wsa883x: Document port mapping property Srinivas Kandagatla (5): ASoC: codecs: wsa883x: parse port-mapping information ASoC: dt-bindings: wsa8840: Document port mapping property ASoC: codecs: wsa884x: parse port-mapping information arm64: dts: x1e80100-crd: fix wsa soundwire port mapping arm64: dts: x1e80100-qcp: fix wsa soundwire port mapping Documentation/devicetree/bindings/sound/qcom,wsa883x.yaml | 8 ++++++++ Documentation/devicetree/bindings/sound/qcom,wsa8840.yaml | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e80100-crd.dts | 4 ++++ arch/arm64/boot/dts/qcom/x1e80100-qcp.dts | 2 ++ sound/soc/codecs/wsa883x.c | 8 ++++++++ sound/soc/codecs/wsa884x.c | 8 ++++++++ 6 files changed, 38 insertions(+) --- base-commit: 9935be184a55dd84fc3275094f2df095491f6ea1 change-id: 20240626-port-map-ef50c3304d4a Best regards, -- Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org>

5 10

[PATCH 0/6] soc: qcom: use scoped device node handling to simplify error paths
by Krzysztof Kozlowski 15 Aug '24

15 Aug '24

Use cleanup.h/scope on device nodes to get rid of of_node_put(). Code is a bit simpler. Best regards, Krzysztof --- Krzysztof Kozlowski (6): soc: qcom: apr: simplify with scoped for each OF child loop soc: qcom: aoss: simplify with scoped for each OF child loop soc: qcom: ice: use scoped device node handling to simplify error paths soc: qcom: ocmem: use scoped device node handling to simplify error paths soc: qcom: pbs: use scoped device node handling to simplify error paths soc: qcom: smp2p: use scoped device node handling to simplify error paths drivers/soc/qcom/apr.c | 5 +---- drivers/soc/qcom/ice.c | 14 +++++--------- drivers/soc/qcom/ocmem.c | 7 ++----- drivers/soc/qcom/qcom-pbs.c | 16 ++++++---------- drivers/soc/qcom/qcom_aoss.c | 8 +++----- drivers/soc/qcom/smp2p.c | 16 ++++------------ 6 files changed, 21 insertions(+), 45 deletions(-) --- base-commit: d54f33dafd9e93367ffaf95318ed371138a6c8c0 change-id: 20240813-b4-cleanup-h-of-node-put-other-7e9c7128abe4 Best regards, -- Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org>

2 7

hctltest1.py crashes with utf-8 decoding error on IEC958 "status"
by GitHub issues - edited 15 Aug '24

15 Aug '24

alsa-project/alsa-python issue #13 was edited from tormodvolden: Version 1.2.12, on Ubuntu 24.04, Python 3.12.3. It seems like the "status" member returned by `snd_ctl_elem_value_get_iec958(self->value, iec958);` on pyalsa/alsahcontrol.c line 1053 starts with 0xff then a 0x00. I haven't been able to find a corresponding node under /proc/asound to check this. The relevant aplay -L entry is ``` iec958:CARD=Stereo,DEV=0 Logitech H570e Stereo, USB Audio IEC958 (S/PDIF) Digital Audio Output ``` Here is a dump of /proc/asound/card1 (card 1 is called "Stereo") <details> ``` $ more $(find /proc/asound/card1 -type f) :::::::::::::: /proc/asound/card1/id :::::::::::::: Stereo :::::::::::::: /proc/asound/card1/pcm0c/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: CAPTURE id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 1 :::::::::::::: /proc/asound/card1/pcm0c/sub0/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: CAPTURE id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 1 :::::::::::::: /proc/asound/card1/pcm0c/sub0/status :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0c/sub0/hw_params :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0c/sub0/sw_params :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0p/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: PLAYBACK id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 0 :::::::::::::: /proc/asound/card1/pcm0p/sub0/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: PLAYBACK id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 0 :::::::::::::: /proc/asound/card1/pcm0p/sub0/status :::::::::::::: state: RUNNING owner_pid : 1928 trigger_time: 270044.979966617 tstamp : 309371.219688371 delay : 1872 avail : 31520 avail_max : 32032 ----- hw_ptr : 1887652896 appl_ptr : 1887654144 :::::::::::::: /proc/asound/card1/pcm0p/sub0/hw_params :::::::::::::: access: MMAP_INTERLEAVED format: S16_LE subformat: STD channels: 2 rate: 48000 (48000/1) period_size: 256 buffer_size: 32768 :::::::::::::: /proc/asound/card1/pcm0p/sub0/sw_params :::::::::::::: tstamp_mode: ENABLE period_step: 1 avail_min: 256 start_threshold: 9223372036854775807 stop_threshold: 32768 silence_threshold: 0 silence_size: 0 boundary: 4611686018427387904 :::::::::::::: /proc/asound/card1/usbid :::::::::::::: 046d:0a56 :::::::::::::: /proc/asound/card1/usbbus :::::::::::::: 003/006 :::::::::::::: /proc/asound/card1/stream0 :::::::::::::: Logitech Inc Logitech H570e Stereo at usb-0000:00:14.0-4.4.2, full speed : USB A udio Playback: Status: Running Interface = 2 Altset = 1 Packet Size = 192 Momentary freq = 48000 Hz (0x30.0000) Interface 2 Altset 1 Format: S16_LE Channels: 2 Endpoint: 0x01 (1 OUT) (ADAPTIVE) Rates: 8000, 16000, 24000, 32000, 44100, 48000 Bits: 16 Channel map: FL FR Capture: Status: Stop Interface 1 Altset 1 Format: S16_LE Channels: 2 Endpoint: 0x81 (1 IN) (ASYNC) Rates: 8000, 16000, 24000, 32000, 44100, 48000 Bits: 16 Channel map: FL FR :::::::::::::: /proc/asound/card1/usbmixer :::::::::::::: USB Mixer: usb_id=0x046d0a56, ctrlif=0, ctlerr=0 Card: Logitech Inc Logitech H570e Stereo at usb-0000:00:14.0-4.4.2, full speed Unit: 2 Control: name="Auto Gain Control", index=0 Info: id=2, control=7, cmask=0x0, channels=1, type="BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 2 Control: name="Headset Capture Volume", index=0 Info: id=2, control=2, cmask=0x0, channels=1, type="S16" Volume: min=-7680, max=1280, dBmin=-3000, dBmax=500 Unit: 2 Control: name="Headset Capture Switch", index=0 Info: id=2, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 5 Control: name="Headset Playback Volume", index=0 Info: id=5, control=2, cmask=0x3, channels=2, type="S16" Volume: min=-15360, max=1280, dBmin=-6000, dBmax=500 Unit: 5 Control: name="Headset Playback Switch", index=0 Info: id=5, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 11 Control: name="Sidetone Playback Volume", index=0 Info: id=11, control=2, cmask=0x0, channels=1, type="S16" Volume: min=0, max=4125, dBmin=0, dBmax=1611 Unit: 11 Control: name="Sidetone Playback Switch", index=0 Info: id=11, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 ``` </details> This is the last part of the output from running `python3 test/hctltest1.py ` ``` ***** count: 1 device: 0 dimensions: None element: <alsahcontrol.Element object at 0x7e418ebb9650> id: (19, 2, 0, 0, 'IEC958 Playback Con Mask', 0) index: 0 interface: 2 is_inactive: False is_locked: False is_owner: False is_readable: True is_tlv_Commandable: False is_tlv_Writable: False is_tlv_readable: False is_user: False is_volatile: False is_writable: False name: IEC958 Playback Con Mask numid: 19 owner: -1 subdevice: 0 type: 5 (IEC958) ----- device: 0 element: <alsahcontrol.Element object at 0x7e418ebb9650> get_array: <built-in method get_array of alsahcontrol.Value object at 0x7e418ebb9620> get_tuple: <built-in method get_tuple of alsahcontrol.Value object at 0x7e418ebb9620> id: (19, 2, 0, 0, 'IEC958 Playback Con Mask', 0) index: 0 interface: 2 name: IEC958 Playback Con Mask numid: 19 read: <built-in method read of alsahcontrol.Value object at 0x7e418ebb9620> set_array: <built-in method set_array of alsahcontrol.Value object at 0x7e418ebb9620> set_tuple: <built-in method set_tuple of alsahcontrol.Value object at 0x7e418ebb9620> subdevice: 0 write: <built-in method write of alsahcontrol.Value object at 0x7e418ebb9620> UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 1: invalid start byte The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/home/tormod/SOURCES/pyalsa-debian/python-pyalsa/test/hctltest1.py", line 69, in <module> value(element1) File "/home/tormod/SOURCES/pyalsa-debian/python-pyalsa/test/hctltest1.py", line 36, in value values = value.get_tuple(info.type, info.count) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ SystemError: <method 'get_tuple' of 'alsahcontrol.Value' objects> returned a result with an exception set ``` info.type is 5 info.count is 1 Issue URL : https://github.com/alsa-project/alsa-python/issues/13 Repository URL: https://github.com/alsa-project/alsa-python

1 0

hctltest1.py crashes with utf-8 decoding error on IEC958 "status"
by GitHub issues - edited 15 Aug '24

15 Aug '24

alsa-project/alsa-python issue #13 was edited from tormodvolden: Version 1.2.12, on Ubuntu 24.04, Python 3.12. It seems like the "status" member returned by `snd_ctl_elem_value_get_iec958(self->value, iec958);` on pyalsa/alsahcontrol.c line 1053 starts with 0xff then a 0x00. I haven't been able to find a corresponding node under /proc/asound to check this. The relevant aplay -L entry is ``` iec958:CARD=Stereo,DEV=0 Logitech H570e Stereo, USB Audio IEC958 (S/PDIF) Digital Audio Output ``` Here is a dump of /proc/asound/card1 (card 1 is called "Stereo") <details> ``` $ more $(find /proc/asound/card1 -type f) :::::::::::::: /proc/asound/card1/id :::::::::::::: Stereo :::::::::::::: /proc/asound/card1/pcm0c/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: CAPTURE id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 1 :::::::::::::: /proc/asound/card1/pcm0c/sub0/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: CAPTURE id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 1 :::::::::::::: /proc/asound/card1/pcm0c/sub0/status :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0c/sub0/hw_params :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0c/sub0/sw_params :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0p/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: PLAYBACK id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 0 :::::::::::::: /proc/asound/card1/pcm0p/sub0/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: PLAYBACK id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 0 :::::::::::::: /proc/asound/card1/pcm0p/sub0/status :::::::::::::: state: RUNNING owner_pid : 1928 trigger_time: 270044.979966617 tstamp : 309371.219688371 delay : 1872 avail : 31520 avail_max : 32032 ----- hw_ptr : 1887652896 appl_ptr : 1887654144 :::::::::::::: /proc/asound/card1/pcm0p/sub0/hw_params :::::::::::::: access: MMAP_INTERLEAVED format: S16_LE subformat: STD channels: 2 rate: 48000 (48000/1) period_size: 256 buffer_size: 32768 :::::::::::::: /proc/asound/card1/pcm0p/sub0/sw_params :::::::::::::: tstamp_mode: ENABLE period_step: 1 avail_min: 256 start_threshold: 9223372036854775807 stop_threshold: 32768 silence_threshold: 0 silence_size: 0 boundary: 4611686018427387904 :::::::::::::: /proc/asound/card1/usbid :::::::::::::: 046d:0a56 :::::::::::::: /proc/asound/card1/usbbus :::::::::::::: 003/006 :::::::::::::: /proc/asound/card1/stream0 :::::::::::::: Logitech Inc Logitech H570e Stereo at usb-0000:00:14.0-4.4.2, full speed : USB A udio Playback: Status: Running Interface = 2 Altset = 1 Packet Size = 192 Momentary freq = 48000 Hz (0x30.0000) Interface 2 Altset 1 Format: S16_LE Channels: 2 Endpoint: 0x01 (1 OUT) (ADAPTIVE) Rates: 8000, 16000, 24000, 32000, 44100, 48000 Bits: 16 Channel map: FL FR Capture: Status: Stop Interface 1 Altset 1 Format: S16_LE Channels: 2 Endpoint: 0x81 (1 IN) (ASYNC) Rates: 8000, 16000, 24000, 32000, 44100, 48000 Bits: 16 Channel map: FL FR :::::::::::::: /proc/asound/card1/usbmixer :::::::::::::: USB Mixer: usb_id=0x046d0a56, ctrlif=0, ctlerr=0 Card: Logitech Inc Logitech H570e Stereo at usb-0000:00:14.0-4.4.2, full speed Unit: 2 Control: name="Auto Gain Control", index=0 Info: id=2, control=7, cmask=0x0, channels=1, type="BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 2 Control: name="Headset Capture Volume", index=0 Info: id=2, control=2, cmask=0x0, channels=1, type="S16" Volume: min=-7680, max=1280, dBmin=-3000, dBmax=500 Unit: 2 Control: name="Headset Capture Switch", index=0 Info: id=2, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 5 Control: name="Headset Playback Volume", index=0 Info: id=5, control=2, cmask=0x3, channels=2, type="S16" Volume: min=-15360, max=1280, dBmin=-6000, dBmax=500 Unit: 5 Control: name="Headset Playback Switch", index=0 Info: id=5, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 11 Control: name="Sidetone Playback Volume", index=0 Info: id=11, control=2, cmask=0x0, channels=1, type="S16" Volume: min=0, max=4125, dBmin=0, dBmax=1611 Unit: 11 Control: name="Sidetone Playback Switch", index=0 Info: id=11, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 ``` </details> This is the last part of the output from running `python3 test/hctltest1.py ` ``` ***** count: 1 device: 0 dimensions: None element: <alsahcontrol.Element object at 0x7e418ebb9650> id: (19, 2, 0, 0, 'IEC958 Playback Con Mask', 0) index: 0 interface: 2 is_inactive: False is_locked: False is_owner: False is_readable: True is_tlv_Commandable: False is_tlv_Writable: False is_tlv_readable: False is_user: False is_volatile: False is_writable: False name: IEC958 Playback Con Mask numid: 19 owner: -1 subdevice: 0 type: 5 (IEC958) ----- device: 0 element: <alsahcontrol.Element object at 0x7e418ebb9650> get_array: <built-in method get_array of alsahcontrol.Value object at 0x7e418ebb9620> get_tuple: <built-in method get_tuple of alsahcontrol.Value object at 0x7e418ebb9620> id: (19, 2, 0, 0, 'IEC958 Playback Con Mask', 0) index: 0 interface: 2 name: IEC958 Playback Con Mask numid: 19 read: <built-in method read of alsahcontrol.Value object at 0x7e418ebb9620> set_array: <built-in method set_array of alsahcontrol.Value object at 0x7e418ebb9620> set_tuple: <built-in method set_tuple of alsahcontrol.Value object at 0x7e418ebb9620> subdevice: 0 write: <built-in method write of alsahcontrol.Value object at 0x7e418ebb9620> UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 1: invalid start byte The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/home/tormod/SOURCES/pyalsa-debian/python-pyalsa/test/hctltest1.py", line 69, in <module> value(element1) File "/home/tormod/SOURCES/pyalsa-debian/python-pyalsa/test/hctltest1.py", line 36, in value values = value.get_tuple(info.type, info.count) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ SystemError: <method 'get_tuple' of 'alsahcontrol.Value' objects> returned a result with an exception set ``` info.type is 5 info.count is 1 Issue URL : https://github.com/alsa-project/alsa-python/issues/13 Repository URL: https://github.com/alsa-project/alsa-python

1 0

hctltest1.py crashes with utf-8 decoding error on IEC958 "status"
by GitHub issues - opened 15 Aug '24

15 Aug '24

alsa-project/alsa-python issue #13 was opened from tormodvolden: It seems like the "status" member returned by `snd_ctl_elem_value_get_iec958(self->value, iec958);` on pyalsa/alsahcontrol.c line 1053 starts with 0xff then a 0x00. I haven't been able to find a corresponding node under /proc/asound to check this. The relevant aplay -L entry is ``` iec958:CARD=Stereo,DEV=0 Logitech H570e Stereo, USB Audio IEC958 (S/PDIF) Digital Audio Output ``` Here is a dump of /proc/asound/card1 (card 1 is called "Stereo") <details> ``` $ more $(find /proc/asound/card1 -type f) :::::::::::::: /proc/asound/card1/id :::::::::::::: Stereo :::::::::::::: /proc/asound/card1/pcm0c/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: CAPTURE id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 1 :::::::::::::: /proc/asound/card1/pcm0c/sub0/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: CAPTURE id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 1 :::::::::::::: /proc/asound/card1/pcm0c/sub0/status :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0c/sub0/hw_params :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0c/sub0/sw_params :::::::::::::: closed :::::::::::::: /proc/asound/card1/pcm0p/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: PLAYBACK id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 0 :::::::::::::: /proc/asound/card1/pcm0p/sub0/info :::::::::::::: card: 1 device: 0 subdevice: 0 stream: PLAYBACK id: USB Audio name: USB Audio subname: subdevice #0 class: 0 subclass: 0 subdevices_count: 1 subdevices_avail: 0 :::::::::::::: /proc/asound/card1/pcm0p/sub0/status :::::::::::::: state: RUNNING owner_pid : 1928 trigger_time: 270044.979966617 tstamp : 309371.219688371 delay : 1872 avail : 31520 avail_max : 32032 ----- hw_ptr : 1887652896 appl_ptr : 1887654144 :::::::::::::: /proc/asound/card1/pcm0p/sub0/hw_params :::::::::::::: access: MMAP_INTERLEAVED format: S16_LE subformat: STD channels: 2 rate: 48000 (48000/1) period_size: 256 buffer_size: 32768 :::::::::::::: /proc/asound/card1/pcm0p/sub0/sw_params :::::::::::::: tstamp_mode: ENABLE period_step: 1 avail_min: 256 start_threshold: 9223372036854775807 stop_threshold: 32768 silence_threshold: 0 silence_size: 0 boundary: 4611686018427387904 :::::::::::::: /proc/asound/card1/usbid :::::::::::::: 046d:0a56 :::::::::::::: /proc/asound/card1/usbbus :::::::::::::: 003/006 :::::::::::::: /proc/asound/card1/stream0 :::::::::::::: Logitech Inc Logitech H570e Stereo at usb-0000:00:14.0-4.4.2, full speed : USB A udio Playback: Status: Running Interface = 2 Altset = 1 Packet Size = 192 Momentary freq = 48000 Hz (0x30.0000) Interface 2 Altset 1 Format: S16_LE Channels: 2 Endpoint: 0x01 (1 OUT) (ADAPTIVE) Rates: 8000, 16000, 24000, 32000, 44100, 48000 Bits: 16 Channel map: FL FR Capture: Status: Stop Interface 1 Altset 1 Format: S16_LE Channels: 2 Endpoint: 0x81 (1 IN) (ASYNC) Rates: 8000, 16000, 24000, 32000, 44100, 48000 Bits: 16 Channel map: FL FR :::::::::::::: /proc/asound/card1/usbmixer :::::::::::::: USB Mixer: usb_id=0x046d0a56, ctrlif=0, ctlerr=0 Card: Logitech Inc Logitech H570e Stereo at usb-0000:00:14.0-4.4.2, full speed Unit: 2 Control: name="Auto Gain Control", index=0 Info: id=2, control=7, cmask=0x0, channels=1, type="BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 2 Control: name="Headset Capture Volume", index=0 Info: id=2, control=2, cmask=0x0, channels=1, type="S16" Volume: min=-7680, max=1280, dBmin=-3000, dBmax=500 Unit: 2 Control: name="Headset Capture Switch", index=0 Info: id=2, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 5 Control: name="Headset Playback Volume", index=0 Info: id=5, control=2, cmask=0x3, channels=2, type="S16" Volume: min=-15360, max=1280, dBmin=-6000, dBmax=500 Unit: 5 Control: name="Headset Playback Switch", index=0 Info: id=5, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 Unit: 11 Control: name="Sidetone Playback Volume", index=0 Info: id=11, control=2, cmask=0x0, channels=1, type="S16" Volume: min=0, max=4125, dBmin=0, dBmax=1611 Unit: 11 Control: name="Sidetone Playback Switch", index=0 Info: id=11, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN" Volume: min=0, max=1, dBmin=0, dBmax=0 ``` </details> This is the last part of the output from running `python3 test/hctltest1.py ` ``` ***** count: 1 device: 0 dimensions: None element: <alsahcontrol.Element object at 0x7e418ebb9650> id: (19, 2, 0, 0, 'IEC958 Playback Con Mask', 0) index: 0 interface: 2 is_inactive: False is_locked: False is_owner: False is_readable: True is_tlv_Commandable: False is_tlv_Writable: False is_tlv_readable: False is_user: False is_volatile: False is_writable: False name: IEC958 Playback Con Mask numid: 19 owner: -1 subdevice: 0 type: 5 (IEC958) ----- device: 0 element: <alsahcontrol.Element object at 0x7e418ebb9650> get_array: <built-in method get_array of alsahcontrol.Value object at 0x7e418ebb9620> get_tuple: <built-in method get_tuple of alsahcontrol.Value object at 0x7e418ebb9620> id: (19, 2, 0, 0, 'IEC958 Playback Con Mask', 0) index: 0 interface: 2 name: IEC958 Playback Con Mask numid: 19 read: <built-in method read of alsahcontrol.Value object at 0x7e418ebb9620> set_array: <built-in method set_array of alsahcontrol.Value object at 0x7e418ebb9620> set_tuple: <built-in method set_tuple of alsahcontrol.Value object at 0x7e418ebb9620> subdevice: 0 write: <built-in method write of alsahcontrol.Value object at 0x7e418ebb9620> UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 1: invalid start byte The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/home/tormod/SOURCES/pyalsa-debian/python-pyalsa/test/hctltest1.py", line 69, in <module> value(element1) File "/home/tormod/SOURCES/pyalsa-debian/python-pyalsa/test/hctltest1.py", line 36, in value values = value.get_tuple(info.type, info.count) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ SystemError: <method 'get_tuple' of 'alsahcontrol.Value' objects> returned a result with an exception set ``` info.type is 5 info.count is 1 Issue URL : https://github.com/alsa-project/alsa-python/issues/13 Repository URL: https://github.com/alsa-project/alsa-python

1 0

[PATCH] ASoc: tas2781: Remove unnecessary line feed and space
by Shenghao Ding 15 Aug '24

15 Aug '24

Remove unnecessary line feed for tasdevice_dsp_create_ctrls, and remove two unnecessary spaces in tas2563_digital_gain_get and tas2563_digital_gain_put. Signed-off-by: Shenghao Ding <shenghao-ding(a)ti.com> --- sound/soc/codecs/tas2781-i2c.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/sound/soc/codecs/tas2781-i2c.c b/sound/soc/codecs/tas2781-i2c.c index 8c9efd9183ff..9a32e0504857 100644 --- a/sound/soc/codecs/tas2781-i2c.c +++ b/sound/soc/codecs/tas2781-i2c.c @@ -157,7 +157,7 @@ static int tas2563_digital_gain_get( mutex_lock(&tas_dev->codec_lock); /* Read the primary device */ - ret = tasdevice_dev_bulk_read(tas_dev, 0, reg, data, 4); + ret = tasdevice_dev_bulk_read(tas_dev, 0, reg, data, 4); if (ret) { dev_err(tas_dev->dev, "%s, get AMP vol error\n", __func__); goto out; @@ -203,7 +203,7 @@ static int tas2563_digital_gain_put( vol = clamp(vol, 0, max); mutex_lock(&tas_dev->codec_lock); /* Read the primary device */ - ret = tasdevice_dev_bulk_read(tas_dev, 0, reg, data, 4); + ret = tasdevice_dev_bulk_read(tas_dev, 0, reg, data, 4); if (ret) { dev_err(tas_dev->dev, "%s, get AMP vol error\n", __func__); rc = -1; @@ -423,8 +423,7 @@ static int tasdevice_configuration_put( return ret; } -static int tasdevice_dsp_create_ctrls( - struct tasdevice_priv *tas_priv) +static int tasdevice_dsp_create_ctrls(struct tasdevice_priv *tas_priv) { struct snd_kcontrol_new *dsp_ctrls; char *prog_name, *conf_name; -- 2.34.1

2 1