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

[PATCH v2 0/8] ASoC: soc-pcm cleanup step5
by Kuninori Morimoto 05 Mar '20

05 Mar '20

Hi Mark These are v2 of soc-pcm cleanup step5. These are based on latest for-5.7 branch. Mainly v2 fixed the points reviewed in v1. [7/8] patch might exchange the behavior Kuninori Morimoto (8): 1) ASoC: soc-pcm: use defined stream 2) ASoC: soc-pcm: remove duplicate be check from dpcm_add_paths() 3) ASoC: soc-pcm: move dpcm_fe_dai_close() 4) ASoC: soc-pcm: add dpcm_fe_dai_cleanup() 5) ASoC: soc-pcm: use snd_soc_dai_get_pcm_stream() at dpcm_set_fe_runtime() 6) ASoC: soc-pcm: check DAI's activity more simply 7) ASoC: soc-pcm: Do Digital Mute for both CPU/Codec in same timing. 8) ASoC: soc-pcm: tidyup dulicate handing at dpcm_fe_dai_startup() sound/soc/soc-pcm.c | 114 +++++++++++++++++++------------------------- 1 file changed, 50 insertions(+), 64 deletions(-) -- 2.17.1

2 12

[PATCH] ASoC: wm8741: Fix typo in Kconfig prompt
by Axel Lin 05 Mar '20

05 Mar '20

Fix trivial copy-n-paste mistake. Signed-off-by: Axel Lin <axel.lin(a)ingics.com> --- sound/soc/codecs/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 6aee70ed43df..fd09656d2f6b 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -1486,7 +1486,7 @@ config SND_SOC_WM8737 depends on SND_SOC_I2C_AND_SPI config SND_SOC_WM8741 - tristate "Wolfson Microelectronics WM8737 DAC" + tristate "Wolfson Microelectronics WM8741 DAC" depends on SND_SOC_I2C_AND_SPI config SND_SOC_WM8750 -- 2.20.1

2 2

[PATCH] ASoc: amd: acp3x: Add missing include <linux/io.h>
by YueHaibing 05 Mar '20

05 Mar '20

gcc 7.4.0 build fails: In file included from sound/soc/amd/acp3x-rt5682-max9836.c:20:0: sound/soc/amd/raven/acp3x.h: In function rv_readl: sound/soc/amd/raven/acp3x.h:113:9: error: implicit declaration of function readl; did you mean rv_readl? [-Werror=implicit-function-declaration] return readl(base_addr - ACP3x_PHY_BASE_ADDRESS); ^~~~~ rv_readl sound/soc/amd/raven/acp3x.h: In function rv_writel: sound/soc/amd/raven/acp3x.h:118:2: error: implicit declaration of function writel; did you mean rv_writel? [-Werror=implicit-function-declaration] writel(val, base_addr - ACP3x_PHY_BASE_ADDRESS); ^~~~~~ rv_writel Add <linux/io.h> to fix this. Reported-by: Hulk Robot <hulkci(a)huawei.com> Fixes: 6b8e4e7db3cd ("ASoC: amd: Add machine driver for Raven based platform") Signed-off-by: YueHaibing <yuehaibing(a)huawei.com> --- sound/soc/amd/acp3x-rt5682-max9836.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/amd/acp3x-rt5682-max9836.c b/sound/soc/amd/acp3x-rt5682-max9836.c index 511b8b1..b4f68c5 100644 --- a/sound/soc/amd/acp3x-rt5682-max9836.c +++ b/sound/soc/amd/acp3x-rt5682-max9836.c @@ -16,6 +16,7 @@ #include <linux/module.h> #include <linux/i2c.h> #include <linux/input.h> +#include <linux/io.h> #include <linux/acpi.h> #include "raven/acp3x.h" -- 2.7.4

3 2

[PATCH] ASoC: Fix SND_SOC_ALL_CODECS imply ac97 fallout
by Geert Uytterhoeven 05 Mar '20

05 Mar '20

On i386 randconfig: sound/soc/codecs/wm9705.o: In function `wm9705_soc_resume': wm9705.c:(.text+0x128): undefined reference to `snd_ac97_reset' sound/soc/codecs/wm9712.o: In function `wm9712_soc_resume': wm9712.c:(.text+0x2d1): undefined reference to `snd_ac97_reset' sound/soc/codecs/wm9713.o: In function `wm9713_soc_resume': wm9713.c:(.text+0x820): undefined reference to `snd_ac97_reset' Fix this by adding the missing dependencies on SND_SOC_AC97_BUS. Reported-by: Randy Dunlap <rdunlap(a)infradead.org> Signed-off-by: Geert Uytterhoeven <geert(a)linux-m68k.org> --- Before commit ea00d95200d02ece ("ASoC: Use imply for SND_SOC_ALL_CODECS"), SND_SOC_ALL_CODECS used: select SND_SOC_WM9705 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) select SND_SOC_WM9712 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) select SND_SOC_WM9713 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) but SND_SOC_AC97_BUS_NEW never existed in upstream. Should there be another dependency> See also "non-existent SND_SOC_AC97_BUS_NEW (was: Re: [PATCH v9] ASoC: pxa: switch to new ac97 bus support)" http://lore.kernel.org/r/CAMuHMdU3uxfBwKd8SkOtZSDV5Ai3CKc3CWRhDy0Cz94T1Hn0i… --- sound/soc/codecs/Kconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 9e9d54e4576ce5ba..a7e89567edbe8b47 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -1610,16 +1610,19 @@ config SND_SOC_WM9090 config SND_SOC_WM9705 tristate + depends on SND_SOC_AC97_BUS select REGMAP_AC97 select AC97_BUS_COMPAT if AC97_BUS_NEW config SND_SOC_WM9712 tristate + depends on SND_SOC_AC97_BUS select REGMAP_AC97 select AC97_BUS_COMPAT if AC97_BUS_NEW config SND_SOC_WM9713 tristate + depends on SND_SOC_AC97_BUS select REGMAP_AC97 select AC97_BUS_COMPAT if AC97_BUS_NEW -- 2.17.1

5 6

Re: [PATCH v4 00/10] PCI: Add and use constant PCI_STATUS_ERROR_BITS and helper pci_status_get_and_clear_errors
by David Miller 04 Mar '20

04 Mar '20

Bjorn, please review and let me know if it is OK to merge this via the networking tree. Thank you.

2 2

Problematic motherboard: Supermicro MBD-X11SAT-F-O
by D. R. Evans 04 Mar '20

04 Mar '20

Where is the correct place to post detailed information about a motherboard that provides only unacceptably low volume with ALSA? I realise that the correct place probably isn't here, but I didn't know where else to ask for a pointer to where I should send the detailed information. I initially raised the problem on alsa-users: https://sourceforge.net/p/alsa/mailman/alsa-user/thread/db705c61-87ba-8d66-… but the suggestions made there led to no progress being made, and I am now as sure as I can be that the issue is not user error. The motherboard is a Supermicro MBD-X11SAT-F-O, and the basic high-level device information is: --------- card 0: PCH [HDA Intel PCH], device 0: ALC888-VD Analog [ALC888-VD Analog] Subdevices: 0/1 Subdevice #0: subdevice #0 --------- Doc Evans -- Web: http://enginehousebooks.com/drevans

1 0

[PATCH] ASoC: tas571x: use symbolic register offsets
by Hermann Lauer 04 Mar '20

04 Mar '20

convert the numeric register offsets in tas5711_reg_defaults to the defined constants TAS571X_* from tas571x.h Tested with a tas5713 chip for which the driver works, too. Signed-off-by: Hermann Lauer <Hermann.Lauer(a)iwr.uni-heidelberg.de> diff --git a/sound/soc/codecs/tas571x.c b/sound/soc/codecs/tas571x.c --- a/sound/soc/codecs/tas571x.c +++ b/sound/soc/codecs/tas571x.c @@ -424,13 +424,13 @@ }; static const struct reg_default tas5711_reg_defaults[] = { - { 0x04, 0x05 }, - { 0x05, 0x40 }, - { 0x06, 0x00 }, - { 0x07, 0xff }, - { 0x08, 0x30 }, - { 0x09, 0x30 }, - { 0x1b, 0x82 }, + { TAS571X_SDI_REG, 0x05 }, + { TAS571X_SYS_CTRL_2_REG, 0x40 }, + { TAS571X_SOFT_MUTE_REG, 0x00 }, + { TAS571X_MVOL_REG, 0xff }, + { TAS571X_CH1_VOL_REG, 0x30 }, + { TAS571X_CH2_VOL_REG, 0x30 }, + { TAS571X_OSC_TRIM_REG, 0x82 }, }; static const struct regmap_config tas5711_regmap_config = {

1 0

Re: [PATCH v4 09/10] PCI: pci-bridge-emul: Use new constant PCI_STATUS_ERROR_BITS
by Bjorn Helgaas 04 Mar '20

04 Mar '20

On Sat, Feb 29, 2020 at 11:28:18PM +0100, Heiner Kallweit wrote: > Use new constant PCI_STATUS_ERROR_BITS to simplify the code. > > Signed-off-by: Heiner Kallweit <hkallweit1(a)gmail.com> Acked-by: Bjorn Helgaas <bhelgaas(a)google.com> > --- > drivers/pci/pci-bridge-emul.c | 14 ++------------ > 1 file changed, 2 insertions(+), 12 deletions(-) > > diff --git a/drivers/pci/pci-bridge-emul.c b/drivers/pci/pci-bridge-emul.c > index fffa77093..4f4f54bc7 100644 > --- a/drivers/pci/pci-bridge-emul.c > +++ b/drivers/pci/pci-bridge-emul.c > @@ -50,12 +50,7 @@ static const struct pci_bridge_reg_behavior pci_regs_behavior[] = { > (PCI_STATUS_CAP_LIST | PCI_STATUS_66MHZ | > PCI_STATUS_FAST_BACK | PCI_STATUS_DEVSEL_MASK) << 16), > .rsvd = GENMASK(15, 10) | ((BIT(6) | GENMASK(3, 0)) << 16), > - .w1c = (PCI_STATUS_PARITY | > - PCI_STATUS_SIG_TARGET_ABORT | > - PCI_STATUS_REC_TARGET_ABORT | > - PCI_STATUS_REC_MASTER_ABORT | > - PCI_STATUS_SIG_SYSTEM_ERROR | > - PCI_STATUS_DETECTED_PARITY) << 16, > + .w1c = PCI_STATUS_ERROR_BITS << 16, > }, > [PCI_CLASS_REVISION / 4] = { .ro = ~0 }, > > @@ -100,12 +95,7 @@ static const struct pci_bridge_reg_behavior pci_regs_behavior[] = { > PCI_STATUS_DEVSEL_MASK) << 16) | > GENMASK(11, 8) | GENMASK(3, 0)), > > - .w1c = (PCI_STATUS_PARITY | > - PCI_STATUS_SIG_TARGET_ABORT | > - PCI_STATUS_REC_TARGET_ABORT | > - PCI_STATUS_REC_MASTER_ABORT | > - PCI_STATUS_SIG_SYSTEM_ERROR | > - PCI_STATUS_DETECTED_PARITY) << 16, > + .w1c = PCI_STATUS_ERROR_BITS << 16, > > .rsvd = ((BIT(6) | GENMASK(4, 0)) << 16), > }, > -- > 2.25.1 > >

1 0

Re: [PATCH v4 05/10] PCI: Add pci_status_get_and_clear_errors
by Bjorn Helgaas 04 Mar '20

04 Mar '20

On Sat, Feb 29, 2020 at 11:24:23PM +0100, Heiner Kallweit wrote: > Several drivers use the following code sequence: > 1. Read PCI_STATUS > 2. Mask out non-error bits > 3. Action based on error bits set > 4. Write back set error bits to clear them > > As this is a repeated pattern, add a helper to the PCI core. > > Signed-off-by: Heiner Kallweit <hkallweit1(a)gmail.com> Acked-by: Bjorn Helgaas <bhelgaas(a)google.com> Nit: if you have any reason to revise this, I would prefer "pci_status_get_and_clear_errors()" (with parens) in the subject. > --- > v4: > - improve commit description > --- > drivers/pci/pci.c | 23 +++++++++++++++++++++++ > include/linux/pci.h | 1 + > 2 files changed, 24 insertions(+) > > diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c > index d828ca835..c16b0ba2a 100644 > --- a/drivers/pci/pci.c > +++ b/drivers/pci/pci.c > @@ -173,6 +173,29 @@ unsigned char pci_bus_max_busnr(struct pci_bus *bus) > } > EXPORT_SYMBOL_GPL(pci_bus_max_busnr); > > +/** > + * pci_status_get_and_clear_errors - return and clear error bits in PCI_STATUS > + * @pdev: the PCI device > + * > + * Returns error bits set in PCI_STATUS and clears them. > + */ > +int pci_status_get_and_clear_errors(struct pci_dev *pdev) > +{ > + u16 status; > + int ret; > + > + ret = pci_read_config_word(pdev, PCI_STATUS, &status); > + if (ret != PCIBIOS_SUCCESSFUL) > + return -EIO; > + > + status &= PCI_STATUS_ERROR_BITS; > + if (status) > + pci_write_config_word(pdev, PCI_STATUS, status); > + > + return status; > +} > +EXPORT_SYMBOL_GPL(pci_status_get_and_clear_errors); > + > #ifdef CONFIG_HAS_IOMEM > void __iomem *pci_ioremap_bar(struct pci_dev *pdev, int bar) > { > diff --git a/include/linux/pci.h b/include/linux/pci.h > index 101d71e0a..7beaf51e9 100644 > --- a/include/linux/pci.h > +++ b/include/linux/pci.h > @@ -1210,6 +1210,7 @@ int pci_select_bars(struct pci_dev *dev, unsigned long flags); > bool pci_device_is_present(struct pci_dev *pdev); > void pci_ignore_hotplug(struct pci_dev *dev); > struct pci_dev *pci_real_dma_dev(struct pci_dev *dev); > +int pci_status_get_and_clear_errors(struct pci_dev *pdev); > > int __printf(6, 7) pci_request_irq(struct pci_dev *dev, unsigned int nr, > irq_handler_t handler, irq_handler_t thread_fn, void *dev_id, > -- > 2.25.1 > >

1 0

Re: [PATCH v4 04/10] PCI: Add constant PCI_STATUS_ERROR_BITS
by Bjorn Helgaas 04 Mar '20

04 Mar '20

On Sat, Feb 29, 2020 at 11:23:44PM +0100, Heiner Kallweit wrote: > This collection of PCI error bits is used in more than one driver, > so move it to the PCI core. > > Signed-off-by: Heiner Kallweit <hkallweit1(a)gmail.com> Acked-by: Bjorn Helgaas <bhelgaas(a)google.com> > --- > v4: > - move new constant to include/linux/pci.h > - improve commit description > --- > drivers/net/ethernet/marvell/skge.h | 7 ------- > drivers/net/ethernet/marvell/sky2.h | 7 ------- > include/linux/pci.h | 7 +++++++ > 3 files changed, 7 insertions(+), 14 deletions(-) > > diff --git a/drivers/net/ethernet/marvell/skge.h b/drivers/net/ethernet/marvell/skge.h > index e76c03c87..e149bdfe1 100644 > --- a/drivers/net/ethernet/marvell/skge.h > +++ b/drivers/net/ethernet/marvell/skge.h > @@ -15,13 +15,6 @@ > #define PCI_VPD_ROM_SZ 7L<<14 /* VPD ROM size 0=256, 1=512, ... */ > #define PCI_REV_DESC 1<<2 /* Reverse Descriptor bytes */ > > -#define PCI_STATUS_ERROR_BITS (PCI_STATUS_DETECTED_PARITY | \ > - PCI_STATUS_SIG_SYSTEM_ERROR | \ > - PCI_STATUS_REC_MASTER_ABORT | \ > - PCI_STATUS_REC_TARGET_ABORT | \ > - PCI_STATUS_SIG_TARGET_ABORT | \ > - PCI_STATUS_PARITY) > - > enum csr_regs { > B0_RAP = 0x0000, > B0_CTST = 0x0004, > diff --git a/drivers/net/ethernet/marvell/sky2.h b/drivers/net/ethernet/marvell/sky2.h > index aee87f838..851d8ed34 100644 > --- a/drivers/net/ethernet/marvell/sky2.h > +++ b/drivers/net/ethernet/marvell/sky2.h > @@ -252,13 +252,6 @@ enum { > }; > > > -#define PCI_STATUS_ERROR_BITS (PCI_STATUS_DETECTED_PARITY | \ > - PCI_STATUS_SIG_SYSTEM_ERROR | \ > - PCI_STATUS_REC_MASTER_ABORT | \ > - PCI_STATUS_REC_TARGET_ABORT | \ > - PCI_STATUS_SIG_TARGET_ABORT | \ > - PCI_STATUS_PARITY) > - > enum csr_regs { > B0_RAP = 0x0000, > B0_CTST = 0x0004, > diff --git a/include/linux/pci.h b/include/linux/pci.h > index 3840a541a..101d71e0a 100644 > --- a/include/linux/pci.h > +++ b/include/linux/pci.h > @@ -42,6 +42,13 @@ > > #include <linux/pci_ids.h> > > +#define PCI_STATUS_ERROR_BITS (PCI_STATUS_DETECTED_PARITY | \ > + PCI_STATUS_SIG_SYSTEM_ERROR | \ > + PCI_STATUS_REC_MASTER_ABORT | \ > + PCI_STATUS_REC_TARGET_ABORT | \ > + PCI_STATUS_SIG_TARGET_ABORT | \ > + PCI_STATUS_PARITY) > + > /* > * The PCI interface treats multi-function devices as independent > * devices. The slot/function address of each device is encoded > -- > 2.25.1 > >

1 0