Re: [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On Sat, Mar 15, 2014 at 12:09:39PM +0100, Jean-Francois Moine wrote: > The DAI link array and the properties (fmt, sysclk slots) are > hard-coded for a single CPU / CODEC link. Sorry, I dropped this patch - it conflicts with Nicolin's patch to force the two ends of the DAI link to have the same format which probably isn't the ideal fix but at least allows sensible looking DTs to be written and parsed. Nicolin is on holiday and I don't have any systems which use simple-card. signature.asc Description: Digital signature
Re: [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On Sat, Mar 15, 2014 at 12:09:39PM +0100, Jean-Francois Moine wrote: The DAI link array and the properties (fmt, sysclk slots) are hard-coded for a single CPU / CODEC link. Sorry, I dropped this patch - it conflicts with Nicolin's patch to force the two ends of the DAI link to have the same format which probably isn't the ideal fix but at least allows sensible looking DTs to be written and parsed. Nicolin is on holiday and I don't have any systems which use simple-card. signature.asc Description: Digital signature
Re: [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On Sat, Mar 15, 2014 at 12:09:39PM +0100, Jean-Francois Moine wrote: > The DAI link array and the properties (fmt, sysclk slots) are > hard-coded for a single CPU / CODEC link. Applied, thanks. signature.asc Description: Digital signature
Re: [alsa-devel] [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On 03/17/2014 12:23 PM, Jean-Francois Moine wrote: Jyri, No, this would not work. The DAI link in the struct snd_soc_card is an array of struct snd_soc_dai_link. There cannot be anything between the elements! Oh.. did not go deep enough. Forget about this comment. Best regards, Jyri -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [alsa-devel] [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On Mon, 17 Mar 2014 11:29:42 +0200
Jyri Sarha wrote:
> On 03/15/2014 01:09 PM, Jean-Francois Moine wrote:
> > The DAI link array and the properties (fmt, sysclk slots) are
> > hard-coded for a single CPU / CODEC link.
> >
> > This patch dynamically allocates the DAI link array and the
> > properties with the aim of supporting many DAI links.
> >
> > Signed-off-by: Jean-Francois Moine
> > ---
> > sound/soc/generic/simple-card.c | 49
> > +
> > 1 file changed, 30 insertions(+), 19 deletions(-)
> >
> > diff --git a/sound/soc/generic/simple-card.c
> > b/sound/soc/generic/simple-card.c
> > index ca7e63e..a55dc46 100644
> > --- a/sound/soc/generic/simple-card.c
> > +++ b/sound/soc/generic/simple-card.c
> > @@ -20,9 +20,11 @@
> >
> > struct simple_card_data {
> > struct snd_soc_card snd_card;
> > - struct asoc_simple_dai cpu_dai;
> > - struct asoc_simple_dai codec_dai;
> > - struct snd_soc_dai_link snd_link;
> > + struct simple_dais {
> > + struct asoc_simple_dai cpu_dai;
> > + struct asoc_simple_dai codec_dai;
> > + } *dais;
> > + struct snd_soc_dai_link dai_link[]; /* dynamically allocated */
> > };
> >
>
> This is only an implementation detail, but wouldn't it produce a cleaner
> implementation if you would write the above structure like this:
>
>struct simple_card_data {
> struct snd_soc_card snd_card;
> struct simple_dai_links {
> struct snd_soc_dai_link dai_link;
> struct asoc_simple_dai cpu_dai;
> struct asoc_simple_dai codec_dai;
> } *dai_links;
>};
>
> or even
>
>struct simple_card_data {
> struct snd_soc_card snd_card;
> struct simple_dai_links {
> struct snd_soc_dai_link dai_link;
> struct asoc_simple_dai cpu_dai;
> struct asoc_simple_dai codec_dai;
> } dai_links[]; /* dynamically allocated */
>};
>
> But, as said this only an implementation detail.
Jyri,
No, this would not work. The DAI link in the struct snd_soc_card is an
array of struct snd_soc_dai_link. There cannot be anything between the
elements!
--
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [alsa-devel] [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On 03/15/2014 01:09 PM, Jean-Francois Moine wrote:
The DAI link array and the properties (fmt, sysclk slots) are
hard-coded for a single CPU / CODEC link.
This patch dynamically allocates the DAI link array and the
properties with the aim of supporting many DAI links.
Signed-off-by: Jean-Francois Moine
---
sound/soc/generic/simple-card.c | 49 +
1 file changed, 30 insertions(+), 19 deletions(-)
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index ca7e63e..a55dc46 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -20,9 +20,11 @@
struct simple_card_data {
struct snd_soc_card snd_card;
- struct asoc_simple_dai cpu_dai;
- struct asoc_simple_dai codec_dai;
- struct snd_soc_dai_link snd_link;
+ struct simple_dais {
+ struct asoc_simple_dai cpu_dai;
+ struct asoc_simple_dai codec_dai;
+ } *dais;
+ struct snd_soc_dai_link dai_link[]; /* dynamically allocated */
};
This is only an implementation detail, but wouldn't it produce a cleaner
implementation if you would write the above structure like this:
struct simple_card_data {
struct snd_soc_card snd_card;
struct simple_dai_links {
struct snd_soc_dai_link dai_link;
struct asoc_simple_dai cpu_dai;
struct asoc_simple_dai codec_dai;
} *dai_links;
};
or even
struct simple_card_data {
struct snd_soc_card snd_card;
struct simple_dai_links {
struct snd_soc_dai_link dai_link;
struct asoc_simple_dai cpu_dai;
struct asoc_simple_dai codec_dai;
} dai_links[]; /* dynamically allocated */
};
But, as said this only an implementation detail.
Best regards,
Jyri
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [alsa-devel] [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On 03/15/2014 01:09 PM, Jean-Francois Moine wrote:
The DAI link array and the properties (fmt, sysclk slots) are
hard-coded for a single CPU / CODEC link.
This patch dynamically allocates the DAI link array and the
properties with the aim of supporting many DAI links.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/soc/generic/simple-card.c | 49 +
1 file changed, 30 insertions(+), 19 deletions(-)
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index ca7e63e..a55dc46 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -20,9 +20,11 @@
struct simple_card_data {
struct snd_soc_card snd_card;
- struct asoc_simple_dai cpu_dai;
- struct asoc_simple_dai codec_dai;
- struct snd_soc_dai_link snd_link;
+ struct simple_dais {
+ struct asoc_simple_dai cpu_dai;
+ struct asoc_simple_dai codec_dai;
+ } *dais;
+ struct snd_soc_dai_link dai_link[]; /* dynamically allocated */
};
This is only an implementation detail, but wouldn't it produce a cleaner
implementation if you would write the above structure like this:
struct simple_card_data {
struct snd_soc_card snd_card;
struct simple_dai_links {
struct snd_soc_dai_link dai_link;
struct asoc_simple_dai cpu_dai;
struct asoc_simple_dai codec_dai;
} *dai_links;
};
or even
struct simple_card_data {
struct snd_soc_card snd_card;
struct simple_dai_links {
struct snd_soc_dai_link dai_link;
struct asoc_simple_dai cpu_dai;
struct asoc_simple_dai codec_dai;
} dai_links[]; /* dynamically allocated */
};
But, as said this only an implementation detail.
Best regards,
Jyri
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [alsa-devel] [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On Mon, 17 Mar 2014 11:29:42 +0200
Jyri Sarha jsa...@ti.com wrote:
On 03/15/2014 01:09 PM, Jean-Francois Moine wrote:
The DAI link array and the properties (fmt, sysclk slots) are
hard-coded for a single CPU / CODEC link.
This patch dynamically allocates the DAI link array and the
properties with the aim of supporting many DAI links.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/soc/generic/simple-card.c | 49
+
1 file changed, 30 insertions(+), 19 deletions(-)
diff --git a/sound/soc/generic/simple-card.c
b/sound/soc/generic/simple-card.c
index ca7e63e..a55dc46 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -20,9 +20,11 @@
struct simple_card_data {
struct snd_soc_card snd_card;
- struct asoc_simple_dai cpu_dai;
- struct asoc_simple_dai codec_dai;
- struct snd_soc_dai_link snd_link;
+ struct simple_dais {
+ struct asoc_simple_dai cpu_dai;
+ struct asoc_simple_dai codec_dai;
+ } *dais;
+ struct snd_soc_dai_link dai_link[]; /* dynamically allocated */
};
This is only an implementation detail, but wouldn't it produce a cleaner
implementation if you would write the above structure like this:
struct simple_card_data {
struct snd_soc_card snd_card;
struct simple_dai_links {
struct snd_soc_dai_link dai_link;
struct asoc_simple_dai cpu_dai;
struct asoc_simple_dai codec_dai;
} *dai_links;
};
or even
struct simple_card_data {
struct snd_soc_card snd_card;
struct simple_dai_links {
struct snd_soc_dai_link dai_link;
struct asoc_simple_dai cpu_dai;
struct asoc_simple_dai codec_dai;
} dai_links[]; /* dynamically allocated */
};
But, as said this only an implementation detail.
Jyri,
No, this would not work. The DAI link in the struct snd_soc_card is an
array of struct snd_soc_dai_link. There cannot be anything between the
elements!
--
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [alsa-devel] [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On 03/17/2014 12:23 PM, Jean-Francois Moine wrote: Jyri, No, this would not work. The DAI link in the struct snd_soc_card is an array of struct snd_soc_dai_link. There cannot be anything between the elements! Oh.. did not go deep enough. Forget about this comment. Best regards, Jyri -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
On Sat, Mar 15, 2014 at 12:09:39PM +0100, Jean-Francois Moine wrote: The DAI link array and the properties (fmt, sysclk slots) are hard-coded for a single CPU / CODEC link. Applied, thanks. signature.asc Description: Digital signature
[PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
The DAI link array and the properties (fmt, sysclk slots) are
hard-coded for a single CPU / CODEC link.
This patch dynamically allocates the DAI link array and the
properties with the aim of supporting many DAI links.
Signed-off-by: Jean-Francois Moine
---
sound/soc/generic/simple-card.c | 49 +
1 file changed, 30 insertions(+), 19 deletions(-)
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index ca7e63e..a55dc46 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -20,9 +20,11 @@
struct simple_card_data {
struct snd_soc_card snd_card;
- struct asoc_simple_dai cpu_dai;
- struct asoc_simple_dai codec_dai;
- struct snd_soc_dai_link snd_link;
+ struct simple_dais {
+ struct asoc_simple_dai cpu_dai;
+ struct asoc_simple_dai codec_dai;
+ } *dais;
+ struct snd_soc_dai_link dai_link[]; /* dynamically allocated */
};
static int __asoc_simple_card_dai_init(struct snd_soc_dai *dai,
@@ -70,11 +72,11 @@ static int asoc_simple_card_dai_init(struct
snd_soc_pcm_runtime *rtd)
struct snd_soc_dai *cpu = rtd->cpu_dai;
int ret;
- ret = __asoc_simple_card_dai_init(codec, >codec_dai);
+ ret = __asoc_simple_card_dai_init(codec, >dais->codec_dai);
if (ret < 0)
return ret;
- ret = __asoc_simple_card_dai_init(cpu, >cpu_dai);
+ ret = __asoc_simple_card_dai_init(cpu, >dais->cpu_dai);
if (ret < 0)
return ret;
@@ -184,7 +186,7 @@ static int asoc_simple_card_parse_of(struct device_node
*node,
np = of_get_child_by_name(node, "simple-audio-card,cpu");
if (np) {
ret = asoc_simple_card_sub_parse_of(np, daifmt,
- >cpu_dai,
+ >dais->cpu_dai,
_link->cpu_of_node,
_link->cpu_dai_name);
of_node_put(np);
@@ -197,7 +199,7 @@ static int asoc_simple_card_parse_of(struct device_node
*node,
np = of_get_child_by_name(node, "simple-audio-card,codec");
if (np) {
ret = asoc_simple_card_sub_parse_of(np, daifmt,
- >codec_dai,
+ >dais->codec_dai,
_link->codec_of_node,
_link->codec_dai_name);
of_node_put(np);
@@ -226,12 +228,12 @@ static int asoc_simple_card_parse_of(struct device_node
*node,
dev_dbg(dev, "platform : %04x\n", daifmt);
dev_dbg(dev, "cpu : %s / %04x / %d\n",
dai_link->cpu_dai_name,
- priv->cpu_dai.fmt,
- priv->cpu_dai.sysclk);
+ priv->dais->cpu_dai.fmt,
+ priv->dais->cpu_dai.sysclk);
dev_dbg(dev, "codec : %s / %04x / %d\n",
dai_link->codec_dai_name,
- priv->codec_dai.fmt,
- priv->codec_dai.sysclk);
+ priv->dais->codec_dai.fmt,
+ priv->dais->codec_dai.sysclk);
/*
* soc_bind_dai_link() will check cpu name
@@ -276,7 +278,9 @@ static int asoc_simple_card_probe(struct platform_device
*pdev)
struct device *dev = >dev;
int ret;
- priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
+ priv = devm_kzalloc(dev,
+ sizeof(*priv) + sizeof(*dai_link),
+ GFP_KERNEL);
if (!priv)
return -ENOMEM;
@@ -285,10 +289,17 @@ static int asoc_simple_card_probe(struct platform_device
*pdev)
*/
priv->snd_card.owner = THIS_MODULE;
priv->snd_card.dev = dev;
- dai_link = >snd_link;
+ dai_link = priv->dai_link;
priv->snd_card.dai_link = dai_link;
priv->snd_card.num_links = 1;
+ /* get room for the other properties */
+ priv->dais = devm_kzalloc(dev,
+ sizeof(*priv->dais),
+ GFP_KERNEL);
+ if (!priv->dais)
+ return -ENOMEM;
+
if (np && of_device_is_available(np)) {
ret = asoc_simple_card_parse_of(np, priv, dev);
@@ -322,13 +333,13 @@ static int asoc_simple_card_probe(struct platform_device
*pdev)
dai_link->codec_name= cinfo->codec;
dai_link->cpu_dai_name = cinfo->cpu_dai.name;
dai_link->codec_dai_name = cinfo->codec_dai.name;
- memcpy(>cpu_dai, >cpu_dai,
- sizeof(priv->cpu_dai));
- memcpy(>codec_dai, >codec_dai,
- sizeof(priv->codec_dai));
+ memcpy(>dais->cpu_dai, >cpu_dai,
+
[PATCH v3 2/4] ASoC: simple-card: dynamically allocate the DAI link and properties
The DAI link array and the properties (fmt, sysclk slots) are
hard-coded for a single CPU / CODEC link.
This patch dynamically allocates the DAI link array and the
properties with the aim of supporting many DAI links.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/soc/generic/simple-card.c | 49 +
1 file changed, 30 insertions(+), 19 deletions(-)
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index ca7e63e..a55dc46 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -20,9 +20,11 @@
struct simple_card_data {
struct snd_soc_card snd_card;
- struct asoc_simple_dai cpu_dai;
- struct asoc_simple_dai codec_dai;
- struct snd_soc_dai_link snd_link;
+ struct simple_dais {
+ struct asoc_simple_dai cpu_dai;
+ struct asoc_simple_dai codec_dai;
+ } *dais;
+ struct snd_soc_dai_link dai_link[]; /* dynamically allocated */
};
static int __asoc_simple_card_dai_init(struct snd_soc_dai *dai,
@@ -70,11 +72,11 @@ static int asoc_simple_card_dai_init(struct
snd_soc_pcm_runtime *rtd)
struct snd_soc_dai *cpu = rtd-cpu_dai;
int ret;
- ret = __asoc_simple_card_dai_init(codec, priv-codec_dai);
+ ret = __asoc_simple_card_dai_init(codec, priv-dais-codec_dai);
if (ret 0)
return ret;
- ret = __asoc_simple_card_dai_init(cpu, priv-cpu_dai);
+ ret = __asoc_simple_card_dai_init(cpu, priv-dais-cpu_dai);
if (ret 0)
return ret;
@@ -184,7 +186,7 @@ static int asoc_simple_card_parse_of(struct device_node
*node,
np = of_get_child_by_name(node, simple-audio-card,cpu);
if (np) {
ret = asoc_simple_card_sub_parse_of(np, daifmt,
- priv-cpu_dai,
+ priv-dais-cpu_dai,
dai_link-cpu_of_node,
dai_link-cpu_dai_name);
of_node_put(np);
@@ -197,7 +199,7 @@ static int asoc_simple_card_parse_of(struct device_node
*node,
np = of_get_child_by_name(node, simple-audio-card,codec);
if (np) {
ret = asoc_simple_card_sub_parse_of(np, daifmt,
- priv-codec_dai,
+ priv-dais-codec_dai,
dai_link-codec_of_node,
dai_link-codec_dai_name);
of_node_put(np);
@@ -226,12 +228,12 @@ static int asoc_simple_card_parse_of(struct device_node
*node,
dev_dbg(dev, platform : %04x\n, daifmt);
dev_dbg(dev, cpu : %s / %04x / %d\n,
dai_link-cpu_dai_name,
- priv-cpu_dai.fmt,
- priv-cpu_dai.sysclk);
+ priv-dais-cpu_dai.fmt,
+ priv-dais-cpu_dai.sysclk);
dev_dbg(dev, codec : %s / %04x / %d\n,
dai_link-codec_dai_name,
- priv-codec_dai.fmt,
- priv-codec_dai.sysclk);
+ priv-dais-codec_dai.fmt,
+ priv-dais-codec_dai.sysclk);
/*
* soc_bind_dai_link() will check cpu name
@@ -276,7 +278,9 @@ static int asoc_simple_card_probe(struct platform_device
*pdev)
struct device *dev = pdev-dev;
int ret;
- priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
+ priv = devm_kzalloc(dev,
+ sizeof(*priv) + sizeof(*dai_link),
+ GFP_KERNEL);
if (!priv)
return -ENOMEM;
@@ -285,10 +289,17 @@ static int asoc_simple_card_probe(struct platform_device
*pdev)
*/
priv-snd_card.owner = THIS_MODULE;
priv-snd_card.dev = dev;
- dai_link = priv-snd_link;
+ dai_link = priv-dai_link;
priv-snd_card.dai_link = dai_link;
priv-snd_card.num_links = 1;
+ /* get room for the other properties */
+ priv-dais = devm_kzalloc(dev,
+ sizeof(*priv-dais),
+ GFP_KERNEL);
+ if (!priv-dais)
+ return -ENOMEM;
+
if (np of_device_is_available(np)) {
ret = asoc_simple_card_parse_of(np, priv, dev);
@@ -322,13 +333,13 @@ static int asoc_simple_card_probe(struct platform_device
*pdev)
dai_link-codec_name= cinfo-codec;
dai_link-cpu_dai_name = cinfo-cpu_dai.name;
dai_link-codec_dai_name = cinfo-codec_dai.name;
- memcpy(priv-cpu_dai, cinfo-cpu_dai,
- sizeof(priv-cpu_dai));
- memcpy(priv-codec_dai, cinfo-codec_dai,
- sizeof(priv-codec_dai));
+
