> 
> That setting is not wrong, but is it sufficient?
> 
> see e.g. what we set for existing platforms which need 24 Mhz in this
> driver:
> 
> DMI quirks:
> 
>       {
>               .callback = sof_rt5682_quirk_cb,
>               .matches = {
>                       DMI_MATCH(DMI_PRODUCT_FAMILY,
> "Google_Hatch"),
>               },
>               .driver_data = (void *)(SOF_RT5682_MCLK_EN |
>                                       SOF_RT5682_MCLK_24MHZ |
>                                       SOF_RT5682_SSP_CODEC(0) |
>                                       SOF_SPEAKER_AMP_PRESENT |
>                                       SOF_RT5682_SSP_AMP(1)),
>       },
> 

Hi Pierre,

CML chromebox is using DMI quirk so we don't need this patch. Sorry
for not noticing it.

Handle 0x0001, DMI type 1, 27 bytes
System Information
        Manufacturer: Google
        Product Name: Duffy
        Version: rev3
        Serial Number: 123456789
        UUID: Not Settable
        Wake-up Type: Reserved
        SKU Number: sku33554439
        Family: Google_Hatch


Reply via email to