[alsa-devel] [PATCH v2] ASoC: soc-core: add snd_soc_of_parse_master_clkdir() for dt
This patch adds snd_soc_of_parse_master_clkdir() and supports below style on dt:
master-clkdir-out;
If this property is absent 0 will be returned, 1 otherwise. 0 : SND_SOC_CLOCK_IN 1 : SND_SOC_CLOCK_OUT
Signed-off-by: Xiubo Li Li.Xiubo@freescale.com --- include/sound/soc.h | 1 + sound/soc/soc-core.c | 11 +++++++++++ 2 files changed, 12 insertions(+)
diff --git a/include/sound/soc.h b/include/sound/soc.h index 5a049d9..690f4f6 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -1170,6 +1170,7 @@ int snd_soc_of_parse_card_name(struct snd_soc_card *card, const char *propname); int snd_soc_of_parse_audio_routing(struct snd_soc_card *card, const char *propname); +unsigned int snd_soc_of_parse_master_clkdir(struct device_node *np); unsigned int snd_soc_of_parse_daifmt(struct device_node *np, const char *prefix); int snd_soc_of_get_dai_name(struct device_node *of_node, diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index be88df5..f155207 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -4472,6 +4472,17 @@ int snd_soc_of_parse_audio_routing(struct snd_soc_card *card, } EXPORT_SYMBOL_GPL(snd_soc_of_parse_audio_routing);
+unsigned int snd_soc_of_parse_master_clkdir(struct device_node *np) +{ + /* + * check "master-clkdir-out" + * false means SND_SOC_CLOCK_IN + * true means SND_SOC_CLOCK_OUT + */ + return !!of_property_read_bool(np, "master-clkdir-out"); +} +EXPORT_SYMBOL_GPL(snd_soc_of_parse_master_clkdir); + unsigned int snd_soc_of_parse_daifmt(struct device_node *np, const char *prefix) {
Hi
I would like to know the current status of this patch, and there are some other patches that for simple-card are depended on this one.
Thanks,
--
This patch adds snd_soc_of_parse_master_clkdir() and supports below style on dt:
master-clkdir-out;
If this property is absent 0 will be returned, 1 otherwise. 0 : SND_SOC_CLOCK_IN 1 : SND_SOC_CLOCK_OUT
Signed-off-by: Xiubo Li Li.Xiubo@freescale.com
include/sound/soc.h | 1 + sound/soc/soc-core.c | 11 +++++++++++ 2 files changed, 12 insertions(+)
diff --git a/include/sound/soc.h b/include/sound/soc.h index 5a049d9..690f4f6 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -1170,6 +1170,7 @@ int snd_soc_of_parse_card_name(struct snd_soc_card *card, const char *propname); int snd_soc_of_parse_audio_routing(struct snd_soc_card *card, const char *propname); +unsigned int snd_soc_of_parse_master_clkdir(struct device_node *np); unsigned int snd_soc_of_parse_daifmt(struct device_node *np, const char *prefix); int snd_soc_of_get_dai_name(struct device_node *of_node, diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index be88df5..f155207 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -4472,6 +4472,17 @@ int snd_soc_of_parse_audio_routing(struct snd_soc_card *card, } EXPORT_SYMBOL_GPL(snd_soc_of_parse_audio_routing);
+unsigned int snd_soc_of_parse_master_clkdir(struct device_node *np) +{
- /*
* check "master-clkdir-out"
* false means SND_SOC_CLOCK_IN
* true means SND_SOC_CLOCK_OUT
*/
- return !!of_property_read_bool(np, "master-clkdir-out");
+} +EXPORT_SYMBOL_GPL(snd_soc_of_parse_master_clkdir);
unsigned int snd_soc_of_parse_daifmt(struct device_node *np, const char *prefix) { -- 1.8.4
On Mon, Jan 06, 2014 at 05:41:59AM +0000, Li.Xiubo@freescale.com wrote:
I would like to know the current status of this patch, and there are some other patches that for simple-card are depended on this one.
You only sent this patch on Thursday during the middle of the holiday period, please allow a reasonable time for review - at least a week even without holidays.
Hi Mark,
You only sent this patch on Thursday during the middle of the holiday period, please allow a reasonable time for review - at least a week even without holidays.
Yes, Sorry.
I was just a little hurry for we are having a holiday for New Year soon.
Thanks,
-- Best Regards, Xiubo
On Thu, Jan 02, 2014 at 10:48:25AM +0800, Xiubo Li wrote:
This patch adds snd_soc_of_parse_master_clkdir() and supports below style on dt:
master-clkdir-out;
If this property is absent 0 will be returned, 1 otherwise. 0 : SND_SOC_CLOCK_IN 1 : SND_SOC_CLOCK_OUT
This is mostly OK code wise, though it would be good to use those constants in the code. However I'm still not sure about it being a good idea - it's similar issues to those with clock IDs but less strong. Do you have a user in mind?
Subject: Re: [PATCH v2] ASoC: soc-core: add snd_soc_of_parse_master_clkdir() for dt
On Thu, Jan 02, 2014 at 10:48:25AM +0800, Xiubo Li wrote:
This patch adds snd_soc_of_parse_master_clkdir() and supports below style on dt:
master-clkdir-out;
If this property is absent 0 will be returned, 1 otherwise. 0 : SND_SOC_CLOCK_IN 1 : SND_SOC_CLOCK_OUT
This is mostly OK code wise, though it would be good to use those constants in the code. However I'm still not sure about it being a good idea - it's similar issues to those with clock IDs but less strong. Do you have a user in mind?
Yes, for the SAI & SGTL5000 DAIs and the existing fsl_ssi.c driver, etc. For SSI DAI device, it will depend on the sysclk directions to do the different clock settings.
For the VF610 series, the SAI will provide the clock to SGTL5000 Codec. Using the simple audio card driver, I must set the CPU and CODEC DAI sysclk dir values differently.
I will send one simple audio card's use case based SAI & SGTL5000 later.
Thanks,
-- Best Regards, Xiubo
participants (3)
-
Li.Xiubo@freescale.com
-
Mark Brown
-
Xiubo Li