April 2015 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH 00/14] Fix fallout from per-user struct clk patches
by Russell King - ARM Linux 07 Apr '15

07 Apr '15

Sorry for posting this soo close to the 4.1 merge window, I had completely forgotten about this chunk of work I did earlier this month. The per-user struct clk patches rather badly broke clkdev and various other places. This was reported, but was forgotten about. Really, the per-user clk stuff should've been reverted, but we've lived with it far too long for that. So, our only other option is to now rush these patches into 4.1 and hope for the best. The series cleans up quite a number of places too... arch/arm/mach-davinci/da850.c | 1 + arch/arm/mach-lpc32xx/clock.c | 5 +- arch/arm/mach-omap1/board-nokia770.c | 2 +- arch/arm/mach-omap2/clkt2xxx_virt_prcm_set.c | 12 +--- arch/arm/mach-omap2/omap_device.c | 24 +++----- arch/arm/mach-pxa/eseries.c | 1 + arch/arm/mach-pxa/lubbock.c | 1 + arch/arm/mach-pxa/tosa.c | 1 + arch/arm/plat-orion/common.c | 6 +- arch/sh/kernel/cpu/sh4a/clock-sh7734.c | 3 +- arch/sh/kernel/cpu/sh4a/clock-sh7757.c | 4 +- arch/sh/kernel/cpu/sh4a/clock-sh7785.c | 4 +- arch/sh/kernel/cpu/sh4a/clock-sh7786.c | 4 +- arch/sh/kernel/cpu/sh4a/clock-shx3.c | 4 +- drivers/clk/clk-s2mps11.c | 4 +- drivers/clk/clkdev.c | 83 ++++++++++++++++++++-------- drivers/media/platform/omap3isp/isp.c | 18 ------ drivers/media/platform/omap3isp/isp.h | 1 - include/linux/clk.h | 27 ++++----- include/linux/clkdev.h | 6 +- include/media/omap3isp.h | 6 -- sound/soc/sh/migor.c | 3 +- 22 files changed, 108 insertions(+), 112 deletions(-) -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net.

10 39

[alsa-devel] [PATCH] ALSA: hda_intel: add AZX_DCAPS_I915_POWERWELL for SKL and BSW
by libin.yang＠intel.com 07 Apr '15

07 Apr '15

From: Libin Yang <libin.yang(a)intel.com> HDMI/DP codec on SKL/BSW is in the power well. The power well must be turned on before probing the HDMI/DP codec. This is a temporary patch, which will power on the powerwell by adding AZX_DCAPS_I915_POWERWELL for SKL and BSW. After restructuring and new flag is added, this patch will be reverted. Signed-off-by: Libin Yang <libin.yang(a)intel.com> --- sound/pci/hda/hda_intel.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 15a8299..e1c2105 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -297,8 +297,12 @@ enum { AZX_DCAPS_PM_RUNTIME | AZX_DCAPS_I915_POWERWELL |\ AZX_DCAPS_SNOOP_TYPE(SCH)) +#define AZX_DCAPS_INTEL_BRASWELL \ + (AZX_DCAPS_INTEL_PCH | AZX_DCAPS_I915_POWERWELL) + #define AZX_DCAPS_INTEL_SKYLAKE \ - (AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG) + (AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG |\ + AZX_DCAPS_I915_POWERWELL) /* quirks for ATI SB / AMD Hudson */ #define AZX_DCAPS_PRESET_ATI_SB \ @@ -1991,7 +1995,7 @@ static const struct pci_device_id azx_ids[] = { .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH_NOPM }, /* Braswell */ { PCI_DEVICE(0x8086, 0x2284), - .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH }, + .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_BRASWELL }, /* ICH6 */ { PCI_DEVICE(0x8086, 0x2668), .driver_data = AZX_DRIVER_ICH | AZX_DCAPS_INTEL_ICH }, -- 1.9.1

2 1

[alsa-devel] [PATCH 1/4] ASoC: wm8804: Add support for hardware reset line
by Charles Keepax 07 Apr '15

07 Apr '15

It is best to use the physical reset if it is available. This patch adds support for a GPIO controlled physical reset for the chip. Signed-off-by: Charles Keepax <ckeepax(a)opensource.wolfsonmicro.com> --- sound/soc/codecs/wm8804.c | 26 +++++++++++++++++++++----- 1 files changed, 21 insertions(+), 5 deletions(-) diff --git a/sound/soc/codecs/wm8804.c b/sound/soc/codecs/wm8804.c index f44da83..cc168c4 100644 --- a/sound/soc/codecs/wm8804.c +++ b/sound/soc/codecs/wm8804.c @@ -13,6 +13,7 @@ #include <linux/module.h> #include <linux/moduleparam.h> #include <linux/init.h> +#include <linux/gpio/consumer.h> #include <linux/delay.h> #include <linux/pm.h> #include <linux/of_device.h> @@ -61,6 +62,8 @@ struct wm8804_priv { struct regulator_bulk_data supplies[WM8804_NUM_SUPPLIES]; struct notifier_block disable_nb[WM8804_NUM_SUPPLIES]; int mclk_div; + + struct gpio_desc *reset; }; static int txsrc_get(struct snd_kcontrol *kcontrol, @@ -182,7 +185,7 @@ static bool wm8804_volatile(struct device *dev, unsigned int reg) } } -static int wm8804_reset(struct wm8804_priv *wm8804) +static int wm8804_soft_reset(struct wm8804_priv *wm8804) { return regmap_write(wm8804->regmap, WM8804_RST_DEVID1, 0x0); } @@ -586,6 +589,14 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) wm8804->regmap = regmap; + wm8804->reset = devm_gpiod_get_optional(dev, "wlf,reset", + GPIOD_OUT_LOW); + if (IS_ERR(wm8804->reset)) { + ret = PTR_ERR(wm8804->reset); + dev_err(dev, "Failed to get reset line: %d\n", ret); + return ret; + } + for (i = 0; i < ARRAY_SIZE(wm8804->supplies); i++) wm8804->supplies[i].supply = wm8804_supply_names[i]; @@ -620,6 +631,9 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) return ret; } + if (wm8804->reset) + gpiod_set_value_cansleep(wm8804->reset, 1); + ret = regmap_read(regmap, WM8804_RST_DEVID1, &id1); if (ret < 0) { dev_err(dev, "Failed to read device ID: %d\n", ret); @@ -648,10 +662,12 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) } dev_info(dev, "revision %c\n", id1 + 'A'); - ret = wm8804_reset(wm8804); - if (ret < 0) { - dev_err(dev, "Failed to issue reset: %d\n", ret); - goto err_reg_enable; + if (!wm8804->reset) { + ret = wm8804_soft_reset(wm8804); + if (ret < 0) { + dev_err(dev, "Failed to issue reset: %d\n", ret); + goto err_reg_enable; + } } ret = snd_soc_register_codec(dev, &soc_codec_dev_wm8804, -- 1.7.2.5

2 5

[alsa-devel] [PATCH v2 1/4] ASoC: wm8804: Add support for hardware reset line
by Charles Keepax 07 Apr '15

07 Apr '15

It is best to use the physical reset if it is available. This patch adds support for a GPIO controlled physical reset for the chip. Signed-off-by: Charles Keepax <ckeepax(a)opensource.wolfsonmicro.com> --- Changes since v1: - Corrected email address on two of the patches Thanks, Charles sound/soc/codecs/wm8804.c | 26 +++++++++++++++++++++----- 1 files changed, 21 insertions(+), 5 deletions(-) diff --git a/sound/soc/codecs/wm8804.c b/sound/soc/codecs/wm8804.c index f44da83..cc168c4 100644 --- a/sound/soc/codecs/wm8804.c +++ b/sound/soc/codecs/wm8804.c @@ -13,6 +13,7 @@ #include <linux/module.h> #include <linux/moduleparam.h> #include <linux/init.h> +#include <linux/gpio/consumer.h> #include <linux/delay.h> #include <linux/pm.h> #include <linux/of_device.h> @@ -61,6 +62,8 @@ struct wm8804_priv { struct regulator_bulk_data supplies[WM8804_NUM_SUPPLIES]; struct notifier_block disable_nb[WM8804_NUM_SUPPLIES]; int mclk_div; + + struct gpio_desc *reset; }; static int txsrc_get(struct snd_kcontrol *kcontrol, @@ -182,7 +185,7 @@ static bool wm8804_volatile(struct device *dev, unsigned int reg) } } -static int wm8804_reset(struct wm8804_priv *wm8804) +static int wm8804_soft_reset(struct wm8804_priv *wm8804) { return regmap_write(wm8804->regmap, WM8804_RST_DEVID1, 0x0); } @@ -586,6 +589,14 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) wm8804->regmap = regmap; + wm8804->reset = devm_gpiod_get_optional(dev, "wlf,reset", + GPIOD_OUT_LOW); + if (IS_ERR(wm8804->reset)) { + ret = PTR_ERR(wm8804->reset); + dev_err(dev, "Failed to get reset line: %d\n", ret); + return ret; + } + for (i = 0; i < ARRAY_SIZE(wm8804->supplies); i++) wm8804->supplies[i].supply = wm8804_supply_names[i]; @@ -620,6 +631,9 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) return ret; } + if (wm8804->reset) + gpiod_set_value_cansleep(wm8804->reset, 1); + ret = regmap_read(regmap, WM8804_RST_DEVID1, &id1); if (ret < 0) { dev_err(dev, "Failed to read device ID: %d\n", ret); @@ -648,10 +662,12 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) } dev_info(dev, "revision %c\n", id1 + 'A'); - ret = wm8804_reset(wm8804); - if (ret < 0) { - dev_err(dev, "Failed to issue reset: %d\n", ret); - goto err_reg_enable; + if (!wm8804->reset) { + ret = wm8804_soft_reset(wm8804); + if (ret < 0) { + dev_err(dev, "Failed to issue reset: %d\n", ret); + goto err_reg_enable; + } } ret = snd_soc_register_codec(dev, &soc_codec_dev_wm8804, -- 1.7.2.5

1 3

[alsa-devel] bug
by Nikita N. 07 Apr '15

07 Apr '15

is any programmer listening here? -- http://www.fastmail.com - A no graphics, no pop-ups email service

10 35

[alsa-devel] [PATCH] ASoC: davinci-mcasp: Fix ruledata setup in davinci_mcasp_startup
by Peter Ujfalusi 07 Apr '15

07 Apr '15

Passing &mcasp->ruledata[dir] to snd_pcm_hw_rule_add() is not correct since commit: 7b3d165a2821 ASoC: davinci-mcasp: Index ruledata in drvdata with substream->stream now sets up the struct based on the substream->stream (0 or 1) while we pass a pointer which we take with dir (1 or 2). This will lead kernel crash. Signed-off-by: Peter Ujfalusi <peter.ujfalusi(a)ti.com> --- sound/soc/davinci/davinci-mcasp.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/sound/soc/davinci/davinci-mcasp.c b/sound/soc/davinci/davinci-mcasp.c index 0b6b1b286201..bb4b78eada58 100644 --- a/sound/soc/davinci/davinci-mcasp.c +++ b/sound/soc/davinci/davinci-mcasp.c @@ -1128,6 +1128,8 @@ static int davinci_mcasp_startup(struct snd_pcm_substream *substream, struct snd_soc_dai *cpu_dai) { struct davinci_mcasp *mcasp = snd_soc_dai_get_drvdata(cpu_dai); + struct davinci_mcasp_ruledata *ruledata = + &mcasp->ruledata[substream->stream]; u32 max_channels = 0; int i, dir; @@ -1149,7 +1151,7 @@ static int davinci_mcasp_startup(struct snd_pcm_substream *substream, if (mcasp->serial_dir[i] == dir) max_channels++; } - mcasp->ruledata[substream->stream].serializers = max_channels; + ruledata->serializers = max_channels; max_channels *= mcasp->tdm_slots; /* * If the already active stream has less channels than the calculated @@ -1172,12 +1174,12 @@ static int davinci_mcasp_startup(struct snd_pcm_substream *substream, if (mcasp->bclk_master && mcasp->bclk_div == 0 && mcasp->sysclk_freq) { int ret; - mcasp->ruledata[substream->stream].mcasp = mcasp; + ruledata->mcasp = mcasp; ret = snd_pcm_hw_rule_add(substream->runtime, 0, SNDRV_PCM_HW_PARAM_RATE, davinci_mcasp_hw_rule_rate, - &mcasp->ruledata[dir], + ruledata, SNDRV_PCM_HW_PARAM_FORMAT, SNDRV_PCM_HW_PARAM_CHANNELS, -1); if (ret) @@ -1185,7 +1187,7 @@ static int davinci_mcasp_startup(struct snd_pcm_substream *substream, ret = snd_pcm_hw_rule_add(substream->runtime, 0, SNDRV_PCM_HW_PARAM_FORMAT, davinci_mcasp_hw_rule_format, - &mcasp->ruledata[dir], + ruledata, SNDRV_PCM_HW_PARAM_RATE, SNDRV_PCM_HW_PARAM_CHANNELS, -1); if (ret) @@ -1193,7 +1195,7 @@ static int davinci_mcasp_startup(struct snd_pcm_substream *substream, ret = snd_pcm_hw_rule_add(substream->runtime, 0, SNDRV_PCM_HW_PARAM_CHANNELS, davinci_mcasp_hw_rule_channels, - &mcasp->ruledata[dir], + ruledata, SNDRV_PCM_HW_PARAM_RATE, SNDRV_PCM_HW_PARAM_FORMAT, -1); if (ret) -- 2.3.5

2 1

[alsa-devel] [PATCH 1/4] ASoC: wm8804: Add support for hardware reset line
by Charles Keepax 07 Apr '15

07 Apr '15

It is best to use the physical reset if it is available. This patch adds support for a GPIO controlled physical reset for the chip. Signed-off-by: Charles Keepax <ckeepax(a)opensource.wolfsonmicro.com> --- sound/soc/codecs/wm8804.c | 26 +++++++++++++++++++++----- 1 files changed, 21 insertions(+), 5 deletions(-) diff --git a/sound/soc/codecs/wm8804.c b/sound/soc/codecs/wm8804.c index f44da83..cc168c4 100644 --- a/sound/soc/codecs/wm8804.c +++ b/sound/soc/codecs/wm8804.c @@ -13,6 +13,7 @@ #include <linux/module.h> #include <linux/moduleparam.h> #include <linux/init.h> +#include <linux/gpio/consumer.h> #include <linux/delay.h> #include <linux/pm.h> #include <linux/of_device.h> @@ -61,6 +62,8 @@ struct wm8804_priv { struct regulator_bulk_data supplies[WM8804_NUM_SUPPLIES]; struct notifier_block disable_nb[WM8804_NUM_SUPPLIES]; int mclk_div; + + struct gpio_desc *reset; }; static int txsrc_get(struct snd_kcontrol *kcontrol, @@ -182,7 +185,7 @@ static bool wm8804_volatile(struct device *dev, unsigned int reg) } } -static int wm8804_reset(struct wm8804_priv *wm8804) +static int wm8804_soft_reset(struct wm8804_priv *wm8804) { return regmap_write(wm8804->regmap, WM8804_RST_DEVID1, 0x0); } @@ -586,6 +589,14 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) wm8804->regmap = regmap; + wm8804->reset = devm_gpiod_get_optional(dev, "wlf,reset", + GPIOD_OUT_LOW); + if (IS_ERR(wm8804->reset)) { + ret = PTR_ERR(wm8804->reset); + dev_err(dev, "Failed to get reset line: %d\n", ret); + return ret; + } + for (i = 0; i < ARRAY_SIZE(wm8804->supplies); i++) wm8804->supplies[i].supply = wm8804_supply_names[i]; @@ -620,6 +631,9 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) return ret; } + if (wm8804->reset) + gpiod_set_value_cansleep(wm8804->reset, 1); + ret = regmap_read(regmap, WM8804_RST_DEVID1, &id1); if (ret < 0) { dev_err(dev, "Failed to read device ID: %d\n", ret); @@ -648,10 +662,12 @@ int wm8804_probe(struct device *dev, struct regmap *regmap) } dev_info(dev, "revision %c\n", id1 + 'A'); - ret = wm8804_reset(wm8804); - if (ret < 0) { - dev_err(dev, "Failed to issue reset: %d\n", ret); - goto err_reg_enable; + if (!wm8804->reset) { + ret = wm8804_soft_reset(wm8804); + if (ret < 0) { + dev_err(dev, "Failed to issue reset: %d\n", ret); + goto err_reg_enable; + } } ret = snd_soc_register_codec(dev, &soc_codec_dev_wm8804, -- 1.7.2.5

2 5

[alsa-devel] [PATCH] ALSA: hda_intel: add AZX_DCAPS_I915_POWERWELL for skl
by libin.yang＠intel.com 07 Apr '15

07 Apr '15

From: Libin Yang <libin.yang(a)intel.com> HDMI/DP codec on SKL is in the power well. The power well must be turned on before probing the HDMI/DP codec. Signed-off-by: Libin Yang <libin.yang(a)intel.com> --- sound/pci/hda/hda_intel.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 15a8299..4744a96 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -298,7 +298,8 @@ enum { AZX_DCAPS_SNOOP_TYPE(SCH)) #define AZX_DCAPS_INTEL_SKYLAKE \ - (AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG) + (AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG |\ + AZX_DCAPS_I915_POWERWELL) /* quirks for ATI SB / AMD Hudson */ #define AZX_DCAPS_PRESET_ATI_SB \ -- 1.9.1

4 19

[alsa-devel] steinberg ur44 - soundcard not connecting, acl problem (systemd?)
by James Miller 06 Apr '15

06 Apr '15

Hi, I've just bought a steinberg ur44. It is reported as having worked in an earlier kernel to the 3.18 rt custom/3.19 low latency generic that I am using. The low latency kernel is generic but the realtime is compiled with the alsa quirks as described here: http://mailman.alsa-project.org/pipermail/alsa-devel/2014-March/074120.html On plugging it in, the card is recognised by udev but is disconnected after seemingly being set up. The same error seems to be occurring when I start the machine with the card plugged in. There seems to be an ACL error reported in journalctl. I include the appropriate lines from the udevadm and from journalctl udevadm: sudo udevadm monitor monitor will print the received events for: UDEV - the event which udev sends out after rule processing KERNEL - the kernel uevent KERNEL[2741.779546] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5 (usb) KERNEL[2741.779705] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.0 (usb) KERNEL[2741.781720] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1 (usb) KERNEL[2741.787500] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3 (sound) KERNEL[2741.787611] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/controlC3 (sound) KERNEL[2741.787813] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0p (sound) KERNEL[2741.787933] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0c (sound) KERNEL[2741.788073] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midiC3D0 (sound) KERNEL[2741.788189] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midi3 (sound) KERNEL[2741.789228] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/dmmidi3 (sound) KERNEL[2741.789294] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midi3 (sound) KERNEL[2741.789315] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/dmmidi3 (sound) KERNEL[2741.789365] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midiC3D0 (sound) KERNEL[2741.789418] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0p (sound) KERNEL[2741.789436] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0c (sound) KERNEL[2741.789487] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/controlC3 (sound) KERNEL[2741.789503] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3 (sound) KERNEL[2741.789556] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.2 (usb) KERNEL[2741.789577] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.3 (usb) KERNEL[2741.790043] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.4 (usb) UDEV [2741.810128] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5 (usb) UDEV [2741.810172] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.3 (usb) UDEV [2741.810195] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.2 (usb) UDEV [2741.810217] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1 (usb) UDEV [2741.810237] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3 (sound) UDEV [2741.810260] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0c (sound) UDEV [2741.810279] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0c (sound) UDEV [2741.810301] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.4 (usb) UDEV [2741.815664] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0p (sound) UDEV [2741.815903] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/pcmC3D0p (sound) UDEV [2741.820639] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midi3 (sound) UDEV [2741.821530] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midi3 (sound) UDEV [2741.823891] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/dmmidi3 (sound) UDEV [2741.823931] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/dmmidi3 (sound) UDEV [2741.826596] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midiC3D0 (sound) UDEV [2741.827398] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/midiC3D0 (sound) UDEV [2741.837736] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/controlC3 (sound) UDEV [2741.839478] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/controlC3 (sound) UDEV [2741.840171] remove /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3 (sound) UDEV [2742.842156] add /devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.0 (usb) journactl (on hotplug add): Apr 06 19:26:09 super-R720 kernel: usb 1-5: USB disconnect, device number 4 Apr 06 19:26:09 super-R720 systemd-udevd[309]: error opening USB device 'descriptors' file Apr 06 19:26:11 super-R720 kernel: usb 1-5: new high-speed USB device number 5 using ehci-pci Apr 06 19:26:12 super-R720 kernel: usb 1-5: New USB device found, idVendor=0499, idProduct=150e Apr 06 19:26:12 super-R720 kernel: usb 1-5: New USB device strings: Mfr=1, Product=2, SerialNumber=0 Apr 06 19:26:12 super-R720 kernel: usb 1-5: Product: Steinberg UR44 Apr 06 19:26:12 super-R720 kernel: usb 1-5: Manufacturer: Yamaha Corporation Apr 06 19:26:12 super-R720 kernel: snd-usb-audio: probe of 1-5:1.1 failed with error -16 Apr 06 19:26:12 super-R720 mtp-probe[3550]: checking bus 1, device 5: "/sys/devices/pci0000:00/0000:00:1a.7/usb1/1-5" Apr 06 19:26:12 super-R720 mtp-probe[3550]: bus: 1, device: 5 was not an MTP device Apr 06 19:26:12 super-R720 systemd-udevd[3547]: error opening ATTR{/sys/devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/controlC3/ Apr 06 19:26:12 super-R720 systemd-udevd[3560]: Failed to apply ACL on /dev/dmmidi3: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3560]: Failed to apply ACL on /dev/dmmidi3: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3559]: Failed to apply ACL on /dev/midi3: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3559]: Failed to apply ACL on /dev/midi3: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3558]: Failed to apply ACL on /dev/snd/midiC3D0: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3558]: Failed to apply ACL on /dev/snd/midiC3D0: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3549]: Failed to apply ACL on /dev/snd/pcmC3D0c: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3549]: Failed to apply ACL on /dev/snd/pcmC3D0c: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3548]: Failed to apply ACL on /dev/snd/pcmC3D0p: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3548]: Failed to apply ACL on /dev/snd/pcmC3D0p: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3547]: Failed to apply ACL on /dev/snd/controlC3: No such file or directory Apr 06 19:26:12 super-R720 systemd-udevd[3547]: Failed to apply ACL on /dev/snd/controlC3: No such file or directory journalctl -b: systemd-udevd[416]: error opening ATTR{/sys/devices/pci0000:00/0000:00:1a.7/usb1/1-5/1-5:1.1/sound/card3/controlC3/. I can compile a kernel reasonably well, and can understand and program in c, c++, but am not well versed in lower level usb hardware coding. I'd be grateful for any help, many thanks! Many thanks -- James Stewart Miller Bsc(hons) Psych.

1 1

[alsa-devel] [asoc:topic/intel 37/37] sound/soc/intel/atom/sst/sst_drv_interface.c:277:19: sparse: cast removes address space of expression
by kbuild test robot 06 Apr '15

06 Apr '15

tree: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/intel head: b97169da06992ef04081e66ed22bbdb23dbf6610 commit: b97169da06992ef04081e66ed22bbdb23dbf6610 [37/37] ASoC: Intel: create atom folder and move atom platform files in reproduce: # apt-get install sparse git checkout b97169da06992ef04081e66ed22bbdb23dbf6610 make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> sound/soc/intel/atom/sst/sst_drv_interface.c:277:19: sparse: cast removes address space of expression >> sound/soc/intel/atom/sst/sst_drv_interface.c:278:17: sparse: incorrect type in argument 2 (different address spaces) sound/soc/intel/atom/sst/sst_drv_interface.c:278:17: expected void const volatile [noderef] <asn:2>*src sound/soc/intel/atom/sst/sst_drv_interface.c:278:17: got void * >> sound/soc/intel/atom/sst/sst_drv_interface.c:285:19: sparse: cast removes address space of expression >> sound/soc/intel/atom/sst/sst_drv_interface.c:285:14: sparse: incorrect type in assignment (different address spaces) sound/soc/intel/atom/sst/sst_drv_interface.c:285:14: expected void [noderef] <asn:2>*addr sound/soc/intel/atom/sst/sst_drv_interface.c:285:14: got void * >> sound/soc/intel/atom/sst/sst_drv_interface.c:372:19: sparse: cast removes address space of expression >> sound/soc/intel/atom/sst/sst_drv_interface.c:373:17: sparse: incorrect type in argument 2 (different address spaces) sound/soc/intel/atom/sst/sst_drv_interface.c:373:17: expected void const volatile [noderef] <asn:2>*src sound/soc/intel/atom/sst/sst_drv_interface.c:373:17: got void * >> sound/soc/intel/atom/sst/sst_drv_interface.c:560:19: sparse: cast removes address space of expression >> sound/soc/intel/atom/sst/sst_drv_interface.c:561:25: sparse: incorrect type in argument 2 (different address spaces) sound/soc/intel/atom/sst/sst_drv_interface.c:561:25: expected void const volatile [noderef] <asn:2>*src sound/soc/intel/atom/sst/sst_drv_interface.c:561:25: got void * -- >> sound/soc/intel/atom/sst/sst_loader.c:55:26: sparse: incorrect type in argument 1 (different address spaces) sound/soc/intel/atom/sst/sst_loader.c:55:26: expected void [noderef] <asn:2>*to sound/soc/intel/atom/sst/sst_loader.c:55:26: got void *dst >> sound/soc/intel/atom/sst/sst_loader.c:55:31: sparse: incorrect type in argument 2 (different address spaces) sound/soc/intel/atom/sst/sst_loader.c:55:31: expected void const *from sound/soc/intel/atom/sst/sst_loader.c:55:31: got void const [noderef] <asn:2>*src >> sound/soc/intel/atom/sst/sst_loader.c:220:43: sparse: incorrect type in argument 2 (different address spaces) sound/soc/intel/atom/sst/sst_loader.c:220:43: expected void *destn sound/soc/intel/atom/sst/sst_loader.c:220:43: got void [noderef] <asn:2>* -- >> sound/soc/intel/atom/sst/sst_pvt.c:114:20: sparse: symbol 'read_shim_data' was not declared. Should it be static? >> sound/soc/intel/atom/sst/sst_pvt.c:127:6: sparse: symbol 'write_shim_data' was not declared. Should it be static? Please review and possibly fold the followup patch. vim +277 sound/soc/intel/atom/sst/sst_drv_interface.c 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 271 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 272 /* update bytes sent */ 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 273 stream->cumm_bytes += bytes; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 274 dev_dbg(dev, "bytes copied %d inc by %ld\n", stream->cumm_bytes, bytes); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 275 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 276 memcpy_fromio(&fw_tstamp, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 @277 ((void *)(ctx->mailbox + ctx->tstamp) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 @278 +(str_id * sizeof(fw_tstamp))), 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 279 sizeof(fw_tstamp)); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 280 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 281 fw_tstamp.bytes_copied = stream->cumm_bytes; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 282 dev_dbg(dev, "bytes sent to fw %llu inc by %ld\n", 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 283 fw_tstamp.bytes_copied, bytes); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 284 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 @285 addr = ((void *)(ctx->mailbox + ctx->tstamp)) + 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 286 (str_id * sizeof(fw_tstamp)); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 287 offset = offsetof(struct snd_sst_tstamp, bytes_copied); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 288 sst_shim_write(addr, offset, fw_tstamp.bytes_copied); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 289 return 0; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 290 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 291 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 292 static int sst_cdev_set_metadata(struct device *dev, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 293 unsigned int str_id, struct snd_compr_metadata *metadata) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 294 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 295 int retval = 0; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 296 struct stream_info *str_info; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 297 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 298 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 299 dev_dbg(dev, "set metadata for stream %d\n", str_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 300 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 301 str_info = get_stream_info(ctx, str_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 302 if (!str_info) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 303 return -EINVAL; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 304 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 305 dev_dbg(dev, "pipe id = %d\n", str_info->pipe_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 306 retval = sst_prepare_and_post_msg(ctx, str_info->task_id, IPC_CMD, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 307 IPC_IA_SET_STREAM_PARAMS_MRFLD, str_info->pipe_id, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 308 sizeof(*metadata), metadata, NULL, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 309 true, true, true, false); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 310 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 311 return retval; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 312 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 313 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 314 static int sst_cdev_stream_pause(struct device *dev, unsigned int str_id) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 315 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 316 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 317 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 318 return sst_pause_stream(ctx, str_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 319 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 320 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 321 static int sst_cdev_stream_pause_release(struct device *dev, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 322 unsigned int str_id) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 323 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 324 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 325 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 326 return sst_resume_stream(ctx, str_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 327 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 328 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 329 static int sst_cdev_stream_start(struct device *dev, unsigned int str_id) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 330 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 331 struct stream_info *str_info; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 332 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 333 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 334 str_info = get_stream_info(ctx, str_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 335 if (!str_info) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 336 return -EINVAL; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 337 str_info->prev = str_info->status; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 338 str_info->status = STREAM_RUNNING; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 339 return sst_start_stream(ctx, str_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 340 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 341 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 342 static int sst_cdev_stream_drop(struct device *dev, unsigned int str_id) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 343 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 344 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 345 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 346 return sst_drop_stream(ctx, str_id); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 347 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 348 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 349 static int sst_cdev_stream_drain(struct device *dev, unsigned int str_id) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 350 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 351 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 352 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 353 return sst_drain_stream(ctx, str_id, false); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 354 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 355 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 356 static int sst_cdev_stream_partial_drain(struct device *dev, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 357 unsigned int str_id) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 358 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 359 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 360 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 361 return sst_drain_stream(ctx, str_id, true); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 362 } 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 363 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 364 static int sst_cdev_tstamp(struct device *dev, unsigned int str_id, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 365 struct snd_compr_tstamp *tstamp) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 366 { 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 367 struct snd_sst_tstamp fw_tstamp = {0,}; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 368 struct stream_info *stream; 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 369 struct intel_sst_drv *ctx = dev_get_drvdata(dev); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 370 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 371 memcpy_fromio(&fw_tstamp, 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 @372 ((void *)(ctx->mailbox + ctx->tstamp) 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 @373 +(str_id * sizeof(fw_tstamp))), 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 374 sizeof(fw_tstamp)); 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 375 7adab122 sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 376 stream = get_stream_info(ctx, str_id); :::::: The code at line 277 was first introduced by commit :::::: 7adab122a57c5ade8efc2e4de67c72b084c31cda ASoC: Intel: sst - add compressed ops handling :::::: TO: Vinod Koul <vinod.koul(a)intel.com> :::::: CC: Mark Brown <broonie(a)kernel.org> --- 0-DAY kernel test infrastructure Open Source Technology Center http://lists.01.org/mailman/listinfo/kbuild Intel Corporation

2 2