On Tue, 18 Jul 2023 13:40:15 +0200, Stephan Gerhold wrote: > The audio codec typically used for the MSM8916 SoC is split into two > parts: the digital codec is part of the SoC, while the analog codec is > part of the PM8916 PMIC. > > The analog codec in the PMIC has no direct connection to the mclk of > the SoC (GCC_CODEC_DIGCODEC_CLK). As the name of the clock suggests > this is supplied to the digital part of the codec. During playback it > will use this clock to transmit the audio data via the "CDC PDM" pins > to the PMIC. In this case the analog codec indirectly receives the > clock signal through the digital codec. > > GCC_CODEC_DIGCODEC_CLK is already managed by the driver of the digital > part of the codec in the SoC. Having this clock on the analog PMIC part > additionally is redundant and incorrect because the analog codec cannot > receive the clock signal without going through the digital codec. > > Cc: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx> > Signed-off-by: Stephan Gerhold <stephan@xxxxxxxxxxx> > --- > .../bindings/sound/qcom,pm8916-wcd-analog-codec.yaml | 10 ---------- > 1 file changed, 10 deletions(-) > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>