mbox series

[v2,0/3] ASoC: max98090: fix PLL unlocked workaround-related

Message ID 20191122073114.219945-1-tzungbi@google.com (mailing list archive)
Headers show
Series ASoC: max98090: fix PLL unlocked workaround-related | expand

Message

Tzung-Bi Shih Nov. 22, 2019, 7:31 a.m. UTC 
This series is a follow up fix for the question:
https://mailman.alsa-project.org/pipermail/alsa-devel/2019-October/157364.html

Changes from v1:
ASoC: max98090: remove msleep in PLL unlocked workaround
- add more comments
https://mailman.alsa-project.org/pipermail/alsa-devel/2019-November/158969.html

ASoC: max98090: exit workaround earlier if PLL is locked
- "sleep first and then check the status"
https://mailman.alsa-project.org/pipermail/alsa-devel/2019-November/158998.html

ASoC: max98090: fix possible race conditions
- fix typo
https://mailman.alsa-project.org/pipermail/alsa-devel/2019-November/158966.html
- add more descriptions in commit message

Tzung-Bi Shih (3):
  ASoC: max98090: remove msleep in PLL unlocked workaround
  ASoC: max98090: exit workaround earlier if PLL is locked
  ASoC: max98090: fix possible race conditions

 sound/soc/codecs/max98090.c | 30 +++++++++++++++++++++---------
 sound/soc/codecs/max98090.h |  1 -
 2 files changed, 21 insertions(+), 10 deletions(-)

Comments

Pierre-Louis Bossart Nov. 22, 2019, 3:21 p.m. UTC | #1 
On 11/22/19 1:31 AM, Tzung-Bi Shih wrote:
> This series is a follow up fix for the question:
> https://mailman.alsa-project.org/pipermail/alsa-devel/2019-October/157364.html

For the series

Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

Hope this is the last time we see this PLL unlock error :-)


> 
> Changes from v1:
> ASoC: max98090: remove msleep in PLL unlocked workaround
> - add more comments
> https://mailman.alsa-project.org/pipermail/alsa-devel/2019-November/158969.html
> 
> ASoC: max98090: exit workaround earlier if PLL is locked
> - "sleep first and then check the status"
> https://mailman.alsa-project.org/pipermail/alsa-devel/2019-November/158998.html
> 
> ASoC: max98090: fix possible race conditions
> - fix typo
> https://mailman.alsa-project.org/pipermail/alsa-devel/2019-November/158966.html
> - add more descriptions in commit message
> 
> Tzung-Bi Shih (3):
>    ASoC: max98090: remove msleep in PLL unlocked workaround
>    ASoC: max98090: exit workaround earlier if PLL is locked
>    ASoC: max98090: fix possible race conditions
> 
>   sound/soc/codecs/max98090.c | 30 +++++++++++++++++++++---------
>   sound/soc/codecs/max98090.h |  1 -
>   2 files changed, 21 insertions(+), 10 deletions(-)
>