* [PATCH 0/2] ASoC: Add support for MikroElektronika PROTO codec
@ 2018-08-29 14:47 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: alsa-devel, devicetree, broonie, robh+dt, alexandre.belloni,
nicolas.ferre
Cc: linux-kernel, linux-arm-kernel, florian.meier, cristian.birsan
This patchset is based on Florian's initial PROTO driver:
https://lkml.org/lkml/2013/5/22/342
My changes made the driver more generic, so that we could connect other
I2S CPU DAIs, by using device-tree.
Tested with sama5d2 I2S driver.
Codrin Ciubotariu (2):
ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
ASoC: mikroe-proto: dt-bindings: add DT bindings for PROTO board
.../bindings/sound/mikroe,mikroe-proto.txt | 23 +++
.../devicetree/bindings/vendor-prefixes.txt | 1 +
sound/soc/atmel/Kconfig | 7 +
sound/soc/atmel/Makefile | 2 +
sound/soc/atmel/mikroe-proto.c | 189 ++++++++++++++++++
5 files changed, 222 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
create mode 100644 sound/soc/atmel/mikroe-proto.c
--
2.17.1
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 0/2] ASoC: Add support for MikroElektronika PROTO codec
@ 2018-08-29 14:47 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: alsa-devel, devicetree, broonie, robh+dt, alexandre.belloni,
nicolas.ferre
Cc: linux-kernel, linux-arm-kernel, florian.meier, cristian.birsan
This patchset is based on Florian's initial PROTO driver:
https://lkml.org/lkml/2013/5/22/342
My changes made the driver more generic, so that we could connect other
I2S CPU DAIs, by using device-tree.
Tested with sama5d2 I2S driver.
Codrin Ciubotariu (2):
ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
ASoC: mikroe-proto: dt-bindings: add DT bindings for PROTO board
.../bindings/sound/mikroe,mikroe-proto.txt | 23 +++
.../devicetree/bindings/vendor-prefixes.txt | 1 +
sound/soc/atmel/Kconfig | 7 +
sound/soc/atmel/Makefile | 2 +
sound/soc/atmel/mikroe-proto.c | 189 ++++++++++++++++++
5 files changed, 222 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
create mode 100644 sound/soc/atmel/mikroe-proto.c
--
2.17.1
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 0/2] ASoC: Add support for MikroElektronika PROTO codec
@ 2018-08-29 14:47 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: linux-arm-kernel
This patchset is based on Florian's initial PROTO driver:
https://lkml.org/lkml/2013/5/22/342
My changes made the driver more generic, so that we could connect other
I2S CPU DAIs, by using device-tree.
Tested with sama5d2 I2S driver.
Codrin Ciubotariu (2):
ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
ASoC: mikroe-proto: dt-bindings: add DT bindings for PROTO board
.../bindings/sound/mikroe,mikroe-proto.txt | 23 +++
.../devicetree/bindings/vendor-prefixes.txt | 1 +
sound/soc/atmel/Kconfig | 7 +
sound/soc/atmel/Makefile | 2 +
sound/soc/atmel/mikroe-proto.c | 189 ++++++++++++++++++
5 files changed, 222 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
create mode 100644 sound/soc/atmel/mikroe-proto.c
--
2.17.1
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
2018-08-29 14:47 ` Codrin Ciubotariu
(?)
@ 2018-08-29 14:47 ` Codrin Ciubotariu
-1 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: alsa-devel, devicetree, broonie, robh+dt, alexandre.belloni,
nicolas.ferre
Cc: linux-kernel, linux-arm-kernel, florian.meier, cristian.birsan
Add support for the MikroElektronika PROTO audio codec board.
URL to the audio chip:
http://www.mikroe.com/add-on-boards/audio-voice/audio-codec-proto/
Signed-off-by: Florian Meier <florian.meier@koalo.de>
Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
sound/soc/atmel/Kconfig | 7 ++
sound/soc/atmel/Makefile | 2 +
sound/soc/atmel/mikroe-proto.c | 189 +++++++++++++++++++++++++++++++++
3 files changed, 198 insertions(+)
create mode 100644 sound/soc/atmel/mikroe-proto.c
diff --git a/sound/soc/atmel/Kconfig b/sound/soc/atmel/Kconfig
index 539ef33e670a..fbf4d3e42c18 100644
--- a/sound/soc/atmel/Kconfig
+++ b/sound/soc/atmel/Kconfig
@@ -97,4 +97,11 @@ config SND_ATMEL_SOC_I2S
help
Say Y or M if you want to add support for Atmel ASoc driver for boards
using I2S.
+
+config SND_SOC_MIKROE_PROTO
+ tristate "Support for Mikroe-PROTO board"
+ depends on OF
+ select SND_SOC_WM8731
+ help
+ Say Y or M if you want to add support for Audio Codec Board PROTO (WM8731).
endif
diff --git a/sound/soc/atmel/Makefile b/sound/soc/atmel/Makefile
index cd87cb4bcff5..9f41bfa0fea3 100644
--- a/sound/soc/atmel/Makefile
+++ b/sound/soc/atmel/Makefile
@@ -17,6 +17,7 @@ snd-soc-sam9x5-wm8731-objs := sam9x5_wm8731.o
snd-atmel-soc-classd-objs := atmel-classd.o
snd-atmel-soc-pdmic-objs := atmel-pdmic.o
snd-atmel-soc-tse850-pcm5142-objs := tse850-pcm5142.o
+snd-soc-mikroe-proto-objs := mikroe-proto.o
obj-$(CONFIG_SND_AT91_SOC_SAM9G20_WM8731) += snd-soc-sam9g20-wm8731.o
obj-$(CONFIG_SND_ATMEL_SOC_WM8904) += snd-atmel-soc-wm8904.o
@@ -24,3 +25,4 @@ obj-$(CONFIG_SND_AT91_SOC_SAM9X5_WM8731) += snd-soc-sam9x5-wm8731.o
obj-$(CONFIG_SND_ATMEL_SOC_CLASSD) += snd-atmel-soc-classd.o
obj-$(CONFIG_SND_ATMEL_SOC_PDMIC) += snd-atmel-soc-pdmic.o
obj-$(CONFIG_SND_ATMEL_SOC_TSE850_PCM5142) += snd-atmel-soc-tse850-pcm5142.o
+obj-$(CONFIG_SND_SOC_MIKROE_PROTO) += snd-soc-mikroe-proto.o
diff --git a/sound/soc/atmel/mikroe-proto.c b/sound/soc/atmel/mikroe-proto.c
new file mode 100644
index 000000000000..9d9dc1c8ac6b
--- /dev/null
+++ b/sound/soc/atmel/mikroe-proto.c
@@ -0,0 +1,189 @@
+/*
+ * ASoC driver for PROTO AudioCODEC (with a WM8731)
+ *
+ * Author: Florian Meier, <koalo@koalo.de>
+ * Copyright 2013
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include <linux/module.h>
+#include <linux/platform_device.h>
+
+#include <sound/core.h>
+#include <sound/pcm.h>
+#include <sound/soc.h>
+#include <sound/jack.h>
+
+#include "../codecs/wm8731.h"
+
+#define XTAL_RATE 12288000 /* This is fixed on this board */
+
+static const unsigned int wm8731_rates_12288000[] = {
+ 8000, 32000, 48000, 96000,
+};
+
+static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
+ .list = wm8731_rates_12288000,
+ .count = ARRAY_SIZE(wm8731_rates_12288000),
+};
+
+static int snd_proto_startup(struct snd_pcm_substream *substream)
+{
+ /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
+ snd_pcm_hw_constraint_list(substream->runtime, 0,
+ SNDRV_PCM_HW_PARAM_RATE,
+ &wm8731_constraints_12288000);
+ return 0;
+}
+
+/* machine stream operations */
+static struct snd_soc_ops snd_proto_ops = {
+ .startup = snd_proto_startup,
+};
+
+static int snd_proto_init(struct snd_soc_pcm_runtime *rtd)
+{
+ struct snd_soc_codec *codec = rtd->codec;
+ struct snd_soc_dai *codec_dai = rtd->codec_dai;
+
+ /* Set proto sysclk */
+ int ret = snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
+ XTAL_RATE, SND_SOC_CLOCK_IN);
+ if (ret < 0) {
+ dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
+ return ret;
+ }
+
+ return 0;
+}
+
+static const struct snd_soc_dapm_widget snd_proto_widget[] = {
+ SND_SOC_DAPM_MIC("Microphone Jack", NULL),
+ SND_SOC_DAPM_HP("Headphone Jack", NULL),
+};
+
+static const struct snd_soc_dapm_route snd_proto_route[] = {
+ /* speaker connected to LHPOUT/RHPOUT */
+ {"Headphone Jack", NULL, "LHPOUT"},
+ {"Headphone Jack", NULL, "RHPOUT"},
+
+ /* mic is connected to Mic Jack, with WM8731 Mic Bias */
+ {"MICIN", NULL, "Mic Bias"},
+ {"Mic Bias", NULL, "Microphone Jack"},
+};
+
+/* audio machine driver */
+static struct snd_soc_card snd_proto = {
+ .name = "snd_mikroe_proto",
+ .owner = THIS_MODULE,
+ .dapm_widgets = snd_proto_widget,
+ .num_dapm_widgets = ARRAY_SIZE(snd_proto_widget),
+ .dapm_routes = snd_proto_route,
+ .num_dapm_routes = ARRAY_SIZE(snd_proto_route),
+};
+
+static int snd_proto_probe(struct platform_device *pdev)
+{
+ struct snd_soc_dai_link *dai;
+ struct device_node *np = pdev->dev.of_node;
+ struct device_node *codec_np, *cpu_np;
+ struct device_node *bitclkmaster = NULL;
+ struct device_node *framemaster = NULL;
+ unsigned int dai_fmt;
+ int ret = 0;
+
+ if (!np) {
+ dev_err(&pdev->dev, "No device node supplied\n");
+ return -EINVAL;
+ }
+
+ snd_proto.dev = &pdev->dev;
+ ret = snd_soc_of_parse_card_name(&snd_proto, "model");
+ if (ret)
+ return ret;
+
+ dai = devm_kzalloc(&pdev->dev, sizeof(*dai), GFP_KERNEL);
+ if (!dai)
+ return -ENOMEM;
+
+ snd_proto.dai_link = dai;
+ snd_proto.num_links = 1;
+
+ dai->name = "WM8731";
+ dai->stream_name = "WM8731 HiFi";
+ dai->codec_dai_name = "wm8731-hifi";
+ dai->init = &snd_proto_init;
+ dai->ops = &snd_proto_ops;
+
+ codec_np = of_parse_phandle(np, "audio-codec", 0);
+ if (!codec_np) {
+ dev_err(&pdev->dev, "audio-codec node missing\n");
+ return -EINVAL;
+ }
+ dai->codec_of_node = codec_np;
+
+ cpu_np = of_parse_phandle(np, "i2s-controller", 0);
+ if (!cpu_np) {
+ dev_err(&pdev->dev, "i2s-controller missing\n");
+ return -EINVAL;
+ }
+ dai->cpu_of_node = cpu_np;
+ dai->platform_of_node = cpu_np;
+
+ dai_fmt = snd_soc_of_parse_daifmt(np, NULL,
+ &bitclkmaster, &framemaster);
+ if (bitclkmaster != framemaster) {
+ dev_err(&pdev->dev, "Must be the same bitclock and frame master\n");
+ return -EINVAL;
+ }
+ if (bitclkmaster) {
+ dai_fmt &= ~SND_SOC_DAIFMT_MASTER_MASK;
+ if (codec_np == bitclkmaster)
+ dai_fmt |= SND_SOC_DAIFMT_CBM_CFM;
+ else
+ dai_fmt |= SND_SOC_DAIFMT_CBS_CFS;
+ }
+ of_node_put(bitclkmaster);
+ of_node_put(framemaster);
+ dai->dai_fmt = dai_fmt;
+
+ of_node_put(codec_np);
+ of_node_put(cpu_np);
+
+ ret = snd_soc_register_card(&snd_proto);
+ if (ret && ret != -EPROBE_DEFER)
+ dev_err(&pdev->dev,
+ "snd_soc_register_card() failed: %d\n", ret);
+
+ return ret;
+}
+
+static int snd_proto_remove(struct platform_device *pdev)
+{
+ return snd_soc_unregister_card(&snd_proto);
+}
+
+static const struct of_device_id snd_proto_of_match[] = {
+ { .compatible = "mikroe,mikroe-proto", },
+ {},
+};
+MODULE_DEVICE_TABLE(of, snd_proto_of_match);
+
+static struct platform_driver snd_proto_driver = {
+ .driver = {
+ .name = "snd-mikroe-proto",
+ .owner = THIS_MODULE,
+ .of_match_table = snd_proto_of_match,
+ },
+ .probe = snd_proto_probe,
+ .remove = snd_proto_remove,
+};
+
+module_platform_driver(snd_proto_driver);
+
+MODULE_AUTHOR("Florian Meier");
+MODULE_DESCRIPTION("ASoC Driver for PROTO board (WM8731)");
+MODULE_LICENSE("GPL");
--
2.17.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 14:47 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: alsa-devel, devicetree, broonie, robh+dt, alexandre.belloni,
nicolas.ferre
Cc: linux-kernel, linux-arm-kernel, florian.meier, cristian.birsan
Add support for the MikroElektronika PROTO audio codec board.
URL to the audio chip:
http://www.mikroe.com/add-on-boards/audio-voice/audio-codec-proto/
Signed-off-by: Florian Meier <florian.meier@koalo.de>
Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
sound/soc/atmel/Kconfig | 7 ++
sound/soc/atmel/Makefile | 2 +
sound/soc/atmel/mikroe-proto.c | 189 +++++++++++++++++++++++++++++++++
3 files changed, 198 insertions(+)
create mode 100644 sound/soc/atmel/mikroe-proto.c
diff --git a/sound/soc/atmel/Kconfig b/sound/soc/atmel/Kconfig
index 539ef33e670a..fbf4d3e42c18 100644
--- a/sound/soc/atmel/Kconfig
+++ b/sound/soc/atmel/Kconfig
@@ -97,4 +97,11 @@ config SND_ATMEL_SOC_I2S
help
Say Y or M if you want to add support for Atmel ASoc driver for boards
using I2S.
+
+config SND_SOC_MIKROE_PROTO
+ tristate "Support for Mikroe-PROTO board"
+ depends on OF
+ select SND_SOC_WM8731
+ help
+ Say Y or M if you want to add support for Audio Codec Board PROTO (WM8731).
endif
diff --git a/sound/soc/atmel/Makefile b/sound/soc/atmel/Makefile
index cd87cb4bcff5..9f41bfa0fea3 100644
--- a/sound/soc/atmel/Makefile
+++ b/sound/soc/atmel/Makefile
@@ -17,6 +17,7 @@ snd-soc-sam9x5-wm8731-objs := sam9x5_wm8731.o
snd-atmel-soc-classd-objs := atmel-classd.o
snd-atmel-soc-pdmic-objs := atmel-pdmic.o
snd-atmel-soc-tse850-pcm5142-objs := tse850-pcm5142.o
+snd-soc-mikroe-proto-objs := mikroe-proto.o
obj-$(CONFIG_SND_AT91_SOC_SAM9G20_WM8731) += snd-soc-sam9g20-wm8731.o
obj-$(CONFIG_SND_ATMEL_SOC_WM8904) += snd-atmel-soc-wm8904.o
@@ -24,3 +25,4 @@ obj-$(CONFIG_SND_AT91_SOC_SAM9X5_WM8731) += snd-soc-sam9x5-wm8731.o
obj-$(CONFIG_SND_ATMEL_SOC_CLASSD) += snd-atmel-soc-classd.o
obj-$(CONFIG_SND_ATMEL_SOC_PDMIC) += snd-atmel-soc-pdmic.o
obj-$(CONFIG_SND_ATMEL_SOC_TSE850_PCM5142) += snd-atmel-soc-tse850-pcm5142.o
+obj-$(CONFIG_SND_SOC_MIKROE_PROTO) += snd-soc-mikroe-proto.o
diff --git a/sound/soc/atmel/mikroe-proto.c b/sound/soc/atmel/mikroe-proto.c
new file mode 100644
index 000000000000..9d9dc1c8ac6b
--- /dev/null
+++ b/sound/soc/atmel/mikroe-proto.c
@@ -0,0 +1,189 @@
+/*
+ * ASoC driver for PROTO AudioCODEC (with a WM8731)
+ *
+ * Author: Florian Meier, <koalo@koalo.de>
+ * Copyright 2013
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include <linux/module.h>
+#include <linux/platform_device.h>
+
+#include <sound/core.h>
+#include <sound/pcm.h>
+#include <sound/soc.h>
+#include <sound/jack.h>
+
+#include "../codecs/wm8731.h"
+
+#define XTAL_RATE 12288000 /* This is fixed on this board */
+
+static const unsigned int wm8731_rates_12288000[] = {
+ 8000, 32000, 48000, 96000,
+};
+
+static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
+ .list = wm8731_rates_12288000,
+ .count = ARRAY_SIZE(wm8731_rates_12288000),
+};
+
+static int snd_proto_startup(struct snd_pcm_substream *substream)
+{
+ /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
+ snd_pcm_hw_constraint_list(substream->runtime, 0,
+ SNDRV_PCM_HW_PARAM_RATE,
+ &wm8731_constraints_12288000);
+ return 0;
+}
+
+/* machine stream operations */
+static struct snd_soc_ops snd_proto_ops = {
+ .startup = snd_proto_startup,
+};
+
+static int snd_proto_init(struct snd_soc_pcm_runtime *rtd)
+{
+ struct snd_soc_codec *codec = rtd->codec;
+ struct snd_soc_dai *codec_dai = rtd->codec_dai;
+
+ /* Set proto sysclk */
+ int ret = snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
+ XTAL_RATE, SND_SOC_CLOCK_IN);
+ if (ret < 0) {
+ dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
+ return ret;
+ }
+
+ return 0;
+}
+
+static const struct snd_soc_dapm_widget snd_proto_widget[] = {
+ SND_SOC_DAPM_MIC("Microphone Jack", NULL),
+ SND_SOC_DAPM_HP("Headphone Jack", NULL),
+};
+
+static const struct snd_soc_dapm_route snd_proto_route[] = {
+ /* speaker connected to LHPOUT/RHPOUT */
+ {"Headphone Jack", NULL, "LHPOUT"},
+ {"Headphone Jack", NULL, "RHPOUT"},
+
+ /* mic is connected to Mic Jack, with WM8731 Mic Bias */
+ {"MICIN", NULL, "Mic Bias"},
+ {"Mic Bias", NULL, "Microphone Jack"},
+};
+
+/* audio machine driver */
+static struct snd_soc_card snd_proto = {
+ .name = "snd_mikroe_proto",
+ .owner = THIS_MODULE,
+ .dapm_widgets = snd_proto_widget,
+ .num_dapm_widgets = ARRAY_SIZE(snd_proto_widget),
+ .dapm_routes = snd_proto_route,
+ .num_dapm_routes = ARRAY_SIZE(snd_proto_route),
+};
+
+static int snd_proto_probe(struct platform_device *pdev)
+{
+ struct snd_soc_dai_link *dai;
+ struct device_node *np = pdev->dev.of_node;
+ struct device_node *codec_np, *cpu_np;
+ struct device_node *bitclkmaster = NULL;
+ struct device_node *framemaster = NULL;
+ unsigned int dai_fmt;
+ int ret = 0;
+
+ if (!np) {
+ dev_err(&pdev->dev, "No device node supplied\n");
+ return -EINVAL;
+ }
+
+ snd_proto.dev = &pdev->dev;
+ ret = snd_soc_of_parse_card_name(&snd_proto, "model");
+ if (ret)
+ return ret;
+
+ dai = devm_kzalloc(&pdev->dev, sizeof(*dai), GFP_KERNEL);
+ if (!dai)
+ return -ENOMEM;
+
+ snd_proto.dai_link = dai;
+ snd_proto.num_links = 1;
+
+ dai->name = "WM8731";
+ dai->stream_name = "WM8731 HiFi";
+ dai->codec_dai_name = "wm8731-hifi";
+ dai->init = &snd_proto_init;
+ dai->ops = &snd_proto_ops;
+
+ codec_np = of_parse_phandle(np, "audio-codec", 0);
+ if (!codec_np) {
+ dev_err(&pdev->dev, "audio-codec node missing\n");
+ return -EINVAL;
+ }
+ dai->codec_of_node = codec_np;
+
+ cpu_np = of_parse_phandle(np, "i2s-controller", 0);
+ if (!cpu_np) {
+ dev_err(&pdev->dev, "i2s-controller missing\n");
+ return -EINVAL;
+ }
+ dai->cpu_of_node = cpu_np;
+ dai->platform_of_node = cpu_np;
+
+ dai_fmt = snd_soc_of_parse_daifmt(np, NULL,
+ &bitclkmaster, &framemaster);
+ if (bitclkmaster != framemaster) {
+ dev_err(&pdev->dev, "Must be the same bitclock and frame master\n");
+ return -EINVAL;
+ }
+ if (bitclkmaster) {
+ dai_fmt &= ~SND_SOC_DAIFMT_MASTER_MASK;
+ if (codec_np == bitclkmaster)
+ dai_fmt |= SND_SOC_DAIFMT_CBM_CFM;
+ else
+ dai_fmt |= SND_SOC_DAIFMT_CBS_CFS;
+ }
+ of_node_put(bitclkmaster);
+ of_node_put(framemaster);
+ dai->dai_fmt = dai_fmt;
+
+ of_node_put(codec_np);
+ of_node_put(cpu_np);
+
+ ret = snd_soc_register_card(&snd_proto);
+ if (ret && ret != -EPROBE_DEFER)
+ dev_err(&pdev->dev,
+ "snd_soc_register_card() failed: %d\n", ret);
+
+ return ret;
+}
+
+static int snd_proto_remove(struct platform_device *pdev)
+{
+ return snd_soc_unregister_card(&snd_proto);
+}
+
+static const struct of_device_id snd_proto_of_match[] = {
+ { .compatible = "mikroe,mikroe-proto", },
+ {},
+};
+MODULE_DEVICE_TABLE(of, snd_proto_of_match);
+
+static struct platform_driver snd_proto_driver = {
+ .driver = {
+ .name = "snd-mikroe-proto",
+ .owner = THIS_MODULE,
+ .of_match_table = snd_proto_of_match,
+ },
+ .probe = snd_proto_probe,
+ .remove = snd_proto_remove,
+};
+
+module_platform_driver(snd_proto_driver);
+
+MODULE_AUTHOR("Florian Meier");
+MODULE_DESCRIPTION("ASoC Driver for PROTO board (WM8731)");
+MODULE_LICENSE("GPL");
--
2.17.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 14:47 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: linux-arm-kernel
Add support for the MikroElektronika PROTO audio codec board.
URL to the audio chip:
http://www.mikroe.com/add-on-boards/audio-voice/audio-codec-proto/
Signed-off-by: Florian Meier <florian.meier@koalo.de>
Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
sound/soc/atmel/Kconfig | 7 ++
sound/soc/atmel/Makefile | 2 +
sound/soc/atmel/mikroe-proto.c | 189 +++++++++++++++++++++++++++++++++
3 files changed, 198 insertions(+)
create mode 100644 sound/soc/atmel/mikroe-proto.c
diff --git a/sound/soc/atmel/Kconfig b/sound/soc/atmel/Kconfig
index 539ef33e670a..fbf4d3e42c18 100644
--- a/sound/soc/atmel/Kconfig
+++ b/sound/soc/atmel/Kconfig
@@ -97,4 +97,11 @@ config SND_ATMEL_SOC_I2S
help
Say Y or M if you want to add support for Atmel ASoc driver for boards
using I2S.
+
+config SND_SOC_MIKROE_PROTO
+ tristate "Support for Mikroe-PROTO board"
+ depends on OF
+ select SND_SOC_WM8731
+ help
+ Say Y or M if you want to add support for Audio Codec Board PROTO (WM8731).
endif
diff --git a/sound/soc/atmel/Makefile b/sound/soc/atmel/Makefile
index cd87cb4bcff5..9f41bfa0fea3 100644
--- a/sound/soc/atmel/Makefile
+++ b/sound/soc/atmel/Makefile
@@ -17,6 +17,7 @@ snd-soc-sam9x5-wm8731-objs := sam9x5_wm8731.o
snd-atmel-soc-classd-objs := atmel-classd.o
snd-atmel-soc-pdmic-objs := atmel-pdmic.o
snd-atmel-soc-tse850-pcm5142-objs := tse850-pcm5142.o
+snd-soc-mikroe-proto-objs := mikroe-proto.o
obj-$(CONFIG_SND_AT91_SOC_SAM9G20_WM8731) += snd-soc-sam9g20-wm8731.o
obj-$(CONFIG_SND_ATMEL_SOC_WM8904) += snd-atmel-soc-wm8904.o
@@ -24,3 +25,4 @@ obj-$(CONFIG_SND_AT91_SOC_SAM9X5_WM8731) += snd-soc-sam9x5-wm8731.o
obj-$(CONFIG_SND_ATMEL_SOC_CLASSD) += snd-atmel-soc-classd.o
obj-$(CONFIG_SND_ATMEL_SOC_PDMIC) += snd-atmel-soc-pdmic.o
obj-$(CONFIG_SND_ATMEL_SOC_TSE850_PCM5142) += snd-atmel-soc-tse850-pcm5142.o
+obj-$(CONFIG_SND_SOC_MIKROE_PROTO) += snd-soc-mikroe-proto.o
diff --git a/sound/soc/atmel/mikroe-proto.c b/sound/soc/atmel/mikroe-proto.c
new file mode 100644
index 000000000000..9d9dc1c8ac6b
--- /dev/null
+++ b/sound/soc/atmel/mikroe-proto.c
@@ -0,0 +1,189 @@
+/*
+ * ASoC driver for PROTO AudioCODEC (with a WM8731)
+ *
+ * Author: Florian Meier, <koalo@koalo.de>
+ * Copyright 2013
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include <linux/module.h>
+#include <linux/platform_device.h>
+
+#include <sound/core.h>
+#include <sound/pcm.h>
+#include <sound/soc.h>
+#include <sound/jack.h>
+
+#include "../codecs/wm8731.h"
+
+#define XTAL_RATE 12288000 /* This is fixed on this board */
+
+static const unsigned int wm8731_rates_12288000[] = {
+ 8000, 32000, 48000, 96000,
+};
+
+static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
+ .list = wm8731_rates_12288000,
+ .count = ARRAY_SIZE(wm8731_rates_12288000),
+};
+
+static int snd_proto_startup(struct snd_pcm_substream *substream)
+{
+ /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
+ snd_pcm_hw_constraint_list(substream->runtime, 0,
+ SNDRV_PCM_HW_PARAM_RATE,
+ &wm8731_constraints_12288000);
+ return 0;
+}
+
+/* machine stream operations */
+static struct snd_soc_ops snd_proto_ops = {
+ .startup = snd_proto_startup,
+};
+
+static int snd_proto_init(struct snd_soc_pcm_runtime *rtd)
+{
+ struct snd_soc_codec *codec = rtd->codec;
+ struct snd_soc_dai *codec_dai = rtd->codec_dai;
+
+ /* Set proto sysclk */
+ int ret = snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
+ XTAL_RATE, SND_SOC_CLOCK_IN);
+ if (ret < 0) {
+ dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
+ return ret;
+ }
+
+ return 0;
+}
+
+static const struct snd_soc_dapm_widget snd_proto_widget[] = {
+ SND_SOC_DAPM_MIC("Microphone Jack", NULL),
+ SND_SOC_DAPM_HP("Headphone Jack", NULL),
+};
+
+static const struct snd_soc_dapm_route snd_proto_route[] = {
+ /* speaker connected to LHPOUT/RHPOUT */
+ {"Headphone Jack", NULL, "LHPOUT"},
+ {"Headphone Jack", NULL, "RHPOUT"},
+
+ /* mic is connected to Mic Jack, with WM8731 Mic Bias */
+ {"MICIN", NULL, "Mic Bias"},
+ {"Mic Bias", NULL, "Microphone Jack"},
+};
+
+/* audio machine driver */
+static struct snd_soc_card snd_proto = {
+ .name = "snd_mikroe_proto",
+ .owner = THIS_MODULE,
+ .dapm_widgets = snd_proto_widget,
+ .num_dapm_widgets = ARRAY_SIZE(snd_proto_widget),
+ .dapm_routes = snd_proto_route,
+ .num_dapm_routes = ARRAY_SIZE(snd_proto_route),
+};
+
+static int snd_proto_probe(struct platform_device *pdev)
+{
+ struct snd_soc_dai_link *dai;
+ struct device_node *np = pdev->dev.of_node;
+ struct device_node *codec_np, *cpu_np;
+ struct device_node *bitclkmaster = NULL;
+ struct device_node *framemaster = NULL;
+ unsigned int dai_fmt;
+ int ret = 0;
+
+ if (!np) {
+ dev_err(&pdev->dev, "No device node supplied\n");
+ return -EINVAL;
+ }
+
+ snd_proto.dev = &pdev->dev;
+ ret = snd_soc_of_parse_card_name(&snd_proto, "model");
+ if (ret)
+ return ret;
+
+ dai = devm_kzalloc(&pdev->dev, sizeof(*dai), GFP_KERNEL);
+ if (!dai)
+ return -ENOMEM;
+
+ snd_proto.dai_link = dai;
+ snd_proto.num_links = 1;
+
+ dai->name = "WM8731";
+ dai->stream_name = "WM8731 HiFi";
+ dai->codec_dai_name = "wm8731-hifi";
+ dai->init = &snd_proto_init;
+ dai->ops = &snd_proto_ops;
+
+ codec_np = of_parse_phandle(np, "audio-codec", 0);
+ if (!codec_np) {
+ dev_err(&pdev->dev, "audio-codec node missing\n");
+ return -EINVAL;
+ }
+ dai->codec_of_node = codec_np;
+
+ cpu_np = of_parse_phandle(np, "i2s-controller", 0);
+ if (!cpu_np) {
+ dev_err(&pdev->dev, "i2s-controller missing\n");
+ return -EINVAL;
+ }
+ dai->cpu_of_node = cpu_np;
+ dai->platform_of_node = cpu_np;
+
+ dai_fmt = snd_soc_of_parse_daifmt(np, NULL,
+ &bitclkmaster, &framemaster);
+ if (bitclkmaster != framemaster) {
+ dev_err(&pdev->dev, "Must be the same bitclock and frame master\n");
+ return -EINVAL;
+ }
+ if (bitclkmaster) {
+ dai_fmt &= ~SND_SOC_DAIFMT_MASTER_MASK;
+ if (codec_np == bitclkmaster)
+ dai_fmt |= SND_SOC_DAIFMT_CBM_CFM;
+ else
+ dai_fmt |= SND_SOC_DAIFMT_CBS_CFS;
+ }
+ of_node_put(bitclkmaster);
+ of_node_put(framemaster);
+ dai->dai_fmt = dai_fmt;
+
+ of_node_put(codec_np);
+ of_node_put(cpu_np);
+
+ ret = snd_soc_register_card(&snd_proto);
+ if (ret && ret != -EPROBE_DEFER)
+ dev_err(&pdev->dev,
+ "snd_soc_register_card() failed: %d\n", ret);
+
+ return ret;
+}
+
+static int snd_proto_remove(struct platform_device *pdev)
+{
+ return snd_soc_unregister_card(&snd_proto);
+}
+
+static const struct of_device_id snd_proto_of_match[] = {
+ { .compatible = "mikroe,mikroe-proto", },
+ {},
+};
+MODULE_DEVICE_TABLE(of, snd_proto_of_match);
+
+static struct platform_driver snd_proto_driver = {
+ .driver = {
+ .name = "snd-mikroe-proto",
+ .owner = THIS_MODULE,
+ .of_match_table = snd_proto_of_match,
+ },
+ .probe = snd_proto_probe,
+ .remove = snd_proto_remove,
+};
+
+module_platform_driver(snd_proto_driver);
+
+MODULE_AUTHOR("Florian Meier");
+MODULE_DESCRIPTION("ASoC Driver for PROTO board (WM8731)");
+MODULE_LICENSE("GPL");
--
2.17.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 2/2] ASoC: mikroe-proto: dt-bindings: add DT bindings for PROTO board
2018-08-29 14:47 ` Codrin Ciubotariu
(?)
@ 2018-08-29 14:47 ` Codrin Ciubotariu
-1 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: alsa-devel, devicetree, broonie, robh+dt, alexandre.belloni,
nicolas.ferre
Cc: linux-kernel, linux-arm-kernel, florian.meier, cristian.birsan
DT binding documentation for this new ASoC driver.
Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
.../bindings/sound/mikroe,mikroe-proto.txt | 23 +++++++++++++++++++
.../devicetree/bindings/vendor-prefixes.txt | 1 +
2 files changed, 24 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
diff --git a/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt b/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
new file mode 100644
index 000000000000..912f8fae11c5
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
@@ -0,0 +1,23 @@
+Mikroe-PROTO audio board
+
+Required properties:
+ - compatible: "mikroe,mikroe-proto"
+ - dai-format: Must be "i2s".
+ - i2s-controller: The phandle of the I2S controller.
+ - audio-codec: The phandle of the WM8731 audio codec.
+Optional properties:
+ - model: The user-visible name of this sound complex.
+ - bitclock-master: Indicates dai-link bit clock master; for details see simple-card.txt (1).
+ - frame-master: Indicates dai-link frame master; for details see simple-card.txt (1).
+
+(1) : There must be the same master for both bit and frame clocks.
+
+Example:
+ sound {
+ compatible = "mikroe,mikroe-proto";
+ model = "wm8731 @ sama5d2_xplained";
+ i2s-controller = <&i2s0>;
+ audio-codec = <&wm8731>;
+ dai-format = "i2s";
+ };
+};
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index f4a98c85340a..8ac3ec1f36e5 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -210,6 +210,7 @@ micron Micron Technology Inc.
minix MINIX Technology Ltd.
miramems MiraMEMS Sensing Technology Co., Ltd.
mitsubishi Mitsubishi Electric Corporation
+mikroe MikroElektronika d.o.o.
mosaixtech Mosaix Technologies, Inc.
motorola Motorola, Inc.
moxa Moxa Inc.
--
2.17.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 2/2] ASoC: mikroe-proto: dt-bindings: add DT bindings for PROTO board
@ 2018-08-29 14:47 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: alsa-devel, devicetree, broonie, robh+dt, alexandre.belloni,
nicolas.ferre
Cc: linux-kernel, linux-arm-kernel, florian.meier, cristian.birsan
DT binding documentation for this new ASoC driver.
Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
.../bindings/sound/mikroe,mikroe-proto.txt | 23 +++++++++++++++++++
.../devicetree/bindings/vendor-prefixes.txt | 1 +
2 files changed, 24 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
diff --git a/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt b/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
new file mode 100644
index 000000000000..912f8fae11c5
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
@@ -0,0 +1,23 @@
+Mikroe-PROTO audio board
+
+Required properties:
+ - compatible: "mikroe,mikroe-proto"
+ - dai-format: Must be "i2s".
+ - i2s-controller: The phandle of the I2S controller.
+ - audio-codec: The phandle of the WM8731 audio codec.
+Optional properties:
+ - model: The user-visible name of this sound complex.
+ - bitclock-master: Indicates dai-link bit clock master; for details see simple-card.txt (1).
+ - frame-master: Indicates dai-link frame master; for details see simple-card.txt (1).
+
+(1) : There must be the same master for both bit and frame clocks.
+
+Example:
+ sound {
+ compatible = "mikroe,mikroe-proto";
+ model = "wm8731 @ sama5d2_xplained";
+ i2s-controller = <&i2s0>;
+ audio-codec = <&wm8731>;
+ dai-format = "i2s";
+ };
+};
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index f4a98c85340a..8ac3ec1f36e5 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -210,6 +210,7 @@ micron Micron Technology Inc.
minix MINIX Technology Ltd.
miramems MiraMEMS Sensing Technology Co., Ltd.
mitsubishi Mitsubishi Electric Corporation
+mikroe MikroElektronika d.o.o.
mosaixtech Mosaix Technologies, Inc.
motorola Motorola, Inc.
moxa Moxa Inc.
--
2.17.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 2/2] ASoC: mikroe-proto: dt-bindings: add DT bindings for PROTO board
@ 2018-08-29 14:47 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 14:47 UTC (permalink / raw)
To: linux-arm-kernel
DT binding documentation for this new ASoC driver.
Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
.../bindings/sound/mikroe,mikroe-proto.txt | 23 +++++++++++++++++++
.../devicetree/bindings/vendor-prefixes.txt | 1 +
2 files changed, 24 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
diff --git a/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt b/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
new file mode 100644
index 000000000000..912f8fae11c5
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/mikroe,mikroe-proto.txt
@@ -0,0 +1,23 @@
+Mikroe-PROTO audio board
+
+Required properties:
+ - compatible: "mikroe,mikroe-proto"
+ - dai-format: Must be "i2s".
+ - i2s-controller: The phandle of the I2S controller.
+ - audio-codec: The phandle of the WM8731 audio codec.
+Optional properties:
+ - model: The user-visible name of this sound complex.
+ - bitclock-master: Indicates dai-link bit clock master; for details see simple-card.txt (1).
+ - frame-master: Indicates dai-link frame master; for details see simple-card.txt (1).
+
+(1) : There must be the same master for both bit and frame clocks.
+
+Example:
+ sound {
+ compatible = "mikroe,mikroe-proto";
+ model = "wm8731 @ sama5d2_xplained";
+ i2s-controller = <&i2s0>;
+ audio-codec = <&wm8731>;
+ dai-format = "i2s";
+ };
+};
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index f4a98c85340a..8ac3ec1f36e5 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -210,6 +210,7 @@ micron Micron Technology Inc.
minix MINIX Technology Ltd.
miramems MiraMEMS Sensing Technology Co., Ltd.
mitsubishi Mitsubishi Electric Corporation
+mikroe MikroElektronika d.o.o.
mosaixtech Mosaix Technologies, Inc.
motorola Motorola, Inc.
moxa Moxa Inc.
--
2.17.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
2018-08-29 14:47 ` Codrin Ciubotariu
(?)
@ 2018-08-29 14:53 ` Mark Brown
-1 siblings, 0 replies; 38+ messages in thread
From: Mark Brown @ 2018-08-29 14:53 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: alsa-devel, devicetree, robh+dt, alexandre.belloni,
nicolas.ferre, linux-kernel, linux-arm-kernel, florian.meier,
cristian.birsan
[-- Attachment #1: Type: text/plain, Size: 970 bytes --]
On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
> +static const unsigned int wm8731_rates_12288000[] = {
> + 8000, 32000, 48000, 96000,
> +};
> +
> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
> + .list = wm8731_rates_12288000,
> + .count = ARRAY_SIZE(wm8731_rates_12288000),
> +};
> +
> +static int snd_proto_startup(struct snd_pcm_substream *substream)
> +{
> + /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
> + snd_pcm_hw_constraint_list(substream->runtime, 0,
> + SNDRV_PCM_HW_PARAM_RATE,
> + &wm8731_constraints_12288000);
> + return 0;
> +}
This bit is better added to the CODEC driver since it'll apply to any
system where there's this clock rate (someone else could come in and add
other rates, no need to do that yourself though it'd be nice of course).
That also has the nice bonus that with that I think you'd be able to use
the graph card rather than a custom driver?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 14:53 ` Mark Brown
0 siblings, 0 replies; 38+ messages in thread
From: Mark Brown @ 2018-08-29 14:53 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: devicetree, alsa-devel, nicolas.ferre, alexandre.belloni,
linux-kernel, robh+dt, florian.meier, cristian.birsan,
linux-arm-kernel
[-- Attachment #1.1: Type: text/plain, Size: 970 bytes --]
On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
> +static const unsigned int wm8731_rates_12288000[] = {
> + 8000, 32000, 48000, 96000,
> +};
> +
> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
> + .list = wm8731_rates_12288000,
> + .count = ARRAY_SIZE(wm8731_rates_12288000),
> +};
> +
> +static int snd_proto_startup(struct snd_pcm_substream *substream)
> +{
> + /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
> + snd_pcm_hw_constraint_list(substream->runtime, 0,
> + SNDRV_PCM_HW_PARAM_RATE,
> + &wm8731_constraints_12288000);
> + return 0;
> +}
This bit is better added to the CODEC driver since it'll apply to any
system where there's this clock rate (someone else could come in and add
other rates, no need to do that yourself though it'd be nice of course).
That also has the nice bonus that with that I think you'd be able to use
the graph card rather than a custom driver?
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 14:53 ` Mark Brown
0 siblings, 0 replies; 38+ messages in thread
From: Mark Brown @ 2018-08-29 14:53 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
> +static const unsigned int wm8731_rates_12288000[] = {
> + 8000, 32000, 48000, 96000,
> +};
> +
> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
> + .list = wm8731_rates_12288000,
> + .count = ARRAY_SIZE(wm8731_rates_12288000),
> +};
> +
> +static int snd_proto_startup(struct snd_pcm_substream *substream)
> +{
> + /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
> + snd_pcm_hw_constraint_list(substream->runtime, 0,
> + SNDRV_PCM_HW_PARAM_RATE,
> + &wm8731_constraints_12288000);
> + return 0;
> +}
This bit is better added to the CODEC driver since it'll apply to any
system where there's this clock rate (someone else could come in and add
other rates, no need to do that yourself though it'd be nice of course).
That also has the nice bonus that with that I think you'd be able to use
the graph card rather than a custom driver?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180829/7bf525a1/attachment.sig>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
2018-08-29 14:53 ` Mark Brown
(?)
@ 2018-08-29 15:20 ` Codrin Ciubotariu
-1 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 15:20 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, devicetree, robh+dt, alexandre.belloni,
nicolas.ferre, linux-kernel, linux-arm-kernel, florian.meier,
cristian.birsan
On 29.08.2018 17:53, Mark Brown wrote:
> On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
>
>> +static const unsigned int wm8731_rates_12288000[] = {
>> + 8000, 32000, 48000, 96000,
>> +};
>> +
>> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
>> + .list = wm8731_rates_12288000,
>> + .count = ARRAY_SIZE(wm8731_rates_12288000),
>> +};
>> +
>> +static int snd_proto_startup(struct snd_pcm_substream *substream)
>> +{
>> + /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
>> + snd_pcm_hw_constraint_list(substream->runtime, 0,
>> + SNDRV_PCM_HW_PARAM_RATE,
>> + &wm8731_constraints_12288000);
>> + return 0;
>> +}
>
> This bit is better added to the CODEC driver since it'll apply to any
> system where there's this clock rate (someone else could come in and add
> other rates, no need to do that yourself though it'd be nice of course).
I could do it.
>
> That also has the nice bonus that with that I think you'd be able to use
> the graph card rather than a custom driver?
The main reason for adding a custom driver and not using graph/simple
card is the snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
XTAL_RATE, SND_SOC_CLOCK_IN) call. This
enables the oscillator found in the wm8731 codec. Since
WM8731_SYSCLK_XTAL is not 0, we can't use system-clock-frequency DT
property to set it.
Best regards,
Codrin
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 15:20 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 15:20 UTC (permalink / raw)
To: Mark Brown
Cc: devicetree, alsa-devel, nicolas.ferre, alexandre.belloni,
linux-kernel, robh+dt, florian.meier, cristian.birsan,
linux-arm-kernel
On 29.08.2018 17:53, Mark Brown wrote:
> On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
>
>> +static const unsigned int wm8731_rates_12288000[] = {
>> + 8000, 32000, 48000, 96000,
>> +};
>> +
>> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
>> + .list = wm8731_rates_12288000,
>> + .count = ARRAY_SIZE(wm8731_rates_12288000),
>> +};
>> +
>> +static int snd_proto_startup(struct snd_pcm_substream *substream)
>> +{
>> + /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
>> + snd_pcm_hw_constraint_list(substream->runtime, 0,
>> + SNDRV_PCM_HW_PARAM_RATE,
>> + &wm8731_constraints_12288000);
>> + return 0;
>> +}
>
> This bit is better added to the CODEC driver since it'll apply to any
> system where there's this clock rate (someone else could come in and add
> other rates, no need to do that yourself though it'd be nice of course).
I could do it.
>
> That also has the nice bonus that with that I think you'd be able to use
> the graph card rather than a custom driver?
The main reason for adding a custom driver and not using graph/simple
card is the snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
XTAL_RATE, SND_SOC_CLOCK_IN) call. This
enables the oscillator found in the wm8731 codec. Since
WM8731_SYSCLK_XTAL is not 0, we can't use system-clock-frequency DT
property to set it.
Best regards,
Codrin
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 15:20 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 15:20 UTC (permalink / raw)
To: linux-arm-kernel
On 29.08.2018 17:53, Mark Brown wrote:
> On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
>
>> +static const unsigned int wm8731_rates_12288000[] = {
>> + 8000, 32000, 48000, 96000,
>> +};
>> +
>> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000 = {
>> + .list = wm8731_rates_12288000,
>> + .count = ARRAY_SIZE(wm8731_rates_12288000),
>> +};
>> +
>> +static int snd_proto_startup(struct snd_pcm_substream *substream)
>> +{
>> + /* Setup constraints, because there is a 12.288 MHz XTAL on the board */
>> + snd_pcm_hw_constraint_list(substream->runtime, 0,
>> + SNDRV_PCM_HW_PARAM_RATE,
>> + &wm8731_constraints_12288000);
>> + return 0;
>> +}
>
> This bit is better added to the CODEC driver since it'll apply to any
> system where there's this clock rate (someone else could come in and add
> other rates, no need to do that yourself though it'd be nice of course).
I could do it.
>
> That also has the nice bonus that with that I think you'd be able to use
> the graph card rather than a custom driver?
The main reason for adding a custom driver and not using graph/simple
card is the snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
XTAL_RATE, SND_SOC_CLOCK_IN) call. This
enables the oscillator found in the wm8731 codec. Since
WM8731_SYSCLK_XTAL is not 0, we can't use system-clock-frequency DT
property to set it.
Best regards,
Codrin
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
2018-08-29 15:20 ` Codrin Ciubotariu
(?)
@ 2018-08-29 15:26 ` Mark Brown
-1 siblings, 0 replies; 38+ messages in thread
From: Mark Brown @ 2018-08-29 15:26 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: alsa-devel, devicetree, robh+dt, alexandre.belloni,
nicolas.ferre, linux-kernel, linux-arm-kernel, florian.meier,
cristian.birsan
[-- Attachment #1: Type: text/plain, Size: 777 bytes --]
On Wed, Aug 29, 2018 at 06:20:59PM +0300, Codrin Ciubotariu wrote:
> On 29.08.2018 17:53, Mark Brown wrote:
> > That also has the nice bonus that with that I think you'd be able to use
> > the graph card rather than a custom driver?
> The main reason for adding a custom driver and not using graph/simple card
> is the snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
> XTAL_RATE, SND_SOC_CLOCK_IN) call. This enables the oscillator found in the
> wm8731 codec. Since WM8731_SYSCLK_XTAL is not 0, we can't use
> system-clock-frequency DT property to set it.
Right, that'd be more substantial surgery on the driver to change the DT
binding so that it's got a clock provider in it; it'd be nice to do but
it's a bit much to make it a blocker so the machine driver is OK.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 15:26 ` Mark Brown
0 siblings, 0 replies; 38+ messages in thread
From: Mark Brown @ 2018-08-29 15:26 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: devicetree, alsa-devel, nicolas.ferre, alexandre.belloni,
linux-kernel, robh+dt, florian.meier, cristian.birsan,
linux-arm-kernel
[-- Attachment #1.1: Type: text/plain, Size: 777 bytes --]
On Wed, Aug 29, 2018 at 06:20:59PM +0300, Codrin Ciubotariu wrote:
> On 29.08.2018 17:53, Mark Brown wrote:
> > That also has the nice bonus that with that I think you'd be able to use
> > the graph card rather than a custom driver?
> The main reason for adding a custom driver and not using graph/simple card
> is the snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
> XTAL_RATE, SND_SOC_CLOCK_IN) call. This enables the oscillator found in the
> wm8731 codec. Since WM8731_SYSCLK_XTAL is not 0, we can't use
> system-clock-frequency DT property to set it.
Right, that'd be more substantial surgery on the driver to change the DT
binding so that it's got a clock provider in it; it'd be nice to do but
it's a bit much to make it a blocker so the machine driver is OK.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 15:26 ` Mark Brown
0 siblings, 0 replies; 38+ messages in thread
From: Mark Brown @ 2018-08-29 15:26 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 29, 2018 at 06:20:59PM +0300, Codrin Ciubotariu wrote:
> On 29.08.2018 17:53, Mark Brown wrote:
> > That also has the nice bonus that with that I think you'd be able to use
> > the graph card rather than a custom driver?
> The main reason for adding a custom driver and not using graph/simple card
> is the snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
> XTAL_RATE, SND_SOC_CLOCK_IN) call. This enables the oscillator found in the
> wm8731 codec. Since WM8731_SYSCLK_XTAL is not 0, we can't use
> system-clock-frequency DT property to set it.
Right, that'd be more substantial surgery on the driver to change the DT
binding so that it's got a clock provider in it; it'd be nice to do but
it's a bit much to make it a blocker so the machine driver is OK.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180829/2d517a19/attachment.sig>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
2018-08-29 15:20 ` Codrin Ciubotariu
(?)
@ 2018-08-29 15:33 ` Codrin Ciubotariu
-1 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 15:33 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, devicetree, robh+dt, alexandre.belloni,
nicolas.ferre, linux-kernel, linux-arm-kernel, florian.meier,
cristian.birsan
On 29.08.2018 18:20, Codrin Ciubotariu wrote:
> On 29.08.2018 17:53, Mark Brown wrote:
>> On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
>>
>>> +static const unsigned int wm8731_rates_12288000[] = {
>>> + 8000, 32000, 48000, 96000,
>>> +};
>>> +
>>> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000
>>> = {
>>> + .list = wm8731_rates_12288000,
>>> + .count = ARRAY_SIZE(wm8731_rates_12288000),
>>> +};
>>> +
>>> +static int snd_proto_startup(struct snd_pcm_substream *substream)
>>> +{
>>> + /* Setup constraints, because there is a 12.288 MHz XTAL on the
>>> board */
>>> + snd_pcm_hw_constraint_list(substream->runtime, 0,
>>> + SNDRV_PCM_HW_PARAM_RATE,
>>> + &wm8731_constraints_12288000);
>>> + return 0;
>>> +}
>>
>> This bit is better added to the CODEC driver since it'll apply to any
>> system where there's this clock rate (someone else could come in and add
>> other rates, no need to do that yourself though it'd be nice of course).
>
> I could do it.
It looks like these constraints are already in the wm8731 driver. I will
wait for more comments and then I will send a v2 without this part.
Thanks and best regards,
Codrin
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 15:33 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 15:33 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, devicetree, robh+dt, alexandre.belloni,
nicolas.ferre, linux-kernel, linux-arm-kernel, florian.meier,
cristian.birsan
On 29.08.2018 18:20, Codrin Ciubotariu wrote:
> On 29.08.2018 17:53, Mark Brown wrote:
>> On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
>>
>>> +static const unsigned int wm8731_rates_12288000[] = {
>>> + 8000, 32000, 48000, 96000,
>>> +};
>>> +
>>> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000
>>> = {
>>> + .list = wm8731_rates_12288000,
>>> + .count = ARRAY_SIZE(wm8731_rates_12288000),
>>> +};
>>> +
>>> +static int snd_proto_startup(struct snd_pcm_substream *substream)
>>> +{
>>> + /* Setup constraints, because there is a 12.288 MHz XTAL on the
>>> board */
>>> + snd_pcm_hw_constraint_list(substream->runtime, 0,
>>> + SNDRV_PCM_HW_PARAM_RATE,
>>> + &wm8731_constraints_12288000);
>>> + return 0;
>>> +}
>>
>> This bit is better added to the CODEC driver since it'll apply to any
>> system where there's this clock rate (someone else could come in and add
>> other rates, no need to do that yourself though it'd be nice of course).
>
> I could do it.
It looks like these constraints are already in the wm8731 driver. I will
wait for more comments and then I will send a v2 without this part.
Thanks and best regards,
Codrin
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 15:33 ` Codrin Ciubotariu
0 siblings, 0 replies; 38+ messages in thread
From: Codrin Ciubotariu @ 2018-08-29 15:33 UTC (permalink / raw)
To: linux-arm-kernel
On 29.08.2018 18:20, Codrin Ciubotariu wrote:
> On 29.08.2018 17:53, Mark Brown wrote:
>> On Wed, Aug 29, 2018 at 05:47:26PM +0300, Codrin Ciubotariu wrote:
>>
>>> +static const unsigned int wm8731_rates_12288000[] = {
>>> +??? 8000, 32000, 48000, 96000,
>>> +};
>>> +
>>> +static struct snd_pcm_hw_constraint_list wm8731_constraints_12288000
>>> = {
>>> +??? .list = wm8731_rates_12288000,
>>> +??? .count = ARRAY_SIZE(wm8731_rates_12288000),
>>> +};
>>> +
>>> +static int snd_proto_startup(struct snd_pcm_substream *substream)
>>> +{
>>> +??? /* Setup constraints, because there is a 12.288 MHz XTAL on the
>>> board */
>>> +??? snd_pcm_hw_constraint_list(substream->runtime, 0,
>>> +?????????????????? SNDRV_PCM_HW_PARAM_RATE,
>>> +?????????????????? &wm8731_constraints_12288000);
>>> +??? return 0;
>>> +}
>>
>> This bit is better added to the CODEC driver since it'll apply to any
>> system where there's this clock rate (someone else could come in and add
>> other rates, no need to do that yourself though it'd be nice of course).
>
> I could do it.
It looks like these constraints are already in the wm8731 driver. I will
wait for more comments and then I will send a v2 without this part.
Thanks and best regards,
Codrin
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
2018-08-29 14:47 ` Codrin Ciubotariu
(?)
@ 2018-08-29 23:43 ` kbuild test robot
-1 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2018-08-29 23:43 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: kbuild-all, alsa-devel, devicetree, broonie, robh+dt,
alexandre.belloni, nicolas.ferre, linux-kernel, linux-arm-kernel,
florian.meier, cristian.birsan
[-- Attachment #1: Type: text/plain, Size: 1874 bytes --]
Hi Codrin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on at91/at91-next]
[also build test ERROR on v4.19-rc1 next-20180829]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Codrin-Ciubotariu/ASoC-Add-driver-for-PROTO-Audio-CODEC-with-a-WM8731/20180830-060610
base: https://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git at91-next
config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
sound/soc/atmel/mikroe-proto.c: In function 'snd_proto_init':
>> sound/soc/atmel/mikroe-proto.c:49:35: error: 'struct snd_soc_pcm_runtime' has no member named 'codec'
struct snd_soc_codec *codec = rtd->codec;
^~
>> sound/soc/atmel/mikroe-proto.c:56:16: error: dereferencing pointer to incomplete type 'struct snd_soc_codec'
dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
^~
vim +49 sound/soc/atmel/mikroe-proto.c
46
47 static int snd_proto_init(struct snd_soc_pcm_runtime *rtd)
48 {
> 49 struct snd_soc_codec *codec = rtd->codec;
50 struct snd_soc_dai *codec_dai = rtd->codec_dai;
51
52 /* Set proto sysclk */
53 int ret = snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
54 XTAL_RATE, SND_SOC_CLOCK_IN);
55 if (ret < 0) {
> 56 dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
57 return ret;
58 }
59
60 return 0;
61 }
62
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 64126 bytes --]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 23:43 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2018-08-29 23:43 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: devicetree, alsa-devel, linux-kernel, cristian.birsan,
alexandre.belloni, robh+dt, nicolas.ferre, broonie,
florian.meier, kbuild-all, linux-arm-kernel
[-- Attachment #1: Type: text/plain, Size: 1874 bytes --]
Hi Codrin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on at91/at91-next]
[also build test ERROR on v4.19-rc1 next-20180829]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Codrin-Ciubotariu/ASoC-Add-driver-for-PROTO-Audio-CODEC-with-a-WM8731/20180830-060610
base: https://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git at91-next
config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
sound/soc/atmel/mikroe-proto.c: In function 'snd_proto_init':
>> sound/soc/atmel/mikroe-proto.c:49:35: error: 'struct snd_soc_pcm_runtime' has no member named 'codec'
struct snd_soc_codec *codec = rtd->codec;
^~
>> sound/soc/atmel/mikroe-proto.c:56:16: error: dereferencing pointer to incomplete type 'struct snd_soc_codec'
dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
^~
vim +49 sound/soc/atmel/mikroe-proto.c
46
47 static int snd_proto_init(struct snd_soc_pcm_runtime *rtd)
48 {
> 49 struct snd_soc_codec *codec = rtd->codec;
50 struct snd_soc_dai *codec_dai = rtd->codec_dai;
51
52 /* Set proto sysclk */
53 int ret = snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
54 XTAL_RATE, SND_SOC_CLOCK_IN);
55 if (ret < 0) {
> 56 dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
57 return ret;
58 }
59
60 return 0;
61 }
62
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 64126 bytes --]
[-- Attachment #3: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-29 23:43 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2018-08-29 23:43 UTC (permalink / raw)
To: linux-arm-kernel
Hi Codrin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on at91/at91-next]
[also build test ERROR on v4.19-rc1 next-20180829]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Codrin-Ciubotariu/ASoC-Add-driver-for-PROTO-Audio-CODEC-with-a-WM8731/20180830-060610
base: https://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git at91-next
config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
sound/soc/atmel/mikroe-proto.c: In function 'snd_proto_init':
>> sound/soc/atmel/mikroe-proto.c:49:35: error: 'struct snd_soc_pcm_runtime' has no member named 'codec'
struct snd_soc_codec *codec = rtd->codec;
^~
>> sound/soc/atmel/mikroe-proto.c:56:16: error: dereferencing pointer to incomplete type 'struct snd_soc_codec'
dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
^~
vim +49 sound/soc/atmel/mikroe-proto.c
46
47 static int snd_proto_init(struct snd_soc_pcm_runtime *rtd)
48 {
> 49 struct snd_soc_codec *codec = rtd->codec;
50 struct snd_soc_dai *codec_dai = rtd->codec_dai;
51
52 /* Set proto sysclk */
53 int ret = snd_soc_dai_set_sysclk(codec_dai, WM8731_SYSCLK_XTAL,
54 XTAL_RATE, SND_SOC_CLOCK_IN);
55 if (ret < 0) {
> 56 dev_err(codec->dev, "Failed to set WM8731 SYSCLK: %d\n", ret);
57 return ret;
58 }
59
60 return 0;
61 }
62
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 64126 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180830/71379246/attachment-0001.gz>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
2018-08-29 14:47 ` Codrin Ciubotariu
@ 2018-08-30 10:12 ` kbuild test robot
-1 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2018-08-30 10:12 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: kbuild-all, alsa-devel, devicetree, broonie, robh+dt,
alexandre.belloni, nicolas.ferre, linux-kernel, linux-arm-kernel,
florian.meier, cristian.birsan
Hi Codrin,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on at91/at91-next]
[also build test WARNING on v4.19-rc1 next-20180830]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Codrin-Ciubotariu/ASoC-Add-driver-for-PROTO-Audio-CODEC-with-a-WM8731/20180830-060610
base: https://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git at91-next
coccinelle warnings: (new ones prefixed by >>)
>> sound/soc/atmel/mikroe-proto.c:178:3-8: No need to set .owner here. The core will do it.
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731)
@ 2018-08-30 10:12 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2018-08-30 10:12 UTC (permalink / raw)
To: linux-arm-kernel
Hi Codrin,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on at91/at91-next]
[also build test WARNING on v4.19-rc1 next-20180830]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Codrin-Ciubotariu/ASoC-Add-driver-for-PROTO-Audio-CODEC-with-a-WM8731/20180830-060610
base: https://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git at91-next
coccinelle warnings: (new ones prefixed by >>)
>> sound/soc/atmel/mikroe-proto.c:178:3-8: No need to set .owner here. The core will do it.
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
2018-08-29 14:47 ` Codrin Ciubotariu
@ 2018-08-30 10:13 ` kbuild test robot
-1 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2018-08-30 10:13 UTC (permalink / raw)
To: Codrin Ciubotariu
Cc: kbuild-all, alsa-devel, devicetree, broonie, robh+dt,
alexandre.belloni, nicolas.ferre, linux-kernel, linux-arm-kernel,
florian.meier, cristian.birsan
From: kbuild test robot <fengguang.wu@intel.com>
sound/soc/atmel/mikroe-proto.c:178:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Fixes: 395c2cec021d ("ASoC: Add driver for PROTO Audio CODEC (with a WM8731)")
CC: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
Signed-off-by: kbuild test robot <fengguang.wu@intel.com>
---
url: https://github.com/0day-ci/linux/commits/Codrin-Ciubotariu/ASoC-Add-driver-for-PROTO-Audio-CODEC-with-a-WM8731/20180830-060610
base: https://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git at91-next
mikroe-proto.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/atmel/mikroe-proto.c
+++ b/sound/soc/atmel/mikroe-proto.c
@@ -175,7 +175,6 @@ MODULE_DEVICE_TABLE(of, snd_proto_of_mat
static struct platform_driver snd_proto_driver = {
.driver = {
.name = "snd-mikroe-proto",
- .owner = THIS_MODULE,
.of_match_table = snd_proto_of_match,
},
.probe = snd_proto_probe,
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
@ 2018-08-30 10:13 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2018-08-30 10:13 UTC (permalink / raw)
To: linux-arm-kernel
From: kbuild test robot <fengguang.wu@intel.com>
sound/soc/atmel/mikroe-proto.c:178:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Fixes: 395c2cec021d ("ASoC: Add driver for PROTO Audio CODEC (with a WM8731)")
CC: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
Signed-off-by: kbuild test robot <fengguang.wu@intel.com>
---
url: https://github.com/0day-ci/linux/commits/Codrin-Ciubotariu/ASoC-Add-driver-for-PROTO-Audio-CODEC-with-a-WM8731/20180830-060610
base: https://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git at91-next
mikroe-proto.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/atmel/mikroe-proto.c
+++ b/sound/soc/atmel/mikroe-proto.c
@@ -175,7 +175,6 @@ MODULE_DEVICE_TABLE(of, snd_proto_of_mat
static struct platform_driver snd_proto_driver = {
.driver = {
.name = "snd-mikroe-proto",
- .owner = THIS_MODULE,
.of_match_table = snd_proto_of_match,
},
.probe = snd_proto_probe,
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
2017-02-21 13:58 ` simon.ho.cnxt
@ 2017-02-21 17:44 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2017-02-21 17:44 UTC (permalink / raw)
To: simon.ho.cnxt; +Cc: alsa-devel, Simon Ho, lgirdwood, tiwai, broonie, kbuild-all
sound/soc/codecs/cx2092x-i2c.c:43:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Simon Ho <simon.ho@conexant.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
cx2092x-i2c.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/codecs/cx2092x-i2c.c
+++ b/sound/soc/codecs/cx2092x-i2c.c
@@ -40,7 +40,6 @@ MODULE_DEVICE_TABLE(i2c, cx2092x_i2c_id)
static struct i2c_driver cx2092x_i2c_driver = {
.driver = {
.name = "cx2092x",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(cx2092x_dt_ids),
},
.id_table = cx2092x_i2c_id,
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
2017-01-31 10:05 ` Mylène Josserand
2017-01-31 11:34 ` kbuild test robot
@ 2017-01-31 11:34 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2017-01-31 11:34 UTC (permalink / raw)
To: Mylène Josserand
Cc: kbuild-all, lgirdwood, broonie, robh+dt, mark.rutland,
maxime.ripard, wens, perex, tiwai, alsa-devel, devicetree,
linux-kernel, linux-arm-kernel, thomas.petazzoni,
mylene.josserand, alexandre.belloni
sound/soc/sunxi/sun8i-codec.c:487:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Mylène Josserand <mylene.josserand@free-electrons.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
sun8i-codec.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/sunxi/sun8i-codec.c
+++ b/sound/soc/sunxi/sun8i-codec.c
@@ -484,7 +484,6 @@ static const struct dev_pm_ops sun8i_cod
static struct platform_driver sun8i_codec_driver = {
.driver = {
.name = "sun8i-codec",
- .owner = THIS_MODULE,
.of_match_table = sun8i_codec_of_match,
.pm = &sun8i_codec_pm_ops,
},
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
@ 2017-01-31 11:34 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2017-01-31 11:34 UTC (permalink / raw)
Cc: mark.rutland, devicetree, alsa-devel, linux-kernel, tiwai,
lgirdwood, robh+dt, wens, broonie, alexandre.belloni, kbuild-all,
maxime.ripard, mylene.josserand, thomas.petazzoni,
linux-arm-kernel
sound/soc/sunxi/sun8i-codec.c:487:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Mylène Josserand <mylene.josserand@free-electrons.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
sun8i-codec.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/sunxi/sun8i-codec.c
+++ b/sound/soc/sunxi/sun8i-codec.c
@@ -484,7 +484,6 @@ static const struct dev_pm_ops sun8i_cod
static struct platform_driver sun8i_codec_driver = {
.driver = {
.name = "sun8i-codec",
- .owner = THIS_MODULE,
.of_match_table = sun8i_codec_of_match,
.pm = &sun8i_codec_pm_ops,
},
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
@ 2017-01-31 11:34 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2017-01-31 11:34 UTC (permalink / raw)
To: linux-arm-kernel
sound/soc/sunxi/sun8i-codec.c:487:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Myl?ne Josserand <mylene.josserand@free-electrons.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
sun8i-codec.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/sunxi/sun8i-codec.c
+++ b/sound/soc/sunxi/sun8i-codec.c
@@ -484,7 +484,6 @@ static const struct dev_pm_ops sun8i_cod
static struct platform_driver sun8i_codec_driver = {
.driver = {
.name = "sun8i-codec",
- .owner = THIS_MODULE,
.of_match_table = sun8i_codec_of_match,
.pm = &sun8i_codec_pm_ops,
},
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
2016-11-09 16:58 [asoc:topic/cs42l42 4/5] sound/soc/codecs/cs42l42.c:1972:3-8: No need to set .owner here. The core will do it kbuild test robot
@ 2016-11-09 16:58 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2016-11-09 16:58 UTC (permalink / raw)
To: James Schulman
Cc: kbuild-all, alsa-devel, Mark Brown, Brian Austin, Paul Handrigan,
Liam Girdwood, Jaroslav Kysela, Takashi Iwai, linux-kernel
sound/soc/codecs/cs42l42.c:1972:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: James Schulman <james.schulman@cirrus.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
cs42l42.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/codecs/cs42l42.c
+++ b/sound/soc/codecs/cs42l42.c
@@ -1969,7 +1969,6 @@ MODULE_DEVICE_TABLE(i2c, cs42l42_id);
static struct i2c_driver cs42l42_i2c_driver = {
.driver = {
.name = "cs42l42",
- .owner = THIS_MODULE,
.pm = &cs42l42_runtime_pm,
.of_match_table = cs42l42_of_match,
},
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
2015-12-01 18:31 ` anish kumar
@ 2015-12-01 19:23 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2015-12-01 19:23 UTC (permalink / raw)
Cc: alsa-devel, broonie, kbuild-all, anish kumar
sound/soc/codecs/max98926.c:587:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: anish kumar <yesanishhere@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
max98926.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/codecs/max98926.c
+++ b/sound/soc/codecs/max98926.c
@@ -584,7 +584,6 @@ MODULE_DEVICE_TABLE(of, max98926_of_matc
static struct i2c_driver max98926_i2c_driver = {
.driver = {
.name = "max98926",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(max98926_of_match),
.pm = NULL,
},
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
2015-06-22 15:36 [asoc:topic/gtm601 2/3] sound/soc/codecs/gtm601.c:86:4-9: No need to set .owner here. The core will do it kbuild test robot
@ 2015-06-22 15:36 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2015-06-22 15:36 UTC (permalink / raw)
To: Marek Belisko
Cc: kbuild-all, Mark Brown, alsa-devel, Liam Girdwood,
Jaroslav Kysela, Takashi Iwai, linux-kernel
sound/soc/codecs/gtm601.c:86:4-9: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Marek Belisko <marek@goldelico.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
gtm601.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/codecs/gtm601.c
+++ b/sound/soc/codecs/gtm601.c
@@ -83,7 +83,6 @@ MODULE_DEVICE_TABLE(of, gtm601_codec_of_
static struct platform_driver gtm601_codec_driver = {
.driver = {
.name = "gtm601",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(gtm601_codec_of_match),
},
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
@ 2015-06-22 15:36 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2015-06-22 15:36 UTC (permalink / raw)
To: Marek Belisko
Cc: kbuild-all, Mark Brown, alsa-devel, Liam Girdwood,
Jaroslav Kysela, Takashi Iwai, linux-kernel
sound/soc/codecs/gtm601.c:86:4-9: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Marek Belisko <marek@goldelico.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
gtm601.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/codecs/gtm601.c
+++ b/sound/soc/codecs/gtm601.c
@@ -83,7 +83,6 @@ MODULE_DEVICE_TABLE(of, gtm601_codec_of_
static struct platform_driver gtm601_codec_driver = {
.driver = {
.name = "gtm601",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(gtm601_codec_of_match),
},
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
[not found] <201412070634.hwbnurjX%fengguang.wu@intel.com>
@ 2014-12-06 22:06 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2014-12-06 22:06 UTC (permalink / raw)
To: Max Filippov
Cc: kbuild-all, Liam Girdwood, Mark Brown, Jaroslav Kysela,
Takashi Iwai, linux-xtensa, alsa-devel, linux-kernel
sound/soc/xtensa/xtfpga-i2s.c:666:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
xtfpga-i2s.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/xtensa/xtfpga-i2s.c
+++ b/sound/soc/xtensa/xtfpga-i2s.c
@@ -663,7 +663,6 @@ static struct platform_driver xtfpga_i2s
.remove = xtfpga_i2s_remove,
.driver = {
.name = "xtfpga-i2s",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(xtfpga_i2s_of_match),
.pm = &xtfpga_i2s_pm_ops,
},
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings
@ 2014-12-06 22:06 ` kbuild test robot
0 siblings, 0 replies; 38+ messages in thread
From: kbuild test robot @ 2014-12-06 22:06 UTC (permalink / raw)
To: Max Filippov
Cc: linux-xtensa, alsa-devel, Takashi Iwai, linux-kernel,
Liam Girdwood, Mark Brown, kbuild-all
sound/soc/xtensa/xtfpga-i2s.c:666:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
xtfpga-i2s.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/xtensa/xtfpga-i2s.c
+++ b/sound/soc/xtensa/xtfpga-i2s.c
@@ -663,7 +663,6 @@ static struct platform_driver xtfpga_i2s
.remove = xtfpga_i2s_remove,
.driver = {
.name = "xtfpga-i2s",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(xtfpga_i2s_of_match),
.pm = &xtfpga_i2s_pm_ops,
},
^ permalink raw reply [flat|nested] 38+ messages in thread
end of thread, other threads:[~2018-08-30 10:14 UTC | newest]
Thread overview: 38+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-29 14:47 [PATCH 0/2] ASoC: Add support for MikroElektronika PROTO codec Codrin Ciubotariu
2018-08-29 14:47 ` Codrin Ciubotariu
2018-08-29 14:47 ` Codrin Ciubotariu
2018-08-29 14:47 ` [PATCH 1/2] ASoC: Add driver for PROTO Audio CODEC (with a WM8731) Codrin Ciubotariu
2018-08-29 14:47 ` Codrin Ciubotariu
2018-08-29 14:47 ` Codrin Ciubotariu
2018-08-29 14:53 ` Mark Brown
2018-08-29 14:53 ` Mark Brown
2018-08-29 14:53 ` Mark Brown
2018-08-29 15:20 ` Codrin Ciubotariu
2018-08-29 15:20 ` Codrin Ciubotariu
2018-08-29 15:20 ` Codrin Ciubotariu
2018-08-29 15:26 ` Mark Brown
2018-08-29 15:26 ` Mark Brown
2018-08-29 15:26 ` Mark Brown
2018-08-29 15:33 ` Codrin Ciubotariu
2018-08-29 15:33 ` Codrin Ciubotariu
2018-08-29 15:33 ` Codrin Ciubotariu
2018-08-29 23:43 ` kbuild test robot
2018-08-29 23:43 ` kbuild test robot
2018-08-29 23:43 ` kbuild test robot
2018-08-30 10:12 ` kbuild test robot
2018-08-30 10:12 ` kbuild test robot
2018-08-30 10:13 ` [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings kbuild test robot
2018-08-30 10:13 ` kbuild test robot
2018-08-29 14:47 ` [PATCH 2/2] ASoC: mikroe-proto: dt-bindings: add DT bindings for PROTO board Codrin Ciubotariu
2018-08-29 14:47 ` Codrin Ciubotariu
2018-08-29 14:47 ` Codrin Ciubotariu
-- strict thread matches above, loose matches on Subject: below --
2017-02-21 17:44 [alsa_devel] [PATCH v3 2/2] ASoC: Add driver support fro Conexant CX2092X DSP kbuild test robot
2017-02-21 13:58 ` simon.ho.cnxt
2017-02-21 17:44 ` [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings kbuild test robot
2017-01-31 11:34 [PATCH v3 3/8] ASoC: Add sun8i digital audio codec kbuild test robot
2017-01-31 10:05 ` Mylène Josserand
2017-01-31 11:34 ` [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings kbuild test robot
2017-01-31 11:34 ` kbuild test robot
2017-01-31 11:34 ` kbuild test robot
2016-11-09 16:58 [asoc:topic/cs42l42 4/5] sound/soc/codecs/cs42l42.c:1972:3-8: No need to set .owner here. The core will do it kbuild test robot
2016-11-09 16:58 ` [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings kbuild test robot
2015-12-01 19:23 [PATCH] ASoC: Add max98926 codec driver kbuild test robot
2015-12-01 18:31 ` anish kumar
2015-12-01 19:23 ` [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings kbuild test robot
2015-06-22 15:36 [asoc:topic/gtm601 2/3] sound/soc/codecs/gtm601.c:86:4-9: No need to set .owner here. The core will do it kbuild test robot
2015-06-22 15:36 ` [PATCH] ASoC: fix platform_no_drv_owner.cocci warnings kbuild test robot
2015-06-22 15:36 ` kbuild test robot
[not found] <201412070634.hwbnurjX%fengguang.wu@intel.com>
2014-12-06 22:06 ` kbuild test robot
2014-12-06 22:06 ` kbuild test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.