On Tue, Aug 15, 2023 at 11:28 AM Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> wrote: > > > > On 8/15/23 13:00, Curtis Malainey wrote: > > On Tue, Aug 15, 2023 at 10:15 AM Brady Norander <bradynorander@xxxxxxxxx> wrote: > >> > >> On Tue, Aug 15, 2023 at 09:56:44AM -0700, Curtis Malainey wrote: > >>> On Tue, Aug 15, 2023 at 8:56 AM Pierre-Louis Bossart < > >>> pierre-louis.bossart@xxxxxxxxxxxxxxx> wrote: > >>>> On 8/15/23 08:52, Brady Norander wrote: > >>>>> AlderLake and RaptorLake Chromebooks currently use the HDA driver by > >>>>> default. Add a quirk to use the SOF driver on these platforms, which is > >>>>> needed for functional internal audio. > >>>>> > >>>>> Signed-off-by: Brady Norander <bradynorander@xxxxxxxxx> > >>>> > >>>> Thanks for the patch > >>>> > >>>> Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> > >>>> > >>> > >>> Now that I am looking closer at the patch, I am very curious why we are not > >>> hitting the following case and using SOF > >> > >> The DMIC detection looks for DMIC endpoints in the NHLT, but these > >> devices don't have NHLT. > > > > There would be the brokenness then. Is the DMIC not something that > > shows up in coreboot? Or is it only defined in the topology? If the > > latter we should probably extend this patch farther back to GLK. > > You don't need to expose mics in NHLT, the premise has always been that > Chromebooks are assumed to expose 4 mics and userspace w/ UCM takes care > of selecting which mics are actually used on what platform. > > Also there's a difference between NHLT (ACPI table) and NHLT blob in > topology used for IPC4. not the same thing. Thanks for the clarification Acked-by Curtis Malainey <cujomalainey@xxxxxxxxxxxx>