February 2023 - Alsa-devel - mailman.alsa-project.org

Re: Firewire Phase24 Motu help
by Takashi Sakamoto 07 Feb '23

07 Feb '23

Hi, (C.C.ed to administrator of alsa-devel mailing list) On Mon, Jan 30, 2023 at 12:40:06PM +0100, M. Armsby wrote: > Hi Takashi, > I'm writing direct because I cannot seem to get a post on AlsaDevel > list. I've realized your two posts, but no reaction since they have no message body: * Firewire ALSA Motu and Phase24 * https://lore.kernel.org/alsa-devel/sokos1-r6xj933yn42y-hmy3gsrfgsma-mnwe9a-… * Firewire Motu ALSA (Re-post) * https://lore.kernel.org/alsa-devel/… [View More]v6inx6-fwmr0r-kvajcd-kfs8u9l6m2qn2cnogd-… Jaroslav, would I ask your opinion for the problem which perhaps dropped the message body from his messages? > I need to repair my two Phase X24 (defekt caps?) and I'm having > problems with Traveler and other Motu interfaces on Linux. > Thanks for any hints or where I should post. > Here's a copy of my post: > Hello everyone! > Thank you for all the hard work on this. > Especially Mr. Takashi Sakamoto. > I'm an experienced studio engineer and was a Beta tester for Samplitude > for years. > Not a programmer and have very little experience with Linux. > So please excuse me blundering in on this professional programmer forum > but I'm hoping my experience will help somehow to find bugs. I'll > provide any information I can or test following instructions. > I really would love to completely change over to Linux but I need > reliable drivers. > How can I help? > Here's my experience so far. > Around November I started testing Linux for audio as I have > occasionally over the last years. I was pleased to find Manjaro and > Sparky would detect Traveler Mk1 and Mk3 and to run so efficiently > compared to Win10. > But things would freeze. > Reaper has a long standing Midi bug so it was difficult at first to see > why my laptop would freeze on occasion. I dissabled hardware midi in > the software. > Then I updated Sparky beginning of December and to my surprise Firewire > didn't work anymore. The Firewire stack must have been removed. > I use: > Dell Laptops 4310, 6510, M4800 > Tested platforms Ardour and Reaper using Pianoteq 8 > No JACK > Manjaro Kernel from 6.0 rc4? and 6.1.1 work for a limited time. > Sparky only works with special kernels Xanmod1 6.1.7-x64v2 but the > audio is completely distorted. > CachyOS Kernel 6.2 rc4 sounds good. > Common problem is freezing or loosing audio. Ardour is less sensitive > than Reaper but Reaper is far more efficient on CPU > (20% compared to 90%and more with one instance of Pianoteq 7 or 8) > Motu Traveler MK3 and MK1 wrong channel assignment from 1+2 (3+4 are > actually 1+2) and freezing or just loosing audio. > Motu 896Mk3 Hybrid not recognised in firewire or USB mode. > Shame cause its the best. > Motu 8pre working similarly. > Phase 24 working much better but still freezes or looses audio after > long period. > Tested in Ardour and Reaper. > I hope that's not too much at once. > Martin Armsby > M. Armsby Thanks for the report including many issues. I apologize to trouble you. In my opinion, the issues are not so easy since each application uses ALSA PCM device by slightly different ways. So we need to investigate the issue step by step to take relevant longer time than easy issues. Before discussing about the issue, I need to ask a help about MOTU devices. As you realized, 896 mk3 is neither supported yet, FireWire only nor Hybrid, in IEEE 1394 bus nor Universal Serial Bus. For example, you can see no entry for the device in module device table[1] in the file which consists of snd-firewire-motu driver. The reason is apparent; no one provides information enough to support it. Then I would like you to help me to integrate for it in the case in IEEE 1394 bus. I maintain a remote repository for my collection of configuration ROM image which includes identity information of each node in IEEE 1394 bus[2]. The information is useful both to make the entry of module device table and to make hardware database[3] in systemd project. When 896 mk3 Hybrid is connected to IEEE 1394 bus and only connected to the bus, Linux FireWire subsystem detects it as '/dev/fw1'. Then the rom image is exported to an node on sysfs. You can retrieve the content by the command line below: $ cat /sys/bus/firewire/devices/fw1/config_rom > motu-896mk3hybrid.img If retrieved successfully, please send it to me. I don't mind to receive it to private message, mailing list, any upload services. [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/sou… [2] https://github.com/takaswie/am-config-roms [3] https://github.com/systemd/systemd/blob/main/hwdb.d/80-ieee1394-unit-functi… Thanks Takashi Sakamoto [View Less]

3 4

[PATCH] ALSA: hda/realtek: fix mute/micmute LEDs don't work for a HP platform.
by Andy Chi 07 Feb '23

07 Feb '23

There is a HP platform needs ALC236_FIXUP_HP_GPIO_LED quirk to make mic-mute/audio-mute working. Signed-off-by: Andy Chi <andy.chi(a)canonical.com> --- sound/pci/hda/patch_realtek.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 4055a8f5880a..d1258dd803e8 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -9433,6 +9433,11 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {… [View More]

2 1

[PATCH] soundwire: export sdw_compute_slave_ports() function
by Vijendar Mukunda 07 Feb '23

07 Feb '23

Export sdw_compute_slave_ports() function to use it in another soundwire manager module. Move sdw_transport_data structure to bus header file to export sdw_compute_slave_ports() function. Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda(a)amd.com> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> Link: https://lore.kernel.org/lkml/20230201165944.3169125-1-Vijendar.Mukunda@amd.… --- drivers/soundwire/bus.h | 9 +++++++++ drivers/… [View More]soundwire/generic_bandwidth_allocation.c | 12 +++--------- 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/drivers/soundwire/bus.h b/drivers/soundwire/bus.h index 7631ef5e71fb..141b99ac58de 100644 --- a/drivers/soundwire/bus.h +++ b/drivers/soundwire/bus.h @@ -144,6 +144,13 @@ struct sdw_master_runtime { struct list_head bus_node; }; +struct sdw_transport_data { + int hstart; + int hstop; + int block_offset; + int sub_block_offset; +}; + struct sdw_dpn_prop *sdw_get_slave_dpn_prop(struct sdw_slave *slave, enum sdw_data_direction direction, unsigned int port_num); @@ -213,5 +220,7 @@ int sdw_bwrite_no_pm_unlocked(struct sdw_bus *bus, u16 dev_num, u32 addr, u8 val void sdw_clear_slave_status(struct sdw_bus *bus, u32 request); int sdw_slave_modalias(const struct sdw_slave *slave, char *buf, size_t size); +void sdw_compute_slave_ports(struct sdw_master_runtime *m_rt, + struct sdw_transport_data *t_data); #endif /* __SDW_BUS_H */ diff --git a/drivers/soundwire/generic_bandwidth_allocation.c b/drivers/soundwire/generic_bandwidth_allocation.c index f7c66083a4dd..39543048baa7 100644 --- a/drivers/soundwire/generic_bandwidth_allocation.c +++ b/drivers/soundwire/generic_bandwidth_allocation.c @@ -28,15 +28,8 @@ struct sdw_group { unsigned int *rates; }; -struct sdw_transport_data { - int hstart; - int hstop; - int block_offset; - int sub_block_offset; -}; - -static void sdw_compute_slave_ports(struct sdw_master_runtime *m_rt, - struct sdw_transport_data *t_data) +void sdw_compute_slave_ports(struct sdw_master_runtime *m_rt, + struct sdw_transport_data *t_data) { struct sdw_slave_runtime *s_rt = NULL; struct sdw_port_runtime *p_rt; @@ -85,6 +78,7 @@ static void sdw_compute_slave_ports(struct sdw_master_runtime *m_rt, } } } +EXPORT_SYMBOL(sdw_compute_slave_ports); static void sdw_compute_master_ports(struct sdw_master_runtime *m_rt, struct sdw_group_params *params, -- 2.34.1 [View Less]

1 0

[PATCH] ASoC: dt-bindings: renesas: adjust to R-Car Gen4
by Kuninori Morimoto 07 Feb '23

07 Feb '23

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> R-Car Gen4 is not compatible with Gen3, this patch adjusts to R-Car Gen4. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- The "required" with if - then - else on "rcar_sound,ssi" is always match to "then" even though it is checking "renesas,rcar_sound-gen4" or not. Why ?? Is it my fault ?? .../bindings/sound/renesas,rsnd.yaml | 62 ++++++++++++++----- 1 file changed, 46 insertions(+), … [View More]16 deletions(-) diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml index d106de00c6b2..9a88b1c34e72 100644 --- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml @@ -106,7 +106,9 @@ properties: items: oneOf: - const: ssi-all + - const: clkin - pattern: '^ssi\.[0-9]$' + - pattern: '^ssiu\.[0-9]$' - pattern: '^src\.[0-9]$' - pattern: '^mix\.[0-1]$' - pattern: '^ctu\.[0-1]$' @@ -254,10 +256,20 @@ properties: no-busif: description: BUSIF is not used when [mem -> SSI] via DMA case $ref: /schemas/types.yaml#/definitions/flag - required: - - interrupts - - dmas - - dma-names + allOf: + - if: + properties: + compatible: + contains: + const: renesas,rcar_sound-gen4 + then: + required: + - interrupts + else: + required: + - interrupts + - dmas + - dma-names additionalProperties: false # For DAI base @@ -307,18 +319,36 @@ allOf: - ssi - adg else: - properties: - reg: - maxItems: 5 - reg-names: - maxItems: 5 - items: - enum: - - scu - - adg - - ssiu - - ssi - - audmapp + if: + properties: + compatible: + contains: + const: renesas,rcar_sound-gen4 + then: + properties: + reg: + maxItems: 4 + reg-names: + maxItems: 4 + items: + enum: + - adg + - ssiu + - ssi + - sdmc + else: + properties: + reg: + maxItems: 5 + reg-names: + maxItems: 5 + items: + enum: + - scu + - adg + - ssiu + - ssi + - audmapp unevaluatedProperties: false -- 2.25.1 [View Less]

3 4

[PATCH] ASoC: codecs: es8326: Fix DTS properties reading
by Alexey Firago 06 Feb '23

06 Feb '23

Seems like properties parsing and reading was copy-pasted, so "everest,interrupt-src" and "everest,interrupt-clk" are saved into the es8326->jack_pol variable. This might lead to wrong settings being saved into the reg 57 (ES8326_HP_DET). Fix this by using proper variables while reading properties. Signed-off-by: Alexey Firago <a.firago(a)yadro.com> --- sound/soc/codecs/es8326.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/es8326.c b/… [View More]

2 1

[PATCH v3 0/2] Two bug fixes for tas5805m codec driver
by Daniel Beer 06 Feb '23

06 Feb '23

This pair of patches fixes two issues which crept in while revising the original submission, at a time when I no longer had access to test hardware. The fixes here have been tested and verified on hardware. Changes v2 -> v3: - Resubmit with cover letter Changes v1 -> v2: - Fix issue reported by kernel test robot Daniel Beer (2): ASoC: tas5805m: rework to avoid scheduling while atomic. ASoC: tas5805m: add missing page switch. sound/soc/codecs/tas5805m.c | 131 +++++++++++++++++… [View More]

2 2

[PATCH] ALSA: hda/realtek: Enable mute/micmute LEDs on HP Elitebook, 645 G9
by Elvis Angelaccio 06 Feb '23

06 Feb '23

The HP Elitebook 645 G9 laptop (with motherboard model 89D2) uses the ALC236 codec and requires the alc236_fixup_hp_mute_led_micmute_vref fixup in order to enable mute/micmute LEDs. Note: the alc236_fixup_hp_gpio_led fixup, which is used by the Elitebook 640 G9, does not work with the 645 G9. Signed-off-by: Elvis Angelaccio <elvis.angelaccio(a)kde.org> --- sound/pci/hda/patch_realtek.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/… [View More]

1 0

[PATCH 00/19] Add soundwire support for Pink Sardine platform
by Vijendar Mukunda 06 Feb '23

06 Feb '23

Pink Sardine(ps) platform is based on ACP6.3 Architecture. ACP6.3 IP has two soundwire controller instance support. This patchset add support for Soundwire controller on Pink Sardine platform. Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda(a)amd.com> Vijendar Mukunda (19): ASoC: amd: ps: create platform devices based on acp config soundwire: amd: Add support for AMD Master driver soundwire: amd: register sdw controller dai ops soundwire: amd: enable build for AMD soundwire … [View More]

8 90

[PATCH v1] ALSA: hda/realtek: Add quirk for ASUS UM3402 using CS35L41
by Stefan Binding 06 Feb '23

06 Feb '23

This Asus Zenbook laptop use Realtek HDA codec combined with 2xCS35L41 Amplifiers using I2C with External Boost. Signed-off-by: Stefan Binding <sbinding(a)opensource.cirrus.com> --- sound/pci/hda/patch_realtek.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 4055a8f5880a..251b58069d6c 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -9480,6 +9480,7 @@ static const struct … [View More]

2 1

Overflow in calculating audio timestamp
by Alan Young 06 Feb '23

06 Feb '23

sound/core/pcm_lib.c:update_audio_tstamp() contains the following calculation: audio_nsecs = div_u64(audio_frames * 1000000000LL, runtime->rate); This will result in a 64-bit overflow after 4.4 days at 48000 Hz, or 1.1 days at 192000. Are you interested in a patch to improve this? The same calculation occurs in a couple of other places. Alan.

4 10