* duplicated test in sound/soc/codecs/sgtl5000.c
@ 2012-01-12 21:13 Julia Lawall
2012-01-12 22:28 ` Mark Brown
0 siblings, 1 reply; 6+ messages in thread
From: Julia Lawall @ 2012-01-12 21:13 UTC (permalink / raw)
To: lrg, broonie, alsa-devel
There is a duplicated test in the function sgtl5000_restore_regs in the
file sound/soc/codecs/sgtl5000.c:
/* this regs depends on the others */
if (reg == SGTL5000_CHIP_ANA_POWER ||
reg == SGTL5000_CHIP_CLK_CTRL ||
reg == SGTL5000_CHIP_LINREG_CTRL ||
reg == SGTL5000_CHIP_LINE_OUT_CTRL ||
reg == SGTL5000_CHIP_CLK_CTRL)
continue;
I don't know if some constant other than SGTL5000_CHIP_CLK_CTRL should be
tested, or if the second test should just be deleted.
julia
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: duplicated test in sound/soc/codecs/sgtl5000.c
2012-01-12 21:13 duplicated test in sound/soc/codecs/sgtl5000.c Julia Lawall
@ 2012-01-12 22:28 ` Mark Brown
2012-01-13 6:21 ` Dong Aisheng-B29396
0 siblings, 1 reply; 6+ messages in thread
From: Mark Brown @ 2012-01-12 22:28 UTC (permalink / raw)
To: Julia Lawall, Fabio Estevam, Wolfram Sang, Dong Aisheng, Shawn Guo
Cc: alsa-devel, lrg
On Thu, Jan 12, 2012 at 10:13:24PM +0100, Julia Lawall wrote:
> There is a duplicated test in the function sgtl5000_restore_regs in the
> file sound/soc/codecs/sgtl5000.c:
> /* this regs depends on the others */
> if (reg == SGTL5000_CHIP_ANA_POWER ||
> reg == SGTL5000_CHIP_CLK_CTRL ||
> reg == SGTL5000_CHIP_LINREG_CTRL ||
> reg == SGTL5000_CHIP_LINE_OUT_CTRL ||
> reg == SGTL5000_CHIP_CLK_CTRL)
> continue;
> I don't know if some constant other than SGTL5000_CHIP_CLK_CTRL should be
> tested, or if the second test should just be deleted.
Adding the people who've been working on the driver. Generally this is
a good idea when reporting issues like this.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: duplicated test in sound/soc/codecs/sgtl5000.c
2012-01-12 22:28 ` Mark Brown
@ 2012-01-13 6:21 ` Dong Aisheng-B29396
2012-01-13 6:37 ` Mark Brown
2012-01-13 8:40 ` Zeng Zhaoming
0 siblings, 2 replies; 6+ messages in thread
From: Dong Aisheng-B29396 @ 2012-01-13 6:21 UTC (permalink / raw)
To: Mark Brown, Julia Lawall, Fabio Estevam, Wolfram Sang, Shawn Guo
Cc: alsa-devel, lrg, zengzm.kernel
> -----Original Message-----
> From: Mark Brown [mailto:broonie@opensource.wolfsonmicro.com]
> Sent: Friday, January 13, 2012 6:28 AM
> To: Julia Lawall; Fabio Estevam; Wolfram Sang; Dong Aisheng-B29396; Shawn Guo
> Cc: lrg@ti.com; alsa-devel@alsa-project.org
> Subject: Re: duplicated test in sound/soc/codecs/sgtl5000.c
> Importance: High
>
> On Thu, Jan 12, 2012 at 10:13:24PM +0100, Julia Lawall wrote:
> > There is a duplicated test in the function sgtl5000_restore_regs in
> > the file sound/soc/codecs/sgtl5000.c:
>
> > /* this regs depends on the others */
> > if (reg == SGTL5000_CHIP_ANA_POWER ||
> > reg == SGTL5000_CHIP_CLK_CTRL ||
> > reg == SGTL5000_CHIP_LINREG_CTRL ||
> > reg == SGTL5000_CHIP_LINE_OUT_CTRL ||
> > reg == SGTL5000_CHIP_CLK_CTRL)
> > continue;
>
> > I don't know if some constant other than SGTL5000_CHIP_CLK_CTRL should
> > be tested, or if the second test should just be deleted.
>
> Adding the people who've been working on the driver. Generally this is a good
> idea when reporting issues like this.
Cc Zhaoming.
Looking at the code, I guess it's a typo error and the second check should be
SGTL5000_CHIP_REF_CTRL since it will restore it again.
Hi Mark,
Since zhaoming leaved Freescale a few month ago,
would you mind if I send a patch to add my name under MODULE_AHTOER as
Co-maintainer for this driver and update Zhaoming's email address?
Regards
Dong Aisheng
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: duplicated test in sound/soc/codecs/sgtl5000.c
2012-01-13 6:21 ` Dong Aisheng-B29396
@ 2012-01-13 6:37 ` Mark Brown
2012-01-13 6:49 ` Dong Aisheng-B29396
2012-01-13 8:40 ` Zeng Zhaoming
1 sibling, 1 reply; 6+ messages in thread
From: Mark Brown @ 2012-01-13 6:37 UTC (permalink / raw)
To: Dong Aisheng-B29396
Cc: alsa-devel, Wolfram Sang, Julia Lawall, zengzm.kernel, Shawn Guo,
Fabio Estevam, lrg
On Fri, Jan 13, 2012 at 06:21:00AM +0000, Dong Aisheng-B29396 wrote:
> Since zhaoming leaved Freescale a few month ago,
> would you mind if I send a patch to add my name under MODULE_AHTOER as
> Co-maintainer for this driver and update Zhaoming's email address?
Typically the way this ends up getting done is that the MODULE_AUTHOR is
left to bitrot but a MAINTAINERS file entry is added which reflects
who's actually working on the driver. Comments also get added
sometimes. But really I don't mind.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: duplicated test in sound/soc/codecs/sgtl5000.c
2012-01-13 6:37 ` Mark Brown
@ 2012-01-13 6:49 ` Dong Aisheng-B29396
0 siblings, 0 replies; 6+ messages in thread
From: Dong Aisheng-B29396 @ 2012-01-13 6:49 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, Wolfram Sang, Julia Lawall, zengzm.kernel, Shawn Guo,
Fabio Estevam, lrg
> -----Original Message-----
> From: alsa-devel-bounces@alsa-project.org [mailto:alsa-devel-bounces@alsa-
> project.org] On Behalf Of Mark Brown
> Sent: Friday, January 13, 2012 2:38 PM
> To: Dong Aisheng-B29396
> Cc: alsa-devel@alsa-project.org; Wolfram Sang; Julia Lawall;
> zengzm.kernel@gmail.com; Shawn Guo; Fabio Estevam; lrg@ti.com
> Subject: Re: [alsa-devel] duplicated test in sound/soc/codecs/sgtl5000.c
> Importance: High
>
> On Fri, Jan 13, 2012 at 06:21:00AM +0000, Dong Aisheng-B29396 wrote:
>
> > Since zhaoming leaved Freescale a few month ago, would you mind if I
> > send a patch to add my name under MODULE_AHTOER as Co-maintainer for
> > this driver and update Zhaoming's email address?
>
> Typically the way this ends up getting done is that the MODULE_AUTHOR is left to
> bitrot but a MAINTAINERS file entry is added which reflects who's actually
> working on the driver. Comments also get added sometimes.
>
Agree. Thanks.
> But really I don't mind.
>
Regards
Dong Aisheng
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: duplicated test in sound/soc/codecs/sgtl5000.c
2012-01-13 6:21 ` Dong Aisheng-B29396
2012-01-13 6:37 ` Mark Brown
@ 2012-01-13 8:40 ` Zeng Zhaoming
1 sibling, 0 replies; 6+ messages in thread
From: Zeng Zhaoming @ 2012-01-13 8:40 UTC (permalink / raw)
To: Dong Aisheng-B29396
Cc: alsa-devel, Mark Brown, Wolfram Sang, Julia Lawall, Shawn Guo,
Fabio Estevam, lrg
On Fri 2012-01-13 06:21:00, Dong Aisheng-B29396 wrote:
> > -----Original Message-----
> > From: Mark Brown [mailto:broonie@opensource.wolfsonmicro.com]
> > Sent: Friday, January 13, 2012 6:28 AM
> > To: Julia Lawall; Fabio Estevam; Wolfram Sang; Dong Aisheng-B29396; Shawn Guo
> > Cc: lrg@ti.com; alsa-devel@alsa-project.org
> > Subject: Re: duplicated test in sound/soc/codecs/sgtl5000.c
> > Importance: High
> >
> > On Thu, Jan 12, 2012 at 10:13:24PM +0100, Julia Lawall wrote:
> > > There is a duplicated test in the function sgtl5000_restore_regs in
> > > the file sound/soc/codecs/sgtl5000.c:
> >
> > > /* this regs depends on the others */
> > > if (reg == SGTL5000_CHIP_ANA_POWER ||
> > > reg == SGTL5000_CHIP_CLK_CTRL ||
> > > reg == SGTL5000_CHIP_LINREG_CTRL ||
> > > reg == SGTL5000_CHIP_LINE_OUT_CTRL ||
> > > reg == SGTL5000_CHIP_CLK_CTRL)
> > > continue;
> >
> > > I don't know if some constant other than SGTL5000_CHIP_CLK_CTRL should
> > > be tested, or if the second test should just be deleted.
> >
> > Adding the people who've been working on the driver. Generally this is a good
> > idea when reporting issues like this.
> Cc Zhaoming.
>
> Looking at the code, I guess it's a typo error and the second check should be
> SGTL5000_CHIP_REF_CTRL since it will restore it again.
Hi, Aisheng and Julia:
Yes, a typo, I will send patch to correct it.
And Julia, the five regs list there since it should be restore in a particular
order, the order is:
1. SGTL5000_CHIP_CLK_CTRL MCLK_FREQ bits (1:0) should be restore after
SGTL5000_CHIP_ANA_POWER PLL bits set.
2. SGTL5000_CHIP_LINREG_CTRL should be set before SGTL5000_CHIP_ANA_POWER LINREG_D
restored.
3. SGTL5000_CHIP_REF_CTRL controls Analog Ground Voltage, I prefer to resotre it
after SGTL5000_CHIP_ANA_POWER restored.
4. for SGTL5000_CHIP_LINE_OUT_CTRL, I forget the real reason, Aisheng, can you help
verify line-out if remove it?
>
> Hi Mark,
> Since zhaoming leaved Freescale a few month ago,
> would you mind if I send a patch to add my name under MODULE_AHTOER as
> Co-maintainer for this driver and update Zhaoming's email address?
>
> Regards
> Dong Aisheng
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-01-13 8:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-12 21:13 duplicated test in sound/soc/codecs/sgtl5000.c Julia Lawall
2012-01-12 22:28 ` Mark Brown
2012-01-13 6:21 ` Dong Aisheng-B29396
2012-01-13 6:37 ` Mark Brown
2012-01-13 6:49 ` Dong Aisheng-B29396
2012-01-13 8:40 ` Zeng Zhaoming
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.