All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol
       [not found] <CGME20200401004223epcas2p2a2b0f54f83ea8fa0c1a1b96660cf4b69@epcas2p2.samsung.com>
@ 2020-04-01  0:42 ` 이경택
  2020-04-01  8:37   ` Mark Brown
  0 siblings, 1 reply; 7+ messages in thread
From: 이경택 @ 2020-04-01  0:42 UTC (permalink / raw)
  To: broonie; +Cc: alsa-devel, lgirdwood, tiwai, hmseo, tkjung, pilsun.jang

Current topology doesn't add prefix of component to new kcontrol.

Signed-off-by: Gyeongtaek Lee <gt82.lee@samsung.com>
---
 sound/soc/soc-topology.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/soc-topology.c b/sound/soc/soc-topology.c
index 575da6aba807..a152409e8746 100644
--- a/sound/soc/soc-topology.c
+++ b/sound/soc/soc-topology.c
@@ -362,7 +362,7 @@ static int soc_tplg_add_kcontrol(struct soc_tplg *tplg,
 	struct snd_soc_component *comp = tplg->comp;
 
 	return soc_tplg_add_dcontrol(comp->card->snd_card,
-				comp->dev, k, NULL, comp, kcontrol);
+				comp->dev, k, comp->name_prefix, comp, kcontrol);
 }
 
 /* remove a mixer kcontrol */
-- 
2.21.0



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol
  2020-04-01  0:42 ` [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol 이경택
@ 2020-04-01  8:37   ` Mark Brown
  2020-04-01  9:22     ` 이경택
  0 siblings, 1 reply; 7+ messages in thread
From: Mark Brown @ 2020-04-01  8:37 UTC (permalink / raw)
  To: �̰���
  Cc: alsa-devel, lgirdwood, tiwai, hmseo, tkjung, pilsun.jang

[-- Attachment #1: Type: text/plain, Size: 367 bytes --]

On Wed, Apr 01, 2020 at 09:42:23AM +0900, �̰��� wrote:
> Current topology doesn't add prefix of component to new kcontrol.

Please only number patches if you are sending them as part of a patch
series, the purpose of numbering is to make sure that things get handled
in the right order.  Without a series it looks like there are some
missing patches.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol
  2020-04-01  8:37   ` Mark Brown
@ 2020-04-01  9:22     ` 이경택
  0 siblings, 0 replies; 7+ messages in thread
From: 이경택 @ 2020-04-01  9:22 UTC (permalink / raw)
  To: 'Mark Brown'
  Cc: alsa-devel, lgirdwood, tiwai, hmseo, tkjung, pilsun.jang

On Wed, Apr 01, 2020 at 09:37:08AM +0100, Mark Brown wrote:
>On Wed, Apr 01, 2020 at 09:42:23AM +0900, �̰��� wrote:
>> Current topology doesn't add prefix of component to new kcontrol.
>
>Please only number patches if you are sending them as part of a patch
>series, the purpose of numbering is to make sure that things get handled
>in the right order.  Without a series it looks like there are some
>missing patches.
>
I'm trying to catch up the merge process now.
Let me know if anything has gone wrong.
I removed the patch number and re-sent the patch again.

Sorry and thank you for your review.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol
  2020-03-30  7:35 ` 이경택
@ 2020-03-31 18:07   ` Mark Brown
  0 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2020-03-31 18:07 UTC (permalink / raw)
  To: �̰���; +Cc: alsa-devel, lgirdwood, tiwai

[-- Attachment #1: Type: text/plain, Size: 267 bytes --]

On Mon, Mar 30, 2020 at 04:35:55PM +0900, �̰��� wrote:

>  	return soc_tplg_add_dcontrol(comp->card->snd_card,
> -				comp->dev, k, NULL, comp, kcontrol);
> +				comp->dev, k, comp->name_prefix, comp,
> kcontrol);

This has word wrapping damage too.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol
       [not found] <CGME20200331065047epcas2p23455307bb2eed7b62a04e387e6ebf9c1@epcas2p2.samsung.com>
@ 2020-03-31  6:50 ` 이경택
  0 siblings, 0 replies; 7+ messages in thread
From: 이경택 @ 2020-03-31  6:50 UTC (permalink / raw)
  To: broonie; +Cc: alsa-devel, lgirdwood, tiwai, hmseo, tkjung, pilsun.jang

Current topology doesn't add prefix of the component to newly created kcontrol.

Signed-off-by: Gyeongtaek Lee <gt82.lee@samsung.com>
---
 sound/soc/soc-topology.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/soc-topology.c b/sound/soc/soc-topology.c index 575da6aba807..a152409e8746 100644
--- a/sound/soc/soc-topology.c
+++ b/sound/soc/soc-topology.c
@@ -362,7 +362,7 @@ static int soc_tplg_add_kcontrol(struct soc_tplg *tplg,
 	struct snd_soc_component *comp = tplg->comp;
 
 	return soc_tplg_add_dcontrol(comp->card->snd_card,
-				comp->dev, k, NULL, comp, kcontrol);
+				comp->dev, k, comp->name_prefix, comp,
kcontrol);
 }
 
 /* remove a mixer kcontrol */
--
2.21.0




^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol
       [not found] <CGME20200331054217epcas2p3e98c724d8be292bf9d71f78119547646@epcas2p3.samsung.com>
@ 2020-03-31  5:42 ` 이경택
  0 siblings, 0 replies; 7+ messages in thread
From: 이경택 @ 2020-03-31  5:42 UTC (permalink / raw)
  To: broonie; +Cc: alsa-devel, lgirdwood, tiwai, hmseo, tkjung, pilsun.jang

Current topology doesn't add prefix of the component to newly created
kcontrol.

Signed-off-by: Gyeongtaek Lee <gt82.lee@samsung.com>
---
 sound/soc/soc-topology.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/soc-topology.c b/sound/soc/soc-topology.c index
575da6aba807..a152409e8746 100644
--- a/sound/soc/soc-topology.c
+++ b/sound/soc/soc-topology.c
@@ -362,7 +362,7 @@ static int soc_tplg_add_kcontrol(struct soc_tplg *tplg,
 	struct snd_soc_component *comp = tplg->comp;
 
 	return soc_tplg_add_dcontrol(comp->card->snd_card,
-				comp->dev, k, NULL, comp, kcontrol);
+				comp->dev, k, comp->name_prefix, comp,
kcontrol);
 }
 
 /* remove a mixer kcontrol */
--
2.21.0




^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol
       [not found] <CGME20200330073555epcas2p47495e0e0435e6cf34e6a5d7e248a3e80@epcas2p4.samsung.com>
@ 2020-03-30  7:35 ` 이경택
  2020-03-31 18:07   ` Mark Brown
  0 siblings, 1 reply; 7+ messages in thread
From: 이경택 @ 2020-03-30  7:35 UTC (permalink / raw)
  To: lgirdwood, broonie, tiwai; +Cc: alsa-devel

Current topology doesn't add prefix of the component to newly created
kcontrol.

Signed-off-by: Gyeongtaek Lee <gt82.lee@samsung.com>
---
 sound/soc/soc-topology.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/soc-topology.c b/sound/soc/soc-topology.c
index 575da6aba807..a152409e8746 100644
--- a/sound/soc/soc-topology.c
+++ b/sound/soc/soc-topology.c
@@ -362,7 +362,7 @@ static int soc_tplg_add_kcontrol(struct soc_tplg *tplg,
 	struct snd_soc_component *comp = tplg->comp;
 
 	return soc_tplg_add_dcontrol(comp->card->snd_card,
-				comp->dev, k, NULL, comp, kcontrol);
+				comp->dev, k, comp->name_prefix, comp,
kcontrol);
 }
 
 /* remove a mixer kcontrol */
-- 
2.21.0



^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-04-01  9:23 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20200401004223epcas2p2a2b0f54f83ea8fa0c1a1b96660cf4b69@epcas2p2.samsung.com>
2020-04-01  0:42 ` [PATCH 2/3] ASoC: topology: use name_prefix for new kcontrol 이경택
2020-04-01  8:37   ` Mark Brown
2020-04-01  9:22     ` 이경택
     [not found] <CGME20200331065047epcas2p23455307bb2eed7b62a04e387e6ebf9c1@epcas2p2.samsung.com>
2020-03-31  6:50 ` 이경택
     [not found] <CGME20200331054217epcas2p3e98c724d8be292bf9d71f78119547646@epcas2p3.samsung.com>
2020-03-31  5:42 ` 이경택
     [not found] <CGME20200330073555epcas2p47495e0e0435e6cf34e6a5d7e248a3e80@epcas2p4.samsung.com>
2020-03-30  7:35 ` 이경택
2020-03-31 18:07   ` Mark Brown

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.