lkml.org 
[lkml]   [2017]   [Sep]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] ASoC: samsung: i2s: Use specific name for i2s dais
On Mon, Aug 28, 2017 at 07:01:00PM +0900, Jaechul Lee wrote:
> Add specific dais name when components are registered. Component and dai
> name will follow their parent dev name, if the name isn't described. In
> case of this driver, each dais will have same name like '11440000.i2s0' by
> fmt_single_name function.
>
> The problem having same name is that TM2 machine driver can't detect
> capture devices correctly. Machine driver doesn't know which one is proper
> to use for cpu dai. The driver just selects to use 'samsung-i2c-sec' that
> doesn't have capture functionality because the component of
> samsung-i2s-sec is located in the first of the component_list.
>
> I add dai name like 'samsung-i2s', 'samsung-i2s-sec' for each dais. The
> reason why adding dai id to 1 is that it doesn't allow to use particular
> dai name in case of when I use 0 for dai id.
>
> Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>
> ---
> sound/soc/samsung/i2s.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/sound/soc/samsung/i2s.c b/sound/soc/samsung/i2s.c
> index af3ba4d4ccc5..9cdd36b9711c 100644
> --- a/sound/soc/samsung/i2s.c
> +++ b/sound/soc/samsung/i2s.c
> @@ -1076,6 +1076,8 @@ static const struct snd_soc_component_driver samsung_i2s_component = {
> .name = "samsung-i2s",
> };
>
> +#define SAMSUNG_I2S_DAI "samsung-i2s"
> +#define SAMSUNG_I2S_DAI_SEC "samsung-i2s-sec"

This should be in i2s.h so next patch would re-use the name, instead of
duplicating it.

Best regards,
Krzysztof

> #define SAMSUNG_I2S_RATES SNDRV_PCM_RATE_8000_96000
>
> #define SAMSUNG_I2S_FMTS (SNDRV_PCM_FMTBIT_S8 | \
> @@ -1093,6 +1095,7 @@ static struct i2s_dai *i2s_alloc_dai(struct platform_device *pdev, bool sec)
> i2s->pdev = pdev;
> i2s->pri_dai = NULL;
> i2s->sec_dai = NULL;
> + i2s->i2s_dai_drv.id = 1;
> i2s->i2s_dai_drv.symmetric_rates = 1;
> i2s->i2s_dai_drv.probe = samsung_i2s_dai_probe;
> i2s->i2s_dai_drv.remove = samsung_i2s_dai_remove;
> @@ -1105,10 +1108,13 @@ static struct i2s_dai *i2s_alloc_dai(struct platform_device *pdev, bool sec)
> i2s->i2s_dai_drv.playback.formats = SAMSUNG_I2S_FMTS;
>
> if (!sec) {
> + i2s->i2s_dai_drv.name = SAMSUNG_I2S_DAI;
> i2s->i2s_dai_drv.capture.channels_min = 1;
> i2s->i2s_dai_drv.capture.channels_max = 2;
> i2s->i2s_dai_drv.capture.rates = SAMSUNG_I2S_RATES;
> i2s->i2s_dai_drv.capture.formats = SAMSUNG_I2S_FMTS;
> + } else {
> + i2s->i2s_dai_drv.name = SAMSUNG_I2S_DAI_SEC;
> }
> return i2s;
> }
> --
> 2.14.1
>

\
 
 \ /
  Last update: 2017-09-03 22:02    [W:0.088 / U:28.024 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site