[alsa-devel] [PATCH] ASoC: intel: cht_bsw_max98090_ti: Add pmc_plt_clk_0 quirk for Chromebook Gnawty

[Pierre-Louis Bossart]

On 12/3/18 12:45 PM, Hans de Goede wrote:
> The Gnawty model Chromebook uses pmc_plt_clk_0 instead of pmc_plt_clk_3
> for the mclk, just like the Clapper and Swanky models.
> 
> This commit adds a DMI based quirk for this.
> 
> This fixing audio no longer working on these devices after
> commit 648e921888ad ("clk: x86: Stop marking clocks as CLK_IS_CRITICAL")
> that commit fixes us unnecessary keeping unused clocks on, but in case of
> the Gnawty that was breaking audio support since we were not using the
> right clock in the cht_bsw_max98090_ti machine driver.
> 
> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=201787
> Cc: stable at vger.kernel.org
> Fixes: 648e921888ad ("clk: x86: Stop marking clocks as CLK_IS_CRITICAL")
> Reported-and-tested-by: Jaime Pérez <19.jaime.91 at gmail.com>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>

Acked-by: Pierre-Louis Bossart <pierre-louis.bossart at linux.intel.com>

> ---
>   sound/soc/intel/boards/cht_bsw_max98090_ti.c | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/sound/soc/intel/boards/cht_bsw_max98090_ti.c b/sound/soc/intel/boards/cht_bsw_max98090_ti.c
> index ad0c98383853..08a5152e635a 100644
> --- a/sound/soc/intel/boards/cht_bsw_max98090_ti.c
> +++ b/sound/soc/intel/boards/cht_bsw_max98090_ti.c
> @@ -396,6 +396,13 @@ static const struct dmi_system_id cht_max98090_quirk_table[] = {
>   		},
>   		.driver_data = (void *)QUIRK_PMC_PLT_CLK_0,
>   	},
> +	{
> +		/* Gnawty model Chromebook (Acer Chromebook CB3-111) */
> +		.matches = {
> +			DMI_MATCH(DMI_PRODUCT_NAME, "Gnawty"),
> +		},
> +		.driver_data = (void *)QUIRK_PMC_PLT_CLK_0,
> +	},
>   	{
>   		/* Swanky model Chromebook (Toshiba Chromebook 2) */
>   		.matches = {
>