Hi, On 12/30/21 13:21, Wolfram Sang wrote: > > Okay, I have a question, after all :) > >> +static const struct acpi_device_id i2c_acpi_known_good_ids[] = { >> + { "10EC5640", 0 }, /* RealTek ALC5640 audio codec */ >> + { "INT33F4", 0 }, /* X-Powers AXP288 PMIC */ >> + { "INT33FD", 0 }, /* Intel Crystal Cove PMIC */ >> + { "NPCE69A", 0 }, /* Asus Transformer keyboard dock */ >> + {} >> +}; > > Can't we add this table to patch 1 and check it within a > acpi_quirk_skip_i2c_client_enumeration(adev)? Yes that will keep all the quirk-handling / ugliness together in a single place, so that is good idea. I will change this for v2 of the series. Regards, Hans