* [PATCH] tlv320aic3x: control additions and cleanups
@ 2008-12-04 17:51 Daniel Mack
2008-12-04 18:32 ` Jarkko Nikula
` (2 more replies)
0 siblings, 3 replies; 19+ messages in thread
From: Daniel Mack @ 2008-12-04 17:51 UTC (permalink / raw)
To: alsa-devel; +Cc: u.luckas
Some minor updates for sound/soc/codecs/tlv320aic3x.c:
- removed "DAC" from name for "Mono playback switch", "HP playback
switch" and "HPCOM playback switch". The registers used in the macro
don't have anything to do with the DAC but mute the whole output,
hence the names were misleading.
- split "Line Playback Switch" into "LineL Playback Switch" and "LineR
Playback Switch"
- split "Line PGA Bypass Playback Volume" into "LineL Left PGA Bypass
Playback Volume" and "LineR Right PGA Bypass Playback Volume"
- split "Line Line2 Bypass Playback Volume" into "LineL Line2 Bypass
Playback Volume" and "LineR Line2 Bypass Playback Volume"
- Added "HP Right PGA Bypass Playback Volume"
Signed-off-by: Daniel Mack <daniel@caiaq.de>
Signed-off-by: Uli Luckas <u.luckas@road.de>
---
sound/soc/codecs/tlv320aic3x.c | 24 ++++++++++++++++--------
1 files changed, 16 insertions(+), 8 deletions(-)
diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c
index 537bbbf..7cdaea3 100644
--- a/sound/soc/codecs/tlv320aic3x.c
+++ b/sound/soc/codecs/tlv320aic3x.c
@@ -253,16 +253,22 @@ static const struct snd_kcontrol_new aic3x_snd_controls[] = {
SOC_DOUBLE_R("Line DAC Playback Volume", DACL1_2_LLOPM_VOL,
DACR1_2_RLOPM_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("Line DAC Playback Switch", LLOPM_CTRL, RLOPM_CTRL, 3,
- 0x01, 0),
- SOC_DOUBLE_R("Line PGA Bypass Playback Volume", PGAL_2_LLOPM_VOL,
- PGAR_2_RLOPM_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("Line Line2 Bypass Playback Volume", LINE2L_2_LLOPM_VOL,
+ SOC_SINGLE("LineL Playback Switch", LLOPM_CTRL, 3, 0x01, 0),
+ SOC_SINGLE("LineR Playback Switch", RLOPM_CTRL, 3, 0x01, 0),
+ SOC_DOUBLE_R("LineL DAC Playback Volume", DACL1_2_LLOPM_VOL,
+ DACR1_2_LLOPM_VOL, 0, 0x7f, 1),
+ SOC_SINGLE("LineL Left PGA Bypass Playback Volume", PGAL_2_LLOPM_VOL,
+ 0, 0x7f, 1),
+ SOC_SINGLE("LineR Right PGA Bypass Playback Volume", PGAR_2_RLOPM_VOL,
+ 0, 0x7f, 1),
+ SOC_DOUBLE_R("LineL Line2 Bypass Playback Volume", LINE2L_2_LLOPM_VOL,
+ LINE2R_2_LLOPM_VOL, 0, 0x7f, 1),
+ SOC_DOUBLE_R("LineR Line2 Bypass Playback Volume", LINE2L_2_RLOPM_VOL,
LINE2R_2_RLOPM_VOL, 0, 0x7f, 1),
SOC_DOUBLE_R("Mono DAC Playback Volume", DACL1_2_MONOLOPM_VOL,
DACR1_2_MONOLOPM_VOL, 0, 0x7f, 1),
- SOC_SINGLE("Mono DAC Playback Switch", MONOLOPM_CTRL, 3, 0x01, 0),
+ SOC_SINGLE("Mono Playback Switch", MONOLOPM_CTRL, 3, 0x01, 0),
SOC_DOUBLE_R("Mono PGA Bypass Playback Volume", PGAL_2_MONOLOPM_VOL,
PGAR_2_MONOLOPM_VOL, 0, 0x7f, 1),
SOC_DOUBLE_R("Mono Line2 Bypass Playback Volume", LINE2L_2_MONOLOPM_VOL,
@@ -270,8 +276,10 @@ static const struct snd_kcontrol_new aic3x_snd_controls[] = {
SOC_DOUBLE_R("HP DAC Playback Volume", DACL1_2_HPLOUT_VOL,
DACR1_2_HPROUT_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("HP DAC Playback Switch", HPLOUT_CTRL, HPROUT_CTRL, 3,
+ SOC_DOUBLE_R("HP Playback Switch", HPLOUT_CTRL, HPROUT_CTRL, 3,
0x01, 0),
+ SOC_DOUBLE_R("HP Right PGA Bypass Playback Volume", PGAR_2_HPLOUT_VOL,
+ PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
0, 0x7f, 1),
SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
@@ -281,7 +289,7 @@ static const struct snd_kcontrol_new aic3x_snd_controls[] = {
SOC_DOUBLE_R("HPCOM DAC Playback Volume", DACL1_2_HPLCOM_VOL,
DACR1_2_HPRCOM_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("HPCOM DAC Playback Switch", HPLCOM_CTRL, HPRCOM_CTRL, 3,
+ SOC_DOUBLE_R("HPCOM Playback Switch", HPLCOM_CTRL, HPRCOM_CTRL, 3,
0x01, 0),
SOC_SINGLE("HPLCOM PGA Bypass Playback Volume", PGAL_2_HPLCOM_VOL,
0, 0x7f, 1),
--
1.5.6.5
^ permalink raw reply related [flat|nested] 19+ messages in thread* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-04 17:51 [PATCH] tlv320aic3x: control additions and cleanups Daniel Mack
@ 2008-12-04 18:32 ` Jarkko Nikula
2008-12-04 18:43 ` Mark Brown
2008-12-04 18:37 ` Mark Brown
2008-12-04 19:40 ` [PATCH] tlv320aic3x.c: split and add some mixer controls Daniel Mack
2 siblings, 1 reply; 19+ messages in thread
From: Jarkko Nikula @ 2008-12-04 18:32 UTC (permalink / raw)
To: Daniel Mack; +Cc: alsa-devel, u.luckas
On Thu, Dec 4, 2008 at 7:51 PM, Daniel Mack <daniel@caiaq.org> wrote:
> Some minor updates for sound/soc/codecs/tlv320aic3x.c:
>
> - removed "DAC" from name for "Mono playback switch", "HP playback
> switch" and "HPCOM playback switch". The registers used in the macro
> don't have anything to do with the DAC but mute the whole output,
> hence the names were misleading.
Hi Daniel!
This is true, I found it also sometime ago that muting those DAC playback
switches muted the whole output even there were bypass paths active but
didn't find time to look it any deeper yet.
I think by quick look, that your fix changing "x DAC Playback Switch" to "x
Playback Switch" separating it thus from "x DAC Playback Volume" is likely
correct one.
I'll try to look this tomorrow or monday and give you a note. Generally I
would say, this should go as a separate fix patch and changes below into
another.
> - split "Line Playback Switch" into "LineL Playback Switch" and "LineR
> Playback Switch"
> - split "Line PGA Bypass Playback Volume" into "LineL Left PGA Bypass
> Playback Volume" and "LineR Right PGA Bypass Playback Volume"
> - split "Line Line2 Bypass Playback Volume" into "LineL Line2 Bypass
> Playback Volume" and "LineR Line2 Bypass Playback Volume"
> - Added "HP Right PGA Bypass Playback Volume"
>
Jarkko
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-04 18:32 ` Jarkko Nikula
@ 2008-12-04 18:43 ` Mark Brown
0 siblings, 0 replies; 19+ messages in thread
From: Mark Brown @ 2008-12-04 18:43 UTC (permalink / raw)
To: Jarkko Nikula; +Cc: u.luckas, alsa-devel
On Thu, Dec 04, 2008 at 08:32:56PM +0200, Jarkko Nikula wrote:
> I'll try to look this tomorrow or monday and give you a note. Generally I
> would say, this should go as a separate fix patch and changes below into
> another.
Yes, I'd tend to agree with that - small, focused patches are easier to
review and if there's any problem with part of a patch it means only
that part needs to be reposted.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-04 17:51 [PATCH] tlv320aic3x: control additions and cleanups Daniel Mack
2008-12-04 18:32 ` Jarkko Nikula
@ 2008-12-04 18:37 ` Mark Brown
2008-12-04 18:47 ` Daniel Mack
2008-12-05 12:45 ` Jarkko Nikula
2008-12-04 19:40 ` [PATCH] tlv320aic3x.c: split and add some mixer controls Daniel Mack
2 siblings, 2 replies; 19+ messages in thread
From: Mark Brown @ 2008-12-04 18:37 UTC (permalink / raw)
To: Daniel Mack; +Cc: alsa-devel, u.luckas
On Thu, Dec 04, 2008 at 06:51:21PM +0100, Daniel Mack wrote:
> - split "Line Playback Switch" into "LineL Playback Switch" and "LineR
> Playback Switch"
> - split "Line PGA Bypass Playback Volume" into "LineL Left PGA Bypass
> Playback Volume" and "LineR Right PGA Bypass Playback Volume"
I can't help thinking that it'd be easier to just use "Left Line" and
"Right Line" but that's an existing issue in the driver.
> Signed-off-by: Daniel Mack <daniel@caiaq.de>
> Signed-off-by: Uli Luckas <u.luckas@road.de>
Did you write this patch or did Uli? If you wrote it the line for Uli
should be an Acked-by, if he wrote it then your Signed-off-by should
come second and it should have a From: for him at the top.
The patch itself looks good.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-04 18:37 ` Mark Brown
@ 2008-12-04 18:47 ` Daniel Mack
2008-12-04 18:53 ` Mark Brown
2008-12-05 13:02 ` Jarkko Nikula
2008-12-05 12:45 ` Jarkko Nikula
1 sibling, 2 replies; 19+ messages in thread
From: Daniel Mack @ 2008-12-04 18:47 UTC (permalink / raw)
To: alsa-devel, u.luckas
Hi Mark,
On Thu, Dec 04, 2008 at 06:37:47PM +0000, Mark Brown wrote:
> > Signed-off-by: Daniel Mack <daniel@caiaq.de>
> > Signed-off-by: Uli Luckas <u.luckas@road.de>
> Did you write this patch or did Uli? If you wrote it the line for Uli
> should be an Acked-by, if he wrote it then your Signed-off-by should
> come second and it should have a From: for him at the top.
We're working on it together and share a git repository. I commited the
patch, sent it over to Uli to double-check it and he signed off on it
after I did it in the first place.
I thought in such cases, two Signed-off-by: lines coming right after the
other is the way to go, but I can also change it to an 'Acked-by' if you
like.
I'll wait for Jarkko's comments and repost.
Thanks,
Daniel
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-04 18:47 ` Daniel Mack
@ 2008-12-04 18:53 ` Mark Brown
2008-12-05 13:02 ` Jarkko Nikula
1 sibling, 0 replies; 19+ messages in thread
From: Mark Brown @ 2008-12-04 18:53 UTC (permalink / raw)
To: Daniel Mack; +Cc: alsa-devel, u.luckas
On Thu, Dec 04, 2008 at 07:47:51PM +0100, Daniel Mack wrote:
> We're working on it together and share a git repository. I commited the
> patch, sent it over to Uli to double-check it and he signed off on it
> after I did it in the first place.
> I thought in such cases, two Signed-off-by: lines coming right after the
> other is the way to go, but I can also change it to an 'Acked-by' if you
> like.
No, that sounds sensible. Normally for cases like that you'd mention
what Uli's involvement in the patch was in the body of the commit
message.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-04 18:47 ` Daniel Mack
2008-12-04 18:53 ` Mark Brown
@ 2008-12-05 13:02 ` Jarkko Nikula
2008-12-05 13:12 ` Mark Brown
1 sibling, 1 reply; 19+ messages in thread
From: Jarkko Nikula @ 2008-12-05 13:02 UTC (permalink / raw)
To: Daniel Mack; +Cc: alsa-devel, u.luckas
On Thu, Dec 4, 2008 at 8:47 PM, Daniel Mack <daniel@caiaq.org> wrote:
> Hi Mark,
>
> On Thu, Dec 04, 2008 at 06:37:47PM +0000, Mark Brown wrote:
> > > Signed-off-by: Daniel Mack <daniel@caiaq.de>
> > > Signed-off-by: Uli Luckas <u.luckas@road.de>
> > Did you write this patch or did Uli? If you wrote it the line for Uli
> > should be an Acked-by, if he wrote it then your Signed-off-by should
> > come second and it should have a From: for him at the top.
>
> We're working on it together and share a git repository. I commited the
> patch, sent it over to Uli to double-check it and he signed off on it
> after I did it in the first place.
>
> I thought in such cases, two Signed-off-by: lines coming right after the
> other is the way to go, but I can also change it to an 'Acked-by' if you
> like.
>
> I'll wait for Jarkko's comments and repost.
>
> I just checked & tested a fix which changes those "xxx DAC Playback Switch"
to "xxx Playback Switch" are correct. That would also allow to develop new
"xxx Playback Volume" controls and gain 9 dB more of output level :-)
Jarkko
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 13:02 ` Jarkko Nikula
@ 2008-12-05 13:12 ` Mark Brown
2008-12-05 13:37 ` Jarkko Nikula
0 siblings, 1 reply; 19+ messages in thread
From: Mark Brown @ 2008-12-05 13:12 UTC (permalink / raw)
To: Jarkko Nikula; +Cc: u.luckas, alsa-devel
On Fri, Dec 05, 2008 at 03:02:20PM +0200, Jarkko Nikula wrote:
> I just checked & tested a fix which changes those "xxx DAC Playback Switch"
> to "xxx Playback Switch" are correct. That would also allow to develop new
> "xxx Playback Volume" controls and gain 9 dB more of output level :-)
So does the patch have your ack?
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 13:12 ` Mark Brown
@ 2008-12-05 13:37 ` Jarkko Nikula
0 siblings, 0 replies; 19+ messages in thread
From: Jarkko Nikula @ 2008-12-05 13:37 UTC (permalink / raw)
To: Mark Brown; +Cc: u.luckas, alsa-devel
On Fri, Dec 5, 2008 at 3:12 PM, Mark Brown <broonie@sirena.org.uk> wrote:
> On Fri, Dec 05, 2008 at 03:02:20PM +0200, Jarkko Nikula wrote:
>
> > I just checked & tested a fix which changes those "xxx DAC Playback
> Switch"
> > to "xxx Playback Switch" are correct. That would also allow to develop
> new
> > "xxx Playback Volume" controls and gain 9 dB more of output level :-)
>
> So does the patch have your ack?
>
I think patch misses control like "HP Left PGA Bypass Playback Volume"?
Jarkko
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-04 18:37 ` Mark Brown
2008-12-04 18:47 ` Daniel Mack
@ 2008-12-05 12:45 ` Jarkko Nikula
2008-12-05 13:00 ` Mark Brown
1 sibling, 1 reply; 19+ messages in thread
From: Jarkko Nikula @ 2008-12-05 12:45 UTC (permalink / raw)
To: Daniel Mack, alsa-devel, u.luckas
On Thu, Dec 4, 2008 at 8:37 PM, Mark Brown <broonie@sirena.org.uk> wrote:
> On Thu, Dec 04, 2008 at 06:51:21PM +0100, Daniel Mack wrote:
>
> > - split "Line Playback Switch" into "LineL Playback Switch" and "LineR
> > Playback Switch"
> > - split "Line PGA Bypass Playback Volume" into "LineL Left PGA Bypass
> > Playback Volume" and "LineR Right PGA Bypass Playback Volume"
>
> I can't help thinking that it'd be easier to just use "Left Line" and
> "Right Line" but that's an existing issue in the driver.
>
Yeah, same thoughts. Started to look only now, but what's the idea to
separate stereo control into two mono control in commit
54f01916297bafc18bd7df4e2300a0544a84fce3? Change like
- SOC_DOUBLE_R("HP PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
- PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
+ SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
+ 0, 0x7f, 1),
+ SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
+ 0, 0x7f, 1),
Jarkko
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 12:45 ` Jarkko Nikula
@ 2008-12-05 13:00 ` Mark Brown
2008-12-05 13:21 ` Jarkko Nikula
0 siblings, 1 reply; 19+ messages in thread
From: Mark Brown @ 2008-12-05 13:00 UTC (permalink / raw)
To: Jarkko Nikula; +Cc: u.luckas, alsa-devel
On Fri, Dec 05, 2008 at 02:45:04PM +0200, Jarkko Nikula wrote:
> On Thu, Dec 4, 2008 at 8:37 PM, Mark Brown <broonie@sirena.org.uk> wrote:
> > I can't help thinking that it'd be easier to just use "Left Line" and
> > "Right Line" but that's an existing issue in the driver.
> Yeah, same thoughts. Started to look only now, but what's the idea to
> separate stereo control into two mono control in commit
> 54f01916297bafc18bd7df4e2300a0544a84fce3? Change like
Several codecs do this - systems often end up doing things like using
the line inputs as unrelated mono paths rather than as a stereo path.
How likely that is to happen with this codec I don't know.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 13:00 ` Mark Brown
@ 2008-12-05 13:21 ` Jarkko Nikula
2008-12-05 13:42 ` Mark Brown
0 siblings, 1 reply; 19+ messages in thread
From: Jarkko Nikula @ 2008-12-05 13:21 UTC (permalink / raw)
To: Mark Brown; +Cc: u.luckas, alsa-devel
On Fri, Dec 5, 2008 at 3:00 PM, Mark Brown <broonie@sirena.org.uk> wrote:
> On Fri, Dec 05, 2008 at 02:45:04PM +0200, Jarkko Nikula wrote:
> > On Thu, Dec 4, 2008 at 8:37 PM, Mark Brown <broonie@sirena.org.uk>
> wrote:
>
> > > I can't help thinking that it'd be easier to just use "Left Line" and
> > > "Right Line" but that's an existing issue in the driver.
>
> > Yeah, same thoughts. Started to look only now, but what's the idea to
> > separate stereo control into two mono control in commit
> > 54f01916297bafc18bd7df4e2300a0544a84fce3? Change like
>
> Several codecs do this - systems often end up doing things like using
> the line inputs as unrelated mono paths rather than as a stereo path.
> How likely that is to happen with this codec I don't know.
>
Or did I read the idea of patch wrong? Before there were stereo contol for
PGAL -> HPL
PGAR -> HPR
and now
PGAL -> HPL
PGAL -> HPR
PGAR seems to be missing?
- SOC_DOUBLE_R("HP PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
- PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
+ SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
+ 0, 0x7f, 1),
+ SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
+ 0, 0x7f, 1),
Codec itself can route all Line2[L | R], PGA [L | R], DAC [L | R] into all
seven output stages. Which are Mono, Line [L |R] and HP [H |L] with HP
having own volumes for HP positive and negative.
Jarkko
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 13:21 ` Jarkko Nikula
@ 2008-12-05 13:42 ` Mark Brown
2008-12-05 13:46 ` Daniel Mack
2008-12-05 15:11 ` Daniel Mack
0 siblings, 2 replies; 19+ messages in thread
From: Mark Brown @ 2008-12-05 13:42 UTC (permalink / raw)
To: Jarkko Nikula; +Cc: u.luckas, alsa-devel
On Fri, Dec 05, 2008 at 03:21:39PM +0200, Jarkko Nikula wrote:
> On Fri, Dec 5, 2008 at 3:00 PM, Mark Brown <broonie@sirena.org.uk> wrote:
> > On Fri, Dec 05, 2008 at 02:45:04PM +0200, Jarkko Nikula wrote:
> > > 54f01916297bafc18bd7df4e2300a0544a84fce3? Change like
> > Several codecs do this - systems often end up doing things like using
> > the line inputs as unrelated mono paths rather than as a stereo path.
> > How likely that is to happen with this codec I don't know.
> Or did I read the idea of patch wrong? Before there were stereo contol for
> PGAL -> HPL
> PGAR -> HPR
> and now
> PGAL -> HPL
> PGAL -> HPR
> PGAR seems to be missing?
Hrm. This looks like it just substitutes two single controls for one
stereo control, which seems reasonable and shouldn't loose any control:
> - SOC_DOUBLE_R("HP PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
> - PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
> + SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
> + 0, 0x7f, 1),
> + SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
> + 0, 0x7f, 1),
Possibly the controls are mislabeled?
Daniel?
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 13:42 ` Mark Brown
@ 2008-12-05 13:46 ` Daniel Mack
2008-12-05 15:11 ` Daniel Mack
1 sibling, 0 replies; 19+ messages in thread
From: Daniel Mack @ 2008-12-05 13:46 UTC (permalink / raw)
To: Mark Brown; +Cc: alsa-devel, u.luckas
On Fri, Dec 05, 2008 at 01:42:58PM +0000, Mark Brown wrote:
> > > On Fri, Dec 05, 2008 at 02:45:04PM +0200, Jarkko Nikula wrote:
> > and now
>
> > PGAL -> HPL
> > PGAL -> HPR
>
> > PGAR seems to be missing?
>
> Hrm. This looks like it just substitutes two single controls for one
> stereo control, which seems reasonable and shouldn't loose any control:
>
> > - SOC_DOUBLE_R("HP PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
> > - PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
> > + SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
> > + 0, 0x7f, 1),
> > + SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
> > + 0, 0x7f, 1),
>
> Possibly the controls are mislabeled?
We could call them "HPR PGA left Bypass Playback Volume" and "HPR PGA
right Bypass Playback Volume" to be more precise.
Daniel
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 13:42 ` Mark Brown
2008-12-05 13:46 ` Daniel Mack
@ 2008-12-05 15:11 ` Daniel Mack
2008-12-05 15:37 ` Jarkko Nikula
1 sibling, 1 reply; 19+ messages in thread
From: Daniel Mack @ 2008-12-05 15:11 UTC (permalink / raw)
To: Mark Brown; +Cc: alsa-devel, u.luckas
Hi,
On Fri, Dec 05, 2008 at 01:42:58PM +0000, Mark Brown wrote:
> Hrm. This looks like it just substitutes two single controls for one
> stereo control, which seems reasonable and shouldn't loose any control:
>
> > - SOC_DOUBLE_R("HP PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
> > - PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
> > + SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
> > + 0, 0x7f, 1),
> > + SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
> > + 0, 0x7f, 1),
>
> Possibly the controls are mislabeled?
As this is commited already, I'd like to make the naming cleanup part of
the patch with the "DAC" string removals. Is that ok for you?
Apart from that, can the other one be applied?
Best regards,
Daniel
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 15:11 ` Daniel Mack
@ 2008-12-05 15:37 ` Jarkko Nikula
2008-12-05 16:24 ` Mark Brown
0 siblings, 1 reply; 19+ messages in thread
From: Jarkko Nikula @ 2008-12-05 15:37 UTC (permalink / raw)
To: Daniel Mack; +Cc: alsa-devel, u.luckas, Mark Brown
On Fri, Dec 5, 2008 at 5:11 PM, Daniel Mack <daniel@caiaq.org> wrote:
>
> As this is commited already, I'd like to make the naming cleanup part of
> the patch with the "DAC" string removals. Is that ok for you?
>
> Apart from that, can the other one be applied?
>
> Yep, I think this fair move forward. It saves my ass as well ;-)
Jarkko
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] tlv320aic3x: control additions and cleanups
2008-12-05 15:37 ` Jarkko Nikula
@ 2008-12-05 16:24 ` Mark Brown
2008-12-05 16:31 ` [PATCH] [RESEND] " Daniel Mack
0 siblings, 1 reply; 19+ messages in thread
From: Mark Brown @ 2008-12-05 16:24 UTC (permalink / raw)
To: Jarkko Nikula; +Cc: u.luckas, alsa-devel
On Fri, Dec 05, 2008 at 05:37:47PM +0200, Jarkko Nikula wrote:
> On Fri, Dec 5, 2008 at 5:11 PM, Daniel Mack <daniel@caiaq.org> wrote:
> > Apart from that, can the other one be applied?
> Yep, I think this fair move forward. It saves my ass as well ;-)
Could you resend the exact patch that was agreed on, just to make sure?
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH] [RESEND] tlv320aic3x: control additions and cleanups
2008-12-05 16:24 ` Mark Brown
@ 2008-12-05 16:31 ` Daniel Mack
0 siblings, 0 replies; 19+ messages in thread
From: Daniel Mack @ 2008-12-05 16:31 UTC (permalink / raw)
To: Mark Brown; +Cc: alsa-devel, u.luckas
- split "Line Playback Switch" into "LineL Playback Switch" and "LineR
Playback Switch"
- split "Line PGA Bypass Playback Volume" into "LineL Left PGA Bypass
Playback Volume" and "LineR Right PGA Bypass Playback Volume"
- split "Line Line2 Bypass Playback Volume" into "LineL Line2 Bypass
Playback Volume" and "LineR Line2 Bypass Playback Volume"
- Added "HP Right PGA Bypass Playback Volume"
Signed-off-by: Daniel Mack <daniel@caiaq.de>
---
Resent the patch Jarkko acked on.
sound/soc/codecs/tlv320aic3x.c | 18 +++++++++++++-----
1 files changed, 13 insertions(+), 5 deletions(-)
diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c
index 537bbbf..6a12dc3 100644
--- a/sound/soc/codecs/tlv320aic3x.c
+++ b/sound/soc/codecs/tlv320aic3x.c
@@ -253,11 +253,17 @@ static const struct snd_kcontrol_new aic3x_snd_controls[] = {
SOC_DOUBLE_R("Line DAC Playback Volume", DACL1_2_LLOPM_VOL,
DACR1_2_RLOPM_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("Line DAC Playback Switch", LLOPM_CTRL, RLOPM_CTRL, 3,
- 0x01, 0),
- SOC_DOUBLE_R("Line PGA Bypass Playback Volume", PGAL_2_LLOPM_VOL,
- PGAR_2_RLOPM_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("Line Line2 Bypass Playback Volume", LINE2L_2_LLOPM_VOL,
+ SOC_SINGLE("LineL Playback Switch", LLOPM_CTRL, 3, 0x01, 0),
+ SOC_SINGLE("LineR Playback Switch", RLOPM_CTRL, 3, 0x01, 0),
+ SOC_DOUBLE_R("LineL DAC Playback Volume", DACL1_2_LLOPM_VOL,
+ DACR1_2_LLOPM_VOL, 0, 0x7f, 1),
+ SOC_SINGLE("LineL Left PGA Bypass Playback Volume", PGAL_2_LLOPM_VOL,
+ 0, 0x7f, 1),
+ SOC_SINGLE("LineR Right PGA Bypass Playback Volume", PGAR_2_RLOPM_VOL,
+ 0, 0x7f, 1),
+ SOC_DOUBLE_R("LineL Line2 Bypass Playback Volume", LINE2L_2_LLOPM_VOL,
+ LINE2R_2_LLOPM_VOL, 0, 0x7f, 1),
+ SOC_DOUBLE_R("LineR Line2 Bypass Playback Volume", LINE2L_2_RLOPM_VOL,
LINE2R_2_RLOPM_VOL, 0, 0x7f, 1),
SOC_DOUBLE_R("Mono DAC Playback Volume", DACL1_2_MONOLOPM_VOL,
@@ -272,6 +278,8 @@ static const struct snd_kcontrol_new aic3x_snd_controls[] = {
DACR1_2_HPROUT_VOL, 0, 0x7f, 1),
SOC_DOUBLE_R("HP DAC Playback Switch", HPLOUT_CTRL, HPROUT_CTRL, 3,
0x01, 0),
+ SOC_DOUBLE_R("HP Right PGA Bypass Playback Volume", PGAR_2_HPLOUT_VOL,
+ PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
0, 0x7f, 1),
SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
--
1.5.6.5
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH] tlv320aic3x.c: split and add some mixer controls
2008-12-04 17:51 [PATCH] tlv320aic3x: control additions and cleanups Daniel Mack
2008-12-04 18:32 ` Jarkko Nikula
2008-12-04 18:37 ` Mark Brown
@ 2008-12-04 19:40 ` Daniel Mack
2 siblings, 0 replies; 19+ messages in thread
From: Daniel Mack @ 2008-12-04 19:40 UTC (permalink / raw)
To: alsa-devel; +Cc: u.luckas
- split "Line Playback Switch" into "LineL Playback Switch" and "LineR
Playback Switch"
- split "Line PGA Bypass Playback Volume" into "LineL Left PGA Bypass
Playback Volume" and "LineR Right PGA Bypass Playback Volume"
- split "Line Line2 Bypass Playback Volume" into "LineL Line2 Bypass
Playback Volume" and "LineR Line2 Bypass Playback Volume"
- Added "HP Right PGA Bypass Playback Volume"
This patch was colaboratively created with Uli Luckas.
Signed-off-by: Daniel Mack <daniel@caiaq.de>
---
I split the patches now - this one omits the removal of misleading
"DAC"s in some widget names.
sound/soc/codecs/tlv320aic3x.c | 18 +++++++++++++-----
1 files changed, 13 insertions(+), 5 deletions(-)
diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c
index 537bbbf..6a12dc3 100644
--- a/sound/soc/codecs/tlv320aic3x.c
+++ b/sound/soc/codecs/tlv320aic3x.c
@@ -253,11 +253,17 @@ static const struct snd_kcontrol_new aic3x_snd_controls[] = {
SOC_DOUBLE_R("Line DAC Playback Volume", DACL1_2_LLOPM_VOL,
DACR1_2_RLOPM_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("Line DAC Playback Switch", LLOPM_CTRL, RLOPM_CTRL, 3,
- 0x01, 0),
- SOC_DOUBLE_R("Line PGA Bypass Playback Volume", PGAL_2_LLOPM_VOL,
- PGAR_2_RLOPM_VOL, 0, 0x7f, 1),
- SOC_DOUBLE_R("Line Line2 Bypass Playback Volume", LINE2L_2_LLOPM_VOL,
+ SOC_SINGLE("LineL Playback Switch", LLOPM_CTRL, 3, 0x01, 0),
+ SOC_SINGLE("LineR Playback Switch", RLOPM_CTRL, 3, 0x01, 0),
+ SOC_DOUBLE_R("LineL DAC Playback Volume", DACL1_2_LLOPM_VOL,
+ DACR1_2_LLOPM_VOL, 0, 0x7f, 1),
+ SOC_SINGLE("LineL Left PGA Bypass Playback Volume", PGAL_2_LLOPM_VOL,
+ 0, 0x7f, 1),
+ SOC_SINGLE("LineR Right PGA Bypass Playback Volume", PGAR_2_RLOPM_VOL,
+ 0, 0x7f, 1),
+ SOC_DOUBLE_R("LineL Line2 Bypass Playback Volume", LINE2L_2_LLOPM_VOL,
+ LINE2R_2_LLOPM_VOL, 0, 0x7f, 1),
+ SOC_DOUBLE_R("LineR Line2 Bypass Playback Volume", LINE2L_2_RLOPM_VOL,
LINE2R_2_RLOPM_VOL, 0, 0x7f, 1),
SOC_DOUBLE_R("Mono DAC Playback Volume", DACL1_2_MONOLOPM_VOL,
@@ -272,6 +278,8 @@ static const struct snd_kcontrol_new aic3x_snd_controls[] = {
DACR1_2_HPROUT_VOL, 0, 0x7f, 1),
SOC_DOUBLE_R("HP DAC Playback Switch", HPLOUT_CTRL, HPROUT_CTRL, 3,
0x01, 0),
+ SOC_DOUBLE_R("HP Right PGA Bypass Playback Volume", PGAR_2_HPLOUT_VOL,
+ PGAR_2_HPROUT_VOL, 0, 0x7f, 1),
SOC_SINGLE("HPL PGA Bypass Playback Volume", PGAL_2_HPLOUT_VOL,
0, 0x7f, 1),
SOC_SINGLE("HPR PGA Bypass Playback Volume", PGAL_2_HPROUT_VOL,
--
1.5.6.5
^ permalink raw reply related [flat|nested] 19+ messages in thread
end of thread, other threads:[~2008-12-05 16:31 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-04 17:51 [PATCH] tlv320aic3x: control additions and cleanups Daniel Mack
2008-12-04 18:32 ` Jarkko Nikula
2008-12-04 18:43 ` Mark Brown
2008-12-04 18:37 ` Mark Brown
2008-12-04 18:47 ` Daniel Mack
2008-12-04 18:53 ` Mark Brown
2008-12-05 13:02 ` Jarkko Nikula
2008-12-05 13:12 ` Mark Brown
2008-12-05 13:37 ` Jarkko Nikula
2008-12-05 12:45 ` Jarkko Nikula
2008-12-05 13:00 ` Mark Brown
2008-12-05 13:21 ` Jarkko Nikula
2008-12-05 13:42 ` Mark Brown
2008-12-05 13:46 ` Daniel Mack
2008-12-05 15:11 ` Daniel Mack
2008-12-05 15:37 ` Jarkko Nikula
2008-12-05 16:24 ` Mark Brown
2008-12-05 16:31 ` [PATCH] [RESEND] " Daniel Mack
2008-12-04 19:40 ` [PATCH] tlv320aic3x.c: split and add some mixer controls Daniel Mack
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.