[PATCH 3/3 v5] ASoC: simple-card: add Device Tree support
Kuninori Morimoto
kuninori.morimoto.gx at renesas.com
Thu Feb 7 11:55:25 EST 2013
Support for loading the simple-card module via devicetree.
It requests cpu/codec information for probing.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx at renesas.com>
---
v3 -> v4
- removed port-name
- it gets dai_name from node and port number
v4 -> v5
- fixup compile error on !CONFIG_OF
.../devicetree/bindings/sound/simple-card.txt | 61 +++++++++
sound/soc/generic/simple-card.c | 134 +++++++++++++++++++-
2 files changed, 190 insertions(+), 5 deletions(-)
create mode 100644 Documentation/devicetree/bindings/sound/simple-card.txt
diff --git a/Documentation/devicetree/bindings/sound/simple-card.txt b/Documentation/devicetree/bindings/sound/simple-card.txt
new file mode 100644
index 0000000..dc94b51
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/simple-card.txt
@@ -0,0 +1,61 @@
+Simple-Card:
+
+Required properties:
+
+ [prefix] means cpu/codec here
+
+- compatible : "simple-audio"
+- simple-audio,card-name : simple-audio card name
+- simple-audio,format : see below
+- simple-audio,[prefix],dai : phandle and port for CPU/CODEC
+- simple-audio,[prefix],frame-master : frame master
+- simple-audio,[prefix],bitclock-master : bitclock master
+
+Optional properties:
+
+- simple-audio,system-clock-frequency : system clock rate if it is connected to both CPU/CODEC
+- simple-audio,bitclock-inversion : bit clock inversion for both CPU/CODEC
+- simple-audio,frame-inversion : frame inversion for both CPU/CODEC
+
+- simple-audio,[prefix],bitclock-inversion : if CPU/CODEC needs clock inversion
+- simple-audio,[prefix],frame-inversion : if CPU/CODEC needs frame inversion
+- simple-audio,[prefix],system-clock-frequency : system clock rate for each CPU/CODEC
+
+simple-audio,format
+ "i2s"
+ "right_j"
+ "left_j"
+ "dsp_a"
+ "dsp_b"
+ "ac97"
+ "pdm"
+ "msb"
+ "lsb"
+
+Example:
+
+sound {
+ compatible = "simple-audio";
+
+ simple-audio,card-name = "FSI2A-AK4648";
+ simple-audio,format = "left_j";
+ simple-audio,cpu,dai = <&sh_fsi2 0>;
+ simple-audio,codec,dai = <&ak4648 0>;
+ simple-audio,codec,bitclock-master;
+ simple-audio,codec,frame-master;
+ simple-audio,codec,system-clock-frequency = <11289600>;
+};
+
+&i2c0 {
+ ak4648: ak4648 at 0x12 {
+ compatible = "asahi-kasei,ak4648";
+ reg = <0x12>;
+ };
+};
+
+sh_fsi2: sh_fsi2 at 0xec230000 {
+ compatible = "renesas,sh_fsi2";
+ reg = <0xec230000 0x400>;
+ interrupt-parent = <&gic>;
+ interrupts = <0 146 0x4>;
+};
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index 6cf8355..0773655 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -9,6 +9,7 @@
* published by the Free Software Foundation.
*/
+#include <linux/of.h>
#include <linux/platform_device.h>
#include <linux/module.h>
#include <sound/simple_card.h>
@@ -52,11 +53,124 @@ static int asoc_simple_card_dai_init(struct snd_soc_pcm_runtime *rtd)
return 0;
}
+#ifdef CONFIG_OF
+static struct device_node*
+__asoc_simple_card_parse_of(struct device_node *np,
+ struct asoc_simple_dai *dai,
+ const char *prefix)
+{
+ const __be32 *list;
+ struct device_node *node;
+ char prop[128];
+ int size, port;
+
+ /* get [prefix]dai = <&phandle port-num> */
+ snprintf(prop, sizeof(prop), "%s,dai", prefix);
+ list = of_get_property(np, prop, &size);
+ if (!list)
+ return NULL;
+
+ if (2 != size / sizeof(*list))
+ return NULL;
+
+ node = of_find_node_by_phandle(be32_to_cpup(list++));
+ if (!node)
+ return NULL;
+
+ of_node_put(node);
+
+ /* get port number */
+ port = be32_to_cpup(list);
+
+ /* get dai-name from node and port */
+ dai->name = snd_soc_of_get_port_dai_name(node, port);
+
+ /* get dai specific format */
+ dai->fmt = snd_soc_of_parse_daifmt(np, prefix);
+
+ /* get "[prefix]system-clock-frequency = <xxx>" */
+ snprintf(prop, sizeof(prop), "%s,system-clock-frequency", prefix);
+ of_property_read_u32(np, prop, &dai->sysclk);
+
+ return node;
+}
+
+static struct asoc_simple_card_info*
+asoc_simple_card_parse_of(struct device_node *np,
+ struct device *dev,
+ struct device_node **of_cpu,
+ struct device_node **of_codec,
+ struct device_node **of_platform)
+{
+ struct asoc_simple_card_info *info;
+ u32 sysclk = 0;
+
+ info = devm_kzalloc(dev, sizeof(*info), GFP_KERNEL);
+ if (!info) {
+ dev_err(dev, "card info alloc failed\n");
+ return NULL;
+ }
+
+ of_property_read_string(np, "simple-audio,card-name", &info->card);
+ info->name = info->card;
+
+ info->daifmt = snd_soc_of_parse_daifmt(np, "simple-audio,") &
+ (SND_SOC_DAIFMT_FORMAT_MASK | SND_SOC_DAIFMT_INV_MASK);
+
+ of_property_read_u32(np,
+ "simple-audio,system-clock-frequency", &sysclk);
+
+ info->cpu_dai.sysclk = sysclk;
+ info->codec_dai.sysclk = sysclk;
+
+ *of_cpu = __asoc_simple_card_parse_of(np, &info->cpu_dai,
+ "simple-audio,cpu");
+ *of_codec = __asoc_simple_card_parse_of(np, &info->codec_dai,
+ "simple-audio,codec");
+
+ /* simple-card assumes platform == cpu */
+ *of_platform = *of_cpu;
+
+ dev_dbg(dev, "card-name : %s\n", info->card);
+ dev_dbg(dev, "platform : %04x / %p\n",
+ info->daifmt,
+ *of_platform);
+ dev_dbg(dev, "cpu : %s / %04x / %d / %p\n",
+ info->cpu_dai.name,
+ info->cpu_dai.fmt,
+ info->cpu_dai.sysclk,
+ *of_cpu);
+ dev_dbg(dev, "codec : %s / %04x / %d / %p\n",
+ info->codec_dai.name,
+ info->codec_dai.fmt,
+ info->codec_dai.sysclk,
+ *of_codec);
+
+ return info;
+}
+#else
+#define asoc_simple_card_parse_of(...) NULL
+#endif
+
static int asoc_simple_card_probe(struct platform_device *pdev)
{
- struct asoc_simple_card_info *cinfo = pdev->dev.platform_data;
+ struct asoc_simple_card_info *cinfo;
+ struct device_node *np = pdev->dev.of_node;
+ struct device_node *of_cpu, *of_codec, *of_platform;
struct device *dev = &pdev->dev;
+ cinfo = NULL;
+ of_cpu = NULL;
+ of_codec = NULL;
+ of_platform = NULL;
+ if (np)
+ cinfo = asoc_simple_card_parse_of(np, dev,
+ &of_cpu,
+ &of_codec,
+ &of_platform);
+ else
+ cinfo = pdev->dev.platform_data;
+
if (!cinfo) {
dev_err(dev, "no info for asoc-simple-card\n");
return -EINVAL;
@@ -64,10 +178,10 @@ static int asoc_simple_card_probe(struct platform_device *pdev)
if (!cinfo->name ||
!cinfo->card ||
- !cinfo->codec ||
- !cinfo->platform ||
- !cinfo->cpu_dai.name ||
- !cinfo->codec_dai.name) {
+ !cinfo->codec_dai.name ||
+ !(cinfo->codec || of_codec) ||
+ !(cinfo->platform || of_platform) ||
+ !(cinfo->cpu_dai.name || of_cpu)) {
dev_err(dev, "insufficient asoc_simple_card_info settings\n");
return -EINVAL;
}
@@ -81,6 +195,9 @@ static int asoc_simple_card_probe(struct platform_device *pdev)
cinfo->snd_link.platform_name = cinfo->platform;
cinfo->snd_link.codec_name = cinfo->codec;
cinfo->snd_link.codec_dai_name = cinfo->codec_dai.name;
+ cinfo->snd_link.cpu_of_node = of_cpu;
+ cinfo->snd_link.codec_of_node = of_codec;
+ cinfo->snd_link.platform_of_node = of_platform;
cinfo->snd_link.init = asoc_simple_card_dai_init;
/*
@@ -102,9 +219,16 @@ static int asoc_simple_card_remove(struct platform_device *pdev)
return snd_soc_unregister_card(&cinfo->snd_card);
}
+static const struct of_device_id asoc_simple_of_match[] = {
+ { .compatible = "simple-audio", },
+ {},
+};
+MODULE_DEVICE_TABLE(of, asoc_simple_of_match);
+
static struct platform_driver asoc_simple_card = {
.driver = {
.name = "asoc-simple-card",
+ .of_match_table = asoc_simple_of_match,
},
.probe = asoc_simple_card_probe,
.remove = asoc_simple_card_remove,
--
1.7.9.5
More information about the devicetree-discuss
mailing list