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

[PATCH v3 0/6] ASoC: mediatek: Add support for MT7986 SoC
by Maso Huang 04 Aug '23

04 Aug '23

Changes in v3: - merge clk api to mt7986-afe-pcm.c, remove [2/7] in v2 - refine based on reviewer's suggestions [1/6] [2/6] - fix the comment format, move in clk api, and simplify based on reviewer's suggestions [3/6] - remove redundant prefix in dt-binding [6/6] Changes in v2: - v1 title: [PATCH 0/7] ASoC: mediatek: Add support for MT79xx SoC - add missing maintainers - rename mt79xx to mt7986 in all files - use clk bulk api in mt7986-afe-clk.c [2/7] - refine mt79xx-afe-pcm.c based on reviewer's suggestions [4/7] - refine dt-binding files based on reviewer's suggestions [6/7] [7/7] - transpose [3/7] and [4/7] in v1 to fix test build errors Maso Huang (6): ASoC: mediatek: mt7986: add common header ASoC: mediatek: mt7986: support etdm in platform driver ASoC: mediatek: mt7986: add platform driver ASoC: mediatek: mt7986: add machine driver with wm8960 ASoC: dt-bindings: mediatek,mt7986-wm8960: add mt7986-wm8960 document ASoC: dt-bindings: mediatek,mt7986-afe: add audio afe document .../bindings/sound/mediatek,mt7986-afe.yaml | 89 +++ .../sound/mediatek,mt7986-wm8960.yaml | 53 ++ sound/soc/mediatek/Kconfig | 20 + sound/soc/mediatek/Makefile | 1 + sound/soc/mediatek/mt7986/Makefile | 9 + sound/soc/mediatek/mt7986/mt7986-afe-common.h | 49 ++ sound/soc/mediatek/mt7986/mt7986-afe-pcm.c | 622 ++++++++++++++++++ sound/soc/mediatek/mt7986/mt7986-dai-etdm.c | 420 ++++++++++++ sound/soc/mediatek/mt7986/mt7986-reg.h | 206 ++++++ sound/soc/mediatek/mt7986/mt7986-wm8960.c | 184 ++++++ 10 files changed, 1653 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/mediatek,mt7986-afe.yaml create mode 100644 Documentation/devicetree/bindings/sound/mediatek,mt7986-wm8960.yaml create mode 100644 sound/soc/mediatek/mt7986/Makefile create mode 100644 sound/soc/mediatek/mt7986/mt7986-afe-common.h create mode 100644 sound/soc/mediatek/mt7986/mt7986-afe-pcm.c create mode 100644 sound/soc/mediatek/mt7986/mt7986-dai-etdm.c create mode 100644 sound/soc/mediatek/mt7986/mt7986-reg.h create mode 100644 sound/soc/mediatek/mt7986/mt7986-wm8960.c -- 2.18.0

5 16

[PATCH v2 0/3] Add PWM-DAC audio support for StarFive JH7110 RISC-V SoC
by Hal Feng 03 Aug '23

03 Aug '23

This patchset adds PWM-DAC audio support for the StarFive JH7110 SoC. The PWM-DAC module does not require a hardware codec, but a dummy codec is needed for the driver. The dummy spdif codec driver, which is already upstream, is compatible with the one which JH7110 PWM-DAC needed. So we use it as the dummy codec driver for the JH7110 PWM-DAC module. The third patch depends on tag next-20230728 in linux-next branch. Changes since v1: - Rebase on tag v6.5-rc3. - Drop patch 1 and 2. - Drop the unneeded space and line in patch 3. - Use the dummy spdif codec driver instead of adding a new one. - Change "dai_link->stop_dma_first = 1" to "dai_link->trigger_stop = SND_SOC_TRIGGER_ORDER_LDC" in patch 4. - Drop the unneeded "status = "okay;" in patch 5. - Change some node names in patch 5. --- v1: https://lore.kernel.org/all/20230626110909.38718-1-hal.feng@starfivetech.co… Hal Feng (3): ASoC: dt-bindings: Add StarFive JH7110 PWM-DAC controller ASoC: starfive: Add JH7110 PWM-DAC driver riscv: dts: starfive: Add JH7110 PWM-DAC support .../sound/starfive,jh7110-pwmdac.yaml | 76 ++ MAINTAINERS | 7 + .../jh7110-starfive-visionfive-2.dtsi | 99 +++ arch/riscv/boot/dts/starfive/jh7110.dtsi | 156 ++++ sound/soc/starfive/Kconfig | 9 + sound/soc/starfive/Makefile | 1 + sound/soc/starfive/jh7110_pwmdac.c | 787 ++++++++++++++++++ 7 files changed, 1135 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/starfive,jh7110-pwmdac.yaml create mode 100644 sound/soc/starfive/jh7110_pwmdac.c base-commit: 6eaae198076080886b9e7d57f4ae06fa782f90ef -- 2.38.1

3 7

[PATCH 0/6] ASoC: SOF: misc updates for 6.6
by Pierre-Louis Bossart 03 Aug '23

03 Aug '23

Cleanups, move of code to promote reuse and Mediatek minor updates. Curtis Malainey (1): ASoC: SOF: Deprecate invalid enums in IPC3 Pierre-Louis Bossart (3): ASoC: SOF: Intel: start simplify the signature of link_slaves_found() ASoC: soc-acpi: move link_slaves_found() ASoC: soc-acpi: improve log messagesin link_slaves_found() Trevor Wu (2): ASoC: SOF: ipc3: update dai_link_fixup for SOF_DAI_MEDIATEK_AFE ASoC: SOF: mediatek: mt8186 modify dram type as non-cache include/linux/soundwire/sdw.h | 5 ++ include/linux/soundwire/sdw_intel.h | 7 +-- include/sound/soc-acpi.h | 6 ++ include/sound/sof/topology.h | 4 +- sound/soc/soc-acpi.c | 73 +++++++++++++++++++++++ sound/soc/sof/intel/hda.c | 81 +------------------------- sound/soc/sof/ipc3-pcm.c | 17 ++++++ sound/soc/sof/mediatek/mt8186/mt8186.c | 40 +++++++------ 8 files changed, 130 insertions(+), 103 deletions(-) -- 2.39.2

3 9

[PATCH] ASoC: intel: use devm_snd_soc_register_dai()
by Kuninori Morimoto 02 Aug '23

02 Aug '23

It is still using snd_soc_{un}register_dai() manually. Let's use devm_snd_soc_register_dai(). Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/intel/avs/pcm.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/sound/soc/intel/avs/pcm.c b/sound/soc/intel/avs/pcm.c index 1fbb2c2fadb5..dd03db31c2d8 100644 --- a/sound/soc/intel/avs/pcm.c +++ b/sound/soc/intel/avs/pcm.c @@ -1309,8 +1309,6 @@ static void avs_component_hda_unregister_dais(struct snd_soc_component *componen for_each_pcm_streams(stream) snd_soc_dapm_free_widget(snd_soc_dai_get_widget(dai, stream)); - - snd_soc_unregister_dai(dai); } } @@ -1375,7 +1373,7 @@ static int avs_component_hda_probe(struct snd_soc_component *component) } } - dai = snd_soc_register_dai(component, &dais[i], false); + dai = devm_snd_soc_register_dai(component->dev, component, &dais[i], false); if (!dai) { dev_err(component->dev, "register dai for %s failed\n", pcm->name); -- 2.25.1

3 3

How do I set up multiple codecs on one I2S - without TDM?
by John Watts 01 Aug '23

01 Aug '23

Hello there! I have an interesting problem that I'm trying to solve. It will require writing adding driver support but right now I'm struggling to understand how I should go about it all. As a background, this is with the Allwinner D1 or T113. It has a 16 channel I2S controller and each channel can be configured to use a specific pin and TDM time slice. Even channels are low LRCLK, odd are high LRCLK. So for my situation I want to have six channels: - Channel 0: Timeslot 0, Pin 0, LRCLK 0 -> ADC 1 - Channel 1: Timeslot 0, Pin 0, LRCLK 1 -> ADC 1 - Channel 2: Timeslot 0, Pin 1, LRCLK 0 -> ADC 2 - Channel 3: Timeslot 0, Pin 1, LRCLK 1 -> ADC 2 - Channel 4: Timeslot 0, Pin 2, LRCLK 0 -> ADC 3 - Channel 5: Timeslot 0, Pin 2, LRCLK 1 -> ADC 3 These ADCs are WM8782s, completely TDM unaware. If they were TDM aware then I could run even more ADCs by having multiple separate TDMs going on each pin. I have tried to look online for I2S controllers that do the same thing but haven't had much luck. The current driver for this controller only supports TDM. So I will have to add support for this. So my questions are: - How do I express this in terms of ALSA and sound card concepts? - How do I express this in the device tree? My immediate thought is that this is a situation where I have a card with 3 dai-links between the I2S controller and each separate codec. I have written this tree as a test: wm8782_1: stereo-adc { compatible = "wlf,wm8782"; Vdda-supply = <&reg_vcc>; Vdd-supply = <&reg_vcc>; #sound-dai-cells = <0>; }; wm8782_2: stereo-adc { compatible = "wlf,wm8782"; Vdda-supply = <&reg_vcc>; Vdd-supply = <&reg_vcc>; #sound-dai-cells = <0>; }; wm8782_3: stereo-adc { compatible = "wlf,wm8782"; Vdda-supply = <&reg_vcc>; Vdd-supply = <&reg_vcc>; #sound-dai-cells = <0>; }; sound { compatible = "simple-audio-card"; simple-audio-card,name = "Testcard In"; simple-audio-card,format = "i2s"; simple-audio-card,bitclock-master = <&sound_cpu>; simple-audio-card,frame-master = <&sound_cpu>; simple-audio-card,mclk-fs = <128>; #address-cells = <1>; #size-cells = <0>; simple-audio-card,dai-link@0 { reg = <0>; sound_cpu: cpu { sound-dai = <&i2s2>; }; codec { sound-dai = <&wm8782_1>; }; }; simple-audio-card,dai-link@1 { reg = <1>; cpu { sound-dai = <&i2s2>; }; codec { sound-dai = <&wm8782_2>; }; }; simple-audio-card,dai-link@2 { reg = <2>; cpu { sound-dai = <&i2s2>; }; codec { sound-dai = <&wm8782_3>; }; }; }; But I get this kernel error: kobject_add_internal failed for 2034000.i2s-wm8782 with -EEXIST, don't try to register things with the same name asoc-simple-card: probe of sound failed with error -12 The WM8782 doesn't support TDM or multiplexing so I'm not exactly sure if this is correct behaviour or my mistake configuring. I tried adding TDM nodes which didn't help. I also tried having a single link with multiple codecs on sound-dai which did create a sound card but it was unclear to me if this was ignoring the other codecs I had listed or not. I also can't test this at this point as the sunxi codec driver assumes that the number of channels is equal to TDM slots, which isn't true any more. Any pointers in the right direction would be a huge help. Surely someone else has had this problem. Thanks, John.

2 3

[PATCH] ASoC: rt5665: add missed regulator_bulk_disable
by Zhang Shurong 01 Aug '23

01 Aug '23

The driver forgets to call regulator_bulk_disable() Add the missed call to fix it. Fixes: 33ada14a26c8 ("ASoC: add rt5665 codec driver") Signed-off-by: Zhang Shurong <zhang_shurong(a)foxmail.com> --- sound/soc/codecs/rt5665.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/sound/soc/codecs/rt5665.c b/sound/soc/codecs/rt5665.c index 83c367af91da..de36cad8a559 100644 --- a/sound/soc/codecs/rt5665.c +++ b/sound/soc/codecs/rt5665.c @@ -4756,6 +4756,13 @@ static void rt5665_calibrate_handler(struct work_struct *work) rt5665_calibrate(rt5665); } +static void rt5665_i2c_disable_regulators(void *data) +{ + struct rt5665_priv *rt5665 = data; + + regulator_bulk_disable(ARRAY_SIZE(rt5665->supplies), rt5665->supplies); +} + static int rt5665_i2c_probe(struct i2c_client *i2c) { struct rt5665_platform_data *pdata = dev_get_platdata(&i2c->dev); @@ -4786,6 +4793,11 @@ static int rt5665_i2c_probe(struct i2c_client *i2c) return ret; } + ret = devm_add_action_or_reset(&i2c->dev, rt5665_i2c_disable_regulators, + rt5665); + if (ret) + return ret; + ret = regulator_bulk_enable(ARRAY_SIZE(rt5665->supplies), rt5665->supplies); if (ret != 0) { -- 2.41.0

2 2

[PATCH 00/23] ASoC: Intel: machine driver updates for 6.6
by Pierre-Louis Bossart 01 Aug '23

01 Aug '23

Lots of small cleanups coming from Bard Liao and Charles Keepax for SoundWire platforms, and minor additions for RVPs and Chromebooks. Balamurugan C (1): ASoC: Intel: soc-acpi: Add entry for rt711-sdca-sdw at link 0 in RPL match table Bard Liao (2): ASoC: Intel: sof_sdw: rename link_id to be_id ASoC: Intel: sof_sdw: break earlier when a adr link contains different codecs Chao Song (1): ASoC: Intel: sof_sdw: add cs35l56 codec info Charles Keepax (13): ASoC: intel: sof_sdw: Use consistent variable naming for links ASoC: intel: sof_sdw: Rename codec_idx to codec_dlc_index ASoC: intel: sof_sdw: Remove some extra line breaks ASoC: intel: sof_sdw: Use a module device table ASoC: intel: sof_sdw: Simplify find_codec_info_acpi ASoC: intel: sof_sdw: Constify parameter to find_codec_part_info ASoC: intel: sof_sdw: Minor tidy up of mc_probe ASoC: intel: sof_sdw: Remove redundant parameters in dai creation ASoC: intel: sof_sdw: Move amp_num initialisation to mc_probe ASoC: intel: sof_sdw: Allow direct specification of CODEC name ASoC: intel: sof_sdw: Clean up DAI link counting ASoC: intel: sof_sdw: Merge codec_conf_alloc into dailink_info ASoC: intel: sof_sdw: Move group_generated logic Pierre-Louis Bossart (3): ASoC: Intel: sof_sdw: reorder SoundWire codecs in Kconfig ASoC: Intel: sof_sdw: allow mockup amplifier to provide feedback ASoC: Intel: sof_sdw: add support for SKU 0AFE Terry Cheong (1): ASoC: Intel: sof_rt5682: add RPL support for MAX98357A speaker Uday M Bhat (1): ASoC: Intel: sof_sdw: Update BT offload config for soundwire config apoorv (1): ASoC: Intel: sof_rt5682 add support for HDMI_In capture sound/soc/intel/boards/Kconfig | 5 +- sound/soc/intel/boards/Makefile | 2 +- sound/soc/intel/boards/sof_rt5682.c | 55 +++ sound/soc/intel/boards/sof_sdw.c | 404 +++++++++--------- sound/soc/intel/boards/sof_sdw_common.h | 7 + sound/soc/intel/boards/sof_sdw_cs_amp.c | 47 ++ .../intel/common/soc-acpi-intel-adl-match.c | 7 + .../intel/common/soc-acpi-intel-rpl-match.c | 27 ++ 8 files changed, 342 insertions(+), 212 deletions(-) create mode 100644 sound/soc/intel/boards/sof_sdw_cs_amp.c -- 2.39.2

2 24

[PATCH v4 0/1] Intel: sof_rt5682: remove quirk flag
by Brent Lu 01 Aug '23

01 Aug '23

We add a helper function to detect amplifier number according to device instance in ACPI table so the SOF_MAX98390_TWEETER_SPEAKER_PRESENT flag and a dmi quirk for 4-amplifier configuration could be safely removed. Also refactor the max_98390_hw_params() function to use an array to handle the TDM parameter. Amplifier number detection and TDM parameter are tested on two Chromebooks. One with 2 MAX98390 and one with 4 MAX98390 amplifier. V2 Changes: - fix some typos in cover letter - use for_each_acpi_dev_match() macro to simplify the codec - use 'unsign int' for codec number variable V3 Changes: - replace pr_err() with dev_err() V4 Changes: - merge two patches to avoid build break - remove redundant assignment in max_98390_hw_params() - refactor MAX_98390_DEVX_NAME macros Brent Lu (2): ASoC: Intel: maxim-common: get codec number from ACPI ASoC: Intel: sof_rt5682: remove SOF_MAX98390_TWEETER_SPEAKER_PRESENT flag Brent Lu (1): ASoC: Intel: maxim-common: get codec number from ACPI table sound/soc/intel/boards/sof_maxim_common.c | 170 +++++++++++++--------- sound/soc/intel/boards/sof_maxim_common.h | 21 ++- sound/soc/intel/boards/sof_rt5682.c | 37 +---- 3 files changed, 111 insertions(+), 117 deletions(-) -- 2.34.1

5 6

[PATCH 0/8] ASoC/SOF/Intel/AMD: cleanups for GCC11 -fanalyzer checks
by Pierre-Louis Bossart 01 Aug '23

01 Aug '23

GCC11 provides an '-fanalyzer' static analysis option which does not provide too many false-positives. This patch cleans-up known problematic code paths to help enable this capability in CI. We've used this for about a month already. Pierre-Louis Bossart (8): ASoC: SOF: sof-client-probes-ipc4: add checks to prevent static analysis warnings ASoC: SOF: ipc3: add checks to prevent static analysis warnings ASoC: SOF: topology: simplify code to prevent static analysis warnings ASoC: SOF: imx: remove error checks on NULL ipc ASoC: SOF: mediatek: remove error checks on NULL ipc ASoC: Intel: bdw_rt286: add checks to avoid static analysis warnings ASoC: Intel: atom: remove static analysis false positive ASoC: amd: acp5x-mach:add checks to avoid static analysis warnings include/linux/firmware/imx/dsp.h | 6 ------ include/linux/firmware/mediatek/mtk-adsp-ipc.h | 6 ------ sound/soc/amd/vangogh/acp5x-mach.c | 3 +++ sound/soc/intel/atom/sst/sst_stream.c | 13 +++++++------ sound/soc/intel/boards/bdw_rt286.c | 6 ++++++ sound/soc/sof/ipc3.c | 2 +- sound/soc/sof/sof-client-probes-ipc4.c | 9 +++++++++ sound/soc/sof/topology.c | 5 +++-- 8 files changed, 29 insertions(+), 21 deletions(-) -- 2.39.2

2 9

Audio output heavily distorted on Xen PV dom0 if not recording (Intel ADL)
by Marek Marczykowski-Górecki 01 Aug '23

01 Aug '23

Hi, I have a rather interesting issue on HP EliteBook 1040, with Intel ADL HDA. Sometimes (often) audio output is heavily distorted, but then opening pavucontrol or just launching `parecord /dev/null` fixes the issue instantly. But, when the mixer or recording is closed, then few seconds later issue comes back. You can hear this at https://cloud.marmarek.net/s/46prqZnwrqDkBdD Initially recording was running, then about 10s stopped and you can hear the issue few seconds later. Then I starter recording again, and the issue is gone. This is how it should sound: https://youtube.com/watch?v=4Tr0otuiQuU I do not see any kernel messages when issue happens. The issue happens on Linux 6.1.26, 6.2.10, and 6.3.2. I haven't tested other versions. It happens only when running under Xen (in PV dom0). The very same Linux started directly works fine. Full dmesg of both native and Xen boots is at https://gist.github.com/marmarek/548ee79282446344a172109bcd943930 When running Linux 6.3.2 there, I noticed that just after boot first playback is okay, but if I start and stop recording it breaks. And then after some time of idle it fixes itself again. I have not observed this behavior on 6.2.10 (it was broken initially too), but I could be also doing some things differently. I have few other ADL-based systems (but not HP), and they do not show this issue, only this HP laptop is affected. Example dmesg from another ADL-based system: https://gist.github.com/marmarek/38c5ba3ec58f9300fbc1842ddc9fa4f5 I'm not sure if that's the relevant difference, but comparing dmesg from working and not working systems, I see those on boot for the problematic case: snd_hda_intel 0000:00:1f.3: DSP detected with PCI class/subclass/prog-if info 0x040100 snd_hda_intel 0000:00:1f.3: Digital mics found on Skylake+ platform, using SOF driver sof-audio-pci-intel-tgl 0000:00:1f.3: DSP detected with PCI class/subclass/prog-if info 0x040100 sof-audio-pci-intel-tgl 0000:00:1f.3: Digital mics found on Skylake+ platform, using SOF driver sof-audio-pci-intel-tgl 0000:00:1f.3: enabling device (0000 -> 0002) xen: registering gsi 16 triggering 0 polarity 1 Already setup the GSI :16 sof-audio-pci-intel-tgl 0000:00:1f.3: DSP detected with PCI class/subclass/prog-if 0x040100 sof-audio-pci-intel-tgl 0000:00:1f.3: bound 0000:00:02.0 (ops i915_audio_component_bind_ops [i915]) sof-audio-pci-intel-tgl 0000:00:1f.3: use msi interrupt mode sof-audio-pci-intel-tgl 0000:00:1f.3: hda codecs found, mask 5 sof-audio-pci-intel-tgl 0000:00:1f.3: using HDA machine driver skl_hda_dsp_generic now sof-audio-pci-intel-tgl 0000:00:1f.3: DMICs detected in NHLT tables: 2 sof-audio-pci-intel-tgl 0000:00:1f.3: Firmware info: version 2:2:0-57864 sof-audio-pci-intel-tgl 0000:00:1f.3: Firmware: ABI 3:22:1 Kernel ABI 3:23:0 sof-audio-pci-intel-tgl 0000:00:1f.3: unknown sof_ext_man header type 3 size 0x30 sof-audio-pci-intel-tgl 0000:00:1f.3: Firmware info: version 2:2:0-57864 sof-audio-pci-intel-tgl 0000:00:1f.3: Firmware: ABI 3:22:1 Kernel ABI 3:23:0 sof-audio-pci-intel-tgl 0000:00:1f.3: Topology: ABI 3:22:1 Kernel ABI 3:23:0 skl_hda_dsp_generic skl_hda_dsp_generic: ASoC: Parent card not yet available, widget card binding deferred cs35l41-hda spi0-CSC3551:00-cs35l41-hda.0: DSP1: Firmware version: 3 cs35l41-hda spi0-CSC3551:00-cs35l41-hda.0: DSP1: cirrus/cs35l41-dsp1-spk-prot-103c8974.wmfw: Fri 27 Aug 2021 14:58:19 W. Europe Daylight Time cs35l41-hda spi0-CSC3551:00-cs35l41-hda.0: DSP1: Firmware: 400a4 vendor: 0x2 v0.43.1, 2 algorithms cs35l41-hda spi0-CSC3551:00-cs35l41-hda.0: DSP1: 0: ID cd v29.63.1 XM@94 YM@e cs35l41-hda spi0-CSC3551:00-cs35l41-hda.0: DSP1: 1: ID f20b v0.1.0 XM@176 YM@0 cs35l41-hda spi0-CSC3551:00-cs35l41-hda.0: DSP1: spk-prot: E:\Amp Tuning\HP\840\0527\103C8972_220527.bin snd_hda_codec_realtek ehdaudio0D0: bound spi0-CSC3551:00-cs35l41-hda.0 (ops cs35l41_hda_comp_ops [snd_hda_scodec_cs35l41]) cs35l41-hda spi0-CSC3551:00-cs35l41-hda.1: DSP1: Firmware version: 3 cs35l41-hda spi0-CSC3551:00-cs35l41-hda.1: DSP1: cirrus/cs35l41-dsp1-spk-prot-103c8974.wmfw: Fri 27 Aug 2021 14:58:19 W. Europe Daylight Time cs35l41-hda spi0-CSC3551:00-cs35l41-hda.1: DSP1: Firmware: 400a4 vendor: 0x2 v0.43.1, 2 algorithms cs35l41-hda spi0-CSC3551:00-cs35l41-hda.1: DSP1: 0: ID cd v29.63.1 XM@94 YM@e cs35l41-hda spi0-CSC3551:00-cs35l41-hda.1: DSP1: 1: ID f20b v0.1.0 XM@176 YM@0 cs35l41-hda spi0-CSC3551:00-cs35l41-hda.1: DSP1: spk-prot: E:\Amp Tuning\HP\840\0527\103C8972_220527.bin snd_hda_codec_realtek ehdaudio0D0: bound spi0-CSC3551:00-cs35l41-hda.1 (ops cs35l41_hda_comp_ops [snd_hda_scodec_cs35l41]) Any ideas? -- Best Regards, Marek Marczykowski-Górecki Invisible Things Lab

3 6