March 2018 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [RFC 0/7] ASoC: rt5651: Drop pdata, move over to device-properties
by Hans de Goede 01 Mar '18

01 Mar '18

Hi Mark, et al., Here is a second attempt to stop having duplicated DMI quirk entries between the rt5651 codec and machine drivers. Note this is only compile tested, I'm looking for some quick feedback if this seems to be going in the right direction, as I'm planning on rebuilding the rest of my jack-detect cleanup / fix series on top of this. The diffstat for this entire set is: Documentation/devicetree/bindings/sound/rt5651.txt | 6 include/sound/rt5651.h | 12 sound/soc/codecs/rt5651.c | 375 +++++++++------------ sound/soc/codecs/rt5651.h | 7 sound/soc/intel/boards/bytcr_rt5651.c | 46 ++ 5 files changed, 213 insertions(+), 233 deletions(-) So besides making the code cleaner (IMHO) it also makes it somewhat smaller and as an added bonus it fixes support the DMIC_EN quirk which was broken. Regards, Hans

2 9

[alsa-devel] [PATCH 0/7] Let Atmel use simple-audio-card
by Ladislav Michl 01 Mar '18

01 Mar '18

Alexandre, following patchset is sent as a bare minimum I need to use MAX9867 codec in master mode on AT91SAM9G20 based board. Codec's MCLK is driven by PCK0 and SSC is wired as follows: RK0 <-- BCLK RF0 <-- LRCLK TD0 --> SDIN RD0 <-- SDOUT Following DT framgment was used: i2c-gpio-0 { codec: max9867@18 { compatible = "maxim,max9867"; reg = <0x18>; #sound-dai-cells = <0>; }; }; sound { compatible = "simple-audio-card"; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_pck0_as_mck>; simple-audio-card,name = "simear4-max9867"; simple-audio-card,format = "i2s"; simple-audio-card,bitclock-master = <&codec_dai>; simple-audio-card,frame-master = <&codec_dai>; simple-audio-card,routing = "LINE_IN", "Line In Jack", "Speaker", "HPOUT"; simple-audio-card,widgets = "Line", "Line In Jack", "Speaker", "Speaker"; cpu_dai: simple-audio-card,cpu { sound-dai = <&ssc0>; }; codec_dai: simple-audio-card,codec { sound-dai = <&codec>; clocks = <&pck0>; clock-frequency = <12384000>; }; }; &ssc0 { status = "okay"; pinctrl-0 = <&pinctrl_board_ssc0_tx &pinctrl_ssc0_rx>; atmel,clk-from-rk-pin; #sound-dai-cells = <0>; }; Codec code itself seems to be unused since it was merged and I'm wondering whenever it worked at all. A comment from codec author would help a lot, just to be sure I did not break anything. By no means this code is ready to be merged (only except those patches with commit log and after carefull consideration :)), I'm posting it as a base for discussion. Thank you, ladis Ladislav Michl (7): ASoC: atmel: Remove redundant dev_err() call in probe function ASoC: atmel_ssc_dai: Fix TCMR settings in I2S slave mode ASoC: simple_card_utils: Set clock frequency ASoC: max9867: Show Kconfig entry ASoC: max9867: Calculate LRCLK divider ASoC: max9867: Fix BSEL value in master mode. ASoC: max9867: Take chip out of shutdown sound/soc/atmel/atmel-classd.c | 7 +-- sound/soc/atmel/atmel-pdmic.c | 7 +-- sound/soc/atmel/atmel_ssc_dai.c | 2 +- sound/soc/codecs/Kconfig | 3 +- sound/soc/codecs/max9867.c | 110 +++------------------------------- sound/soc/generic/simple-card-utils.c | 4 ++ 6 files changed, 19 insertions(+), 114 deletions(-) -- 2.15.1

4 19

[alsa-devel] [PATCH - THREAD_SAFE_API 1/1] pcm: Do not access lock_enabled if thread safe API
by twischer＠de.adit-jv.com 01 Mar '18

01 Mar '18

From: Timo Wischer <twischer(a)de.adit-jv.com> is not enabled. Without this commit compiling fails when THREAD_SAFE_API is not enabled Signed-off-by: Timo Wischer <twischer(a)de.adit-jv.com> diff --git a/src/pcm/pcm.c b/src/pcm/pcm.c index 1be3833..d53ed98 100644 --- a/src/pcm/pcm.c +++ b/src/pcm/pcm.c @@ -823,8 +823,11 @@ int snd_pcm_async(snd_pcm_t *pcm, int sig, pid_t pid) sig = SIGIO; if (pid == 0) pid = getpid(); + +#ifdef THREAD_SAFE_API /* async handler may lead to a deadlock; suppose no multi thread */ pcm->lock_enabled = 0; +#endif return pcm->ops->async(pcm->op_arg, sig, pid); } #endif -- 2.7.4

2 1

[alsa-devel] [PATCH] sound/pci/ice1712: replace strcpy() with scnprintf()
by Joey Pabalinas 01 Mar '18

01 Mar '18

Replace unsafe uses of strcpy() to copy the name argument into the sid.name buffer with scnprintf() to guard against possible buffer overflows. Even though we don't actually care about the return value in this specific case, scnprintf() is still preferred over snprintf() due to scnprintf() returning the much more logical length of what was *actually* encoded into the destination array instead of returning length the resulting string *would* be, assuming it all fit into the destination array as snprintf() does. Maybe one day someone will use the return value, and since the behavior is exactly the same apart from the return value it would be better to account for that possibility and program defensively. Signed-off-by: Joey Pabalinas <joeypabalinas(a)gmail.com> 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/sound/pci/ice1712/juli.c b/sound/pci/ice1712/juli.c index 0dbaccf61f33270608..0abacc64168f895d53 100644 --- a/sound/pci/ice1712/juli.c +++ b/sound/pci/ice1712/juli.c @@ -26,6 +26,7 @@ #include <linux/delay.h> #include <linux/interrupt.h> #include <linux/init.h> +#include <linux/kernel.h> #include <linux/slab.h> #include <sound/core.h> #include <sound/tlv.h> @@ -425,10 +426,9 @@ DECLARE_TLV_DB_SCALE(juli_master_db_scale, -6350, 50, 1); static struct snd_kcontrol *ctl_find(struct snd_card *card, const char *name) { - struct snd_ctl_elem_id sid; - memset(&sid, 0, sizeof(sid)); - /* FIXME: strcpy is bad. */ - strcpy(sid.name, name); + struct snd_ctl_elem_id sid = {0}; + + scnprintf(sid.name, sizeof(sid.name), "%s", name); sid.iface = SNDRV_CTL_ELEM_IFACE_MIXER; return snd_ctl_find_id(card, &sid); } diff --git a/sound/pci/ice1712/quartet.c b/sound/pci/ice1712/quartet.c index d145b5eb7ff86d978d..332f6226548c6a089a 100644 --- a/sound/pci/ice1712/quartet.c +++ b/sound/pci/ice1712/quartet.c @@ -25,6 +25,7 @@ #include <linux/delay.h> #include <linux/interrupt.h> #include <linux/init.h> +#include <linux/kernel.h> #include <linux/slab.h> #include <sound/core.h> #include <sound/tlv.h> @@ -785,10 +786,9 @@ DECLARE_TLV_DB_SCALE(qtet_master_db_scale, -6350, 50, 1); static struct snd_kcontrol *ctl_find(struct snd_card *card, const char *name) { - struct snd_ctl_elem_id sid; - memset(&sid, 0, sizeof(sid)); - /* FIXME: strcpy is bad. */ - strcpy(sid.name, name); + struct snd_ctl_elem_id sid = {0}; + + scnprintf(sid.name, sizeof(sid.name), "%s", name); sid.iface = SNDRV_CTL_ELEM_IFACE_MIXER; return snd_ctl_find_id(card, &sid); } -- 2.16.2

3 6

[alsa-devel] [PATCH v1 0/4] ASoC: Add support for DAC PCM1789
by Mylène Josserand 01 Mar '18

01 Mar '18

Hello everyone, You will find in this series the support of Texas Instrument's DAC PCM1789. This DAC is very minimalist and is similar to PCM1792a except for some differences in registers. Series based on asoc tree, "for-next" branch (last commit 130c3888dfdc). It is important to notice that this DAC needs to always have clocks enabled (even without any data) otherwise it will be in a "desynchronized" state and can not send data correctly. This issue has been solved by performing a reset each time a sound is played (see patch 04). This reset can produce a "pop" noise. Thank you in advance for any review. Best regards, Mylène Mylène Josserand (4): ASoC: codecs: pcm179x: Add PCM1789 id ASoC: codecs: pcm179x: Add support for PCM1789 ASoC: codecs: pcm179x: Add reset gpio ASoC: codecs: pcm179x: Add trigger function to perform a reset .../devicetree/bindings/sound/pcm179x.txt | 2 +- sound/soc/codecs/pcm179x-i2c.c | 17 +- sound/soc/codecs/pcm179x.c | 242 ++++++++++++++++++++- sound/soc/codecs/pcm179x.h | 10 +- 4 files changed, 264 insertions(+), 7 deletions(-) -- 2.11.0

4 14

[alsa-devel] [PATCH v2] ASoC: atmel: Remove redundant dev_err() call in probe function
by Ladislav Michl 01 Mar '18

01 Mar '18

There is an error message within devm_ioremap_resource already, so remove the redundant dev_err() call. Signed-off-by: Ladislav Michl <ladis(a)linux-mips.org> Acked-by: Nicolas Ferre <nicolas.ferre(a)microchip.com> --- Changes: - v2: None, just added Ack. sound/soc/atmel/atmel-classd.c | 7 ++----- sound/soc/atmel/atmel-pdmic.c | 7 ++----- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/sound/soc/atmel/atmel-classd.c b/sound/soc/atmel/atmel-classd.c index 9baaf6bc5fbd..3d70061901a0 100644 --- a/sound/soc/atmel/atmel-classd.c +++ b/sound/soc/atmel/atmel-classd.c @@ -583,11 +583,8 @@ static int atmel_classd_probe(struct platform_device *pdev) res = platform_get_resource(pdev, IORESOURCE_MEM, 0); io_base = devm_ioremap_resource(dev, res); - if (IS_ERR(io_base)) { - ret = PTR_ERR(io_base); - dev_err(dev, "failed to remap register memory: %d\n", ret); - return ret; - } + if (IS_ERR(io_base)) + return PTR_ERR(io_base); dd->phy_base = res->start; dd->dev = dev; diff --git a/sound/soc/atmel/atmel-pdmic.c b/sound/soc/atmel/atmel-pdmic.c index 94f852574a44..915c2b064de1 100644 --- a/sound/soc/atmel/atmel-pdmic.c +++ b/sound/soc/atmel/atmel-pdmic.c @@ -633,11 +633,8 @@ static int atmel_pdmic_probe(struct platform_device *pdev) res = platform_get_resource(pdev, IORESOURCE_MEM, 0); io_base = devm_ioremap_resource(dev, res); - if (IS_ERR(io_base)) { - ret = PTR_ERR(io_base); - dev_err(dev, "failed to remap register memory: %d\n", ret); - return ret; - } + if (IS_ERR(io_base)) + return PTR_ERR(io_base); dd->phy_base = res->start; -- 2.16.2

2 1