• Stephan Gerhold's avatar
    ASoC: dt-bindings: pm8916-analog-codec: Drop invalid mclk · 469c6d9c
    Stephan Gerhold authored
    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@linaro.org>
    Signed-off-by: default avatarStephan Gerhold <stephan@gerhold.net>
    Reviewed-by: default avatarRob Herring <robh@kernel.org>
    Link: https://lore.kernel.org/r/20230718-pm8916-mclk-v1-3-4b4a58b4240a@gerhold.netSigned-off-by: default avatarMark Brown <broonie@kernel.org>
    469c6d9c
qcom,pm8916-wcd-analog-codec.yaml 4.33 KB