[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20221214221035.25347427@wsk>
Date: Wed, 14 Dec 2022 22:10:35 +0100
From: Lukasz Majewski <lukma@...x.de>
To: Mark Brown <broonie@...nel.org>
Cc: Liam Girdwood <lgirdwood@...il.com>,
Jaroslav Kysela <perex@...ex.cz>,
Takashi Iwai <tiwai@...e.com>,
Charles Keepax <ckeepax@...nsource.cirrus.com>,
Stephen Kitt <steve@....org>, patches@...nsource.cirrus.com,
alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/4] ASoC: wm8940: Rewrite code to set proper clocks
Hi Mark,
> On Wed, Dec 14, 2022 at 01:37:41PM +0100, Lukasz Majewski wrote:
>
> > Without this change, the wm8940 driver is not working when
> > set_sysclk callback (wm8940_set_dai_sysclk) is called with
> > frequency not listed in the switch clause.
>
> Your commit log doesn't actually describe what this change is...
>
> > This change adjusts this driver to allow non-standard frequency
> > set (just after the boot) being adjusted afterwards by the sound
> > system core code.
>
> This doesn't appear to correspond to the commit. The change will
> result in the driver automatically configuring it's PLL.
>
Yes, indeed - I will update the description.
The problem with the old code was that after boot up - the frequency
was not in the 'switch' values, so the driver aborted in early init.
> > Code in this commit is based on previous change done for wm8974
> > (SHA1: 51b2bb3f2568e6d9d81a001d38b8d70c2ba4af99).
>
> Please include human readable descriptions of things like commits and
> issues being discussed in e-mail in your mails, this makes them much
> easier for humans to read especially when they have no internet
> access. I do frequently catch up on my mail on flights or while
> otherwise travelling so this is even more pressing for me than just
> being about making things a bit easier to read.
Ok. I will provide proper description.
>
> > @@ -496,7 +505,6 @@ static int wm8940_set_bias_level(struct
> > snd_soc_component *component, return ret;
> > }
> > }
> > -
> > /* ensure bufioen and biasen */
> > pwr_reg |= (1 << 2) | (1 << 3);
> > /* set vmid to 300k for standby */
>
> Unrelated (and questionable) whitespace change.
Ok. I will remove it.
Thanks for the review.
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@...x.de
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists