lkml.org 
[lkml]   [2020]   [Aug]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 0/3] ASoC: da7219: Reorganise device/codec level probe/remove
Date
Hi, Adam,

> -----Original Message-----
> From: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
> Sent: Tuesday, August 11, 2020 11:57 AM
> To: Mark Brown <broonie@kernel.org>; Takashi Iwai <tiwai@suse.com>; Liam
> Girdwood <lgirdwood@gmail.com>; Jaroslav Kysela <perex@perex.cz>
> Cc: Zhi, Yong <yong.zhi@intel.com>; Pierre-Louis Bossart <pierre-
> louis.bossart@linux.intel.com>; alsa-devel@alsa-project.org; linux-
> kernel@vger.kernel.org; Support Opensource
> <support.opensource@diasemi.com>
> Subject: [PATCH 0/3] ASoC: da7219: Reorganise device/codec level
> probe/remove
>
> This patch set reorganises and fixes device and codec level probe/remove
> handling within the driver, to allow clean probe and remove at the codec level.
>
> This set relates to an issue raised by Yong Zhi where a codec level re-probe
> would fail due to clks still being registered from the previous instantiation.
> In addition some improvements around regulator handling and soft reset have
> also been included.
>

Just to let you know that I tested this series on the JSL platform, the original issue was resolved, also sanity checked basic use case like plug/unplug and
suspend/resume, no regression found.

thanks,
yong
> Adam Thomson (3):
> ASoC: da7219: Move required devm_* allocations to device level code
> ASoC: da7219: Move soft reset handling to codec level probe
> ASoC: da7219: Fix clock handling around codec level probe
>
> sound/soc/codecs/da7219-aad.c | 85 +++++---
> sound/soc/codecs/da7219-aad.h | 3 +
> sound/soc/codecs/da7219.c | 493 +++++++++++++++++++++++-----------------
> --
> sound/soc/codecs/da7219.h | 1 +
> 4 files changed, 328 insertions(+), 254 deletions(-)
>
> --
> 1.9.1

\
 
 \ /
  Last update: 2020-08-18 18:40    [W:0.079 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site