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

[alsa-devel] Alsa_development
by Srinivas Reddy 12 Jul '18

12 Jul '18

Hi ALSA Team,

1 0

[alsa-devel] [PATCH v3 0/5] Add support for audio on SDM845 SoC
by Rohit kumar 11 Jul '18

11 Jul '18

This provides initial patchset to support audio on Qualcomm Techonologies Inc. SDM845 SoC. Currently, it supports audio playback/capture over Primary MI2S and Quaternary TDM ports. Changes since v2: - Added new file to parse dai-links from device tree and use it in sdm845 and apq8096 machine driver. - Removed qcom prefix from device tree properties as suggested by Robb. - Updated apq8096 machine driver to use common APIs. Rohit kumar (5): ASoC: qcom: dt-bindings: Add sdm845 machine bindings ASoC: qcom: Add support to parse common audio device nodes ASoC: qcom: add sdm845 sound card support ASoC: dt-bindings: Update dt binding name for apq8096 ASoC: qcom: apq8096: Use common APIs to parse device nodes .../devicetree/bindings/sound/qcom,apq8096.txt | 15 +- .../devicetree/bindings/sound/qcom,sdm845.txt | 80 +++++ sound/soc/qcom/Kconfig | 14 + sound/soc/qcom/Makefile | 3 + sound/soc/qcom/apq8096.c | 151 +------- sound/soc/qcom/common.c | 150 ++++++++ sound/soc/qcom/common.h | 14 + sound/soc/qcom/sdm845.c | 390 +++++++++++++++++++++ 8 files changed, 674 insertions(+), 143 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/qcom,sdm845.txt create mode 100644 sound/soc/qcom/common.c create mode 100644 sound/soc/qcom/common.h create mode 100644 sound/soc/qcom/sdm845.c -- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc., is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.

5 23

[alsa-devel] [PATCH v2] ASoC: hdac_hdmi: Program pin-port-mux during jack report event
by Sriram Periyasamy 11 Jul '18

11 Jul '18

During d3/d0 cycle, the connection selection index of all pins points to the default value. This needs to be restored to ensure audio is restored after d3/d0 cycle. So store the connection selection index and program it during jack report event which gets invoked in cases like d3/d0 cycle, hot plug detection when multiple displays are connected. Signed-off-by: Sriram Periyasamy <sriramx.periyasamy(a)intel.com> Signed-off-by: Sanyog Kale <sanyog.r.kale(a)intel.com> Signed-off-by: Jeeja KP <jeeja.kp(a)intel.com> Signed-off-by: Guneshwor Singh <guneshwor.o.singh(a)intel.com> --- v1 -> v2: - Runtime resume the device explicitly in order to use snd_hdac_codec_write() sound/soc/codecs/hdac_hdmi.c | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c index 84f7a7a36e4b..dd5863830eef 100644 --- a/sound/soc/codecs/hdac_hdmi.c +++ b/sound/soc/codecs/hdac_hdmi.c @@ -83,6 +83,7 @@ struct hdac_hdmi_pin { struct list_head head; hda_nid_t nid; bool mst_capable; + int conn_index; struct hdac_hdmi_port *ports; int num_ports; struct hdac_ext_device *edev; @@ -141,6 +142,9 @@ struct hdac_hdmi_priv { #define hdev_to_hdmi_priv(_hdev) ((to_ehdac_device(_hdev))->private_data) +static int hdac_hdmi_port_select_set(struct hdac_ext_device *edev, + struct hdac_hdmi_port *port); + static struct hdac_hdmi_pcm * hdac_hdmi_get_pcm_from_cvt(struct hdac_hdmi_priv *hdmi, struct hdac_hdmi_cvt *cvt) @@ -167,6 +171,30 @@ static void hdac_hdmi_jack_report(struct hdac_hdmi_pcm *pcm, if (is_connect) { /* + * Increment the usage count to ensure that the device + * is runtime active to send verb across. + */ + pm_runtime_get_sync(&edev->hdev.dev); + + /* set the device if pin is mst_capable */ + if (hdac_hdmi_port_select_set(edev, port) < 0) { + dev_err(&edev->hdev.dev, + "port %d device select fail\n", port->id); + return; + } + + /* + * Restore the connection selection index of the + * respective pin. + */ + if (port->pin->conn_index > 0) + snd_hdac_codec_write(&edev->hdev, port->pin->nid, + 0, AC_VERB_SET_CONNECT_SEL, + port->pin->conn_index - 1); + + pm_runtime_put_sync(&edev->hdev.dev); + + /* * Report Jack connect event when a device is connected * for the first time where same PCM is attached to multiple * ports. @@ -903,6 +931,9 @@ static int hdac_hdmi_set_pin_port_mux(struct snd_kcontrol *kcontrol, } } + if (ucontrol->value.enumerated.item[0] > 0) + port->pin->conn_index = ucontrol->value.enumerated.item[0]; + /* * Jack status is not reported during device probe as the * PCMs are not registered by then. So report it here. -- 2.7.4

2 5

[alsa-devel] [PATCH v2] regmap: slimbus: add support to multi read/write
by Srinivas Kandagatla 11 Jul '18

11 Jul '18

SLIMbus supports upto 16 bytes in value management messages, so add support to read/writes upto 16 bytes. This also removes redundant single register reg_read/reg_write. Also useful for paged register access on SLIMbus interfaced codecs. Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> --- drivers/base/regmap/regmap-slimbus.c | 23 +++++++---------------- 1 file changed, 7 insertions(+), 16 deletions(-) diff --git a/drivers/base/regmap/regmap-slimbus.c b/drivers/base/regmap/regmap-slimbus.c index 91d501eda8a9..0968059f1ef5 100644 --- a/drivers/base/regmap/regmap-slimbus.c +++ b/drivers/base/regmap/regmap-slimbus.c @@ -7,33 +7,24 @@ #include "internal.h" -static int regmap_slimbus_byte_reg_read(void *context, unsigned int reg, - unsigned int *val) +static int regmap_slimbus_write(void *context, const void *data, size_t count) { struct slim_device *sdev = context; - int v; - v = slim_readb(sdev, reg); - - if (v < 0) - return v; - - *val = v; - - return 0; + return slim_write(sdev, *(u16 *)data, count - 2, (u8 *)data + 2); } -static int regmap_slimbus_byte_reg_write(void *context, unsigned int reg, - unsigned int val) +static int regmap_slimbus_read(void *context, const void *reg, size_t reg_size, + void *val, size_t val_size) { struct slim_device *sdev = context; - return slim_writeb(sdev, reg, val); + return slim_read(sdev, *(u16 *)reg, val_size, val); } static struct regmap_bus regmap_slimbus_bus = { - .reg_write = regmap_slimbus_byte_reg_write, - .reg_read = regmap_slimbus_byte_reg_read, + .write = regmap_slimbus_write, + .read = regmap_slimbus_read, .reg_format_endian_default = REGMAP_ENDIAN_LITTLE, .val_format_endian_default = REGMAP_ENDIAN_LITTLE, }; -- 2.16.2

2 1

[alsa-devel] [PATCH 0/2] ASoC: dpcm: add runtime rate merge of BE.
by Jerome Brunet 11 Jul '18

11 Jul '18

This patchset adds the possibility to merge BE rates in the dpcm runtime hw, like it is done for formats and channels. This first patch is a simple rework of format and channel merge to ease the addition of rate merge. Jerome Brunet (2): ASoC: dpcm: rework runtime stream merge ASoC: dpcm: add rate merge to the BE stream merge include/sound/soc.h | 2 ++ sound/soc/soc-pcm.c | 99 ++++++++++++++++++++++++++++++++++++++++------------- 2 files changed, 77 insertions(+), 24 deletions(-) -- 2.14.4

2 4

[alsa-devel] poll blocked after prepare state
by Harald Schuster 11 Jul '18

11 Jul '18

Hi I have a problem when I got an underrun that my playback path is not working correctly any longer. I debugged the problem and see that the snd_pcm_direct_prepare function is called and than the snd_pcm_dshare_start function. After this the snd_pcm_write_areas is called and the snd_pcm_wait_nocheck never returns. He is blocked in the poll function. Is there any solution for this problem?! best regards Harald

2 14

[alsa-devel] [PATCH - usb-audio 1/1] ALSA: usb-audio: Allow changing from a bad sample rate
by Adam Goode 11 Jul '18

11 Jul '18

If the audio device is externally clocked and set to a rate that does not match the external clock, the clock will never be valid and we cannot set the rate successfully. To fix this, allow a rate change even if the clock is initially invalid, and validate again after the rate is changed. This fixes problems with MOTU UltraLite AVB hardware over USB. Signed-off-by: Adam Goode <agoode(a)google.com> diff --git a/sound/usb/clock.c b/sound/usb/clock.c index c79749613fa6..9e9019464f91 100644 --- a/sound/usb/clock.c +++ b/sound/usb/clock.c @@ -513,14 +513,26 @@ static int set_sample_rate_v2v3(struct snd_usb_audio *chip, int iface, bool writeable; u32 bmControls; + /* First, try to find a valid clock. This may trigger + * automatic clock selection if the current clock is not + * valid. */ clock = snd_usb_clock_find_source(chip, fmt->protocol, fmt->clock, true); - if (clock < 0) - return clock; + if (clock < 0) { + /* We did not find a valid clock, but that might be + * because the current sample rate does not match an + * external clock source. Try again without validation + * and we will do another validation after setting the + * rate. */ + clock = snd_usb_clock_find_source(chip, fmt->protocol, + fmt->clock, false); + if (clock < 0) + return clock; + } prev_rate = get_sample_rate_v2v3(chip, iface, fmt->altsetting, clock); if (prev_rate == rate) - return 0; + return uac_clock_source_is_valid(chip, fmt->protocol, clock) ? 0 : -ENXIO; if (fmt->protocol == UAC_VERSION_3) { struct uac3_clock_source_descriptor *cs_desc; @@ -577,7 +589,8 @@ static int set_sample_rate_v2v3(struct snd_usb_audio *chip, int iface, snd_usb_set_interface_quirk(dev); } - return 0; + /* validate clock after rate change */ + return uac_clock_source_is_valid(chip, fmt->protocol, clock) ? 0 : -ENXIO; } int snd_usb_init_sample_rate(struct snd_usb_audio *chip, int iface, -- 2.18.0.203.gfac676dfb9-goog

2 1

[alsa-devel] [PATCH - snd_interval_refine 1/1] pcm: snd_interval_refine_first/last: exclude value only if also excluded before
by twischer＠de.adit-jv.com 11 Jul '18

11 Jul '18

From: Timo Wischer <twischer(a)de.adit-jv.com> Without this commit the following intervals [x y), (x y) were be replaced to (y-1 y) by snd_interval_refine_last(). This was also done if y-1 is part of the previous interval. With this changes it will be replaced with [y-1 y) in case of y-1 is part of the previous interval. A similar behavior will be used for snd_interval_refine_first(). This commit adapts the changes for alsa-lib of commit 9bb985c ("pcm: snd_interval_refine_first/last: exclude value only if also excluded before") Signed-off-by: Timo Wischer <twischer(a)de.adit-jv.com> --- That this fix is also required for kernel was spotted by Takashi in [1]. [1] http://mailman.alsa-project.org/pipermail/alsa-devel/2018-July/137764.html diff --git a/sound/core/pcm_lib.c b/sound/core/pcm_lib.c index faa6786..729a85a 100644 --- a/sound/core/pcm_lib.c +++ b/sound/core/pcm_lib.c @@ -629,27 +629,33 @@ EXPORT_SYMBOL(snd_interval_refine); static int snd_interval_refine_first(struct snd_interval *i) { + const unsigned int last_max = i->max; + if (snd_BUG_ON(snd_interval_empty(i))) return -EINVAL; if (snd_interval_single(i)) return 0; i->max = i->min; - i->openmax = i->openmin; - if (i->openmax) + if (i->openmin) i->max++; + /* only exclude max value if also excluded before refine */ + i->openmax = (i->openmax && i->max >= last_max); return 1; } static int snd_interval_refine_last(struct snd_interval *i) { + const unsigned int last_min = i->min; + if (snd_BUG_ON(snd_interval_empty(i))) return -EINVAL; if (snd_interval_single(i)) return 0; i->min = i->max; - i->openmin = i->openmax; - if (i->openmin) + if (i->openmax) i->min--; + /* only exclude min value if also excluded before refine */ + i->openmin = (i->openmin && i->min <= last_min); return 1; } -- 2.7.4

2 1

[alsa-devel] [PATCH v5 0/9] soundwire: Add multi link support
by Shreyas NC 11 Jul '18

11 Jul '18

Currently, in the SoundWire subsystem, the concept of stream is limited to a Master and one or more Slaves(Codecs). This series aims to add support for multiple Master(s) sharing the same reference clock and synchronized in the hardware. This patch series adds: - Fix to avoid duplicate stream state assignment - Avoid incorrect stream release after configuring Master - Helpers to lock bus instances part of the stream - Boiler plate conversion of code to support a list of Master runtime - Support multi link bank switch to support synchronization between multiple masters - Add Intel platform ops for pre/post bank switch changes in v5: - Add patch to fix duplicate stream state assignment - fix incorrect exit which was releasing the stream after configuring master - In sdw_release_bus_lock(), release the mutex in reverse order as suggested by Takashi changes in v4: - Added changes in sdw_stream_runtime structure to track Masters added to stream - Add changes to support ml_bankswitch only if both the hardware supports and the stream is handled by multiple masters Sanyog Kale (2): Documentation: soundwire: Add documentation for multi link soundwire: Add support to lock across bus instances Shreyas NC (6): soundwire: Fix duplicate stream state assignment soundwire: fix incorrect exit after configuring stream soundwire: Initialize completion for defer messages soundwire: keep track of Masters in a stream soundwire: Add support for multi link bank switch soundwire: intel: Add pre/post bank switch ops Vinod Koul (1): soundwire: Handle multiple master instances in a stream Documentation/driver-api/soundwire/stream.rst | 28 ++ drivers/soundwire/bus.c | 6 + drivers/soundwire/bus.h | 4 + drivers/soundwire/intel.c | 65 ++++ drivers/soundwire/stream.c | 502 +++++++++++++++++++------- include/linux/soundwire/sdw.h | 12 +- 6 files changed, 483 insertions(+), 134 deletions(-) -- 2.7.4

3 18

[alsa-devel] [PATCH] ASoC: Intel: bytcr_rt5640: Add quirk for the Lenovo Miix2 8 tablet
by Hans de Goede 10 Jul '18

10 Jul '18

Add a quirk for the Lenovo Miix2 8 tablet, this tablet uses a digital mic on DMIC1 and has a mono-speaker. The jack-detect uses the default settings.. Reported-and-tested-by: russianneuromancer(a)ya.ru Signed-off-by: Hans de Goede <hdegoede(a)redhat.com> --- sound/soc/intel/boards/bytcr_rt5640.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/sound/soc/intel/boards/bytcr_rt5640.c b/sound/soc/intel/boards/bytcr_rt5640.c index 718f022b3aa0..d32844f94d74 100644 --- a/sound/soc/intel/boards/bytcr_rt5640.c +++ b/sound/soc/intel/boards/bytcr_rt5640.c @@ -562,6 +562,19 @@ static const struct dmi_system_id byt_rt5640_quirk_table[] = { BYT_RT5640_SSP0_AIF1 | BYT_RT5640_MCLK_EN), }, + { /* Lenovo Miix 2 8 */ + .matches = { + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "20326"), + DMI_EXACT_MATCH(DMI_BOARD_NAME, "Hiking"), + }, + .driver_data = (void *)(BYT_RT5640_DMIC1_MAP | + BYT_RT5640_JD_SRC_JD2_IN4N | + BYT_RT5640_OVCD_TH_2000UA | + BYT_RT5640_OVCD_SF_0P75 | + BYT_RT5640_MONO_SPEAKER | + BYT_RT5640_MCLK_EN), + }, { /* MSI S100 tablet */ .matches = { DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Micro-Star International Co., Ltd."), -- 2.17.1

3 2