On Thu, 2022-09-08 at 13:02 +0300, Andy Shevchenko wrote: > On Thu, Sep 08, 2022 at 09:48:29AM +0200, Josef Johansson wrote: > > On 9/8/22 08:07, Josef Johansson wrote: > > > On 9/7/22 21:47, Wolfram Sang wrote: > > > > On Tue, Sep 06, 2022 at 06:55:07PM +0300, Andy Shevchenko > > > > wrote: > > > > > ACPI core in conjunction with platform driver core provides > > > > > an infrastructure to enumerate ACPI devices. Use it in order > > > > > to remove a lot of boilerplate code. > > > > > > > > > > Signed-off-by: Andy Shevchenko < > > > > > andriy.shevchenko@xxxxxxxxxxxxxxx> > > > > Josef, do you have resources to test this patch before I apply > > > > it? > > > > > > > Yes, I'll make that happen today. > > Hi, > > > > I compiled with linux-6.0.0-rc4 with your patch on top. > > > > Have been running flawless so far. Boot showed no problems. > > > > Thanks! > > Thank you! > Hi, We just noticed that this change prevents the usage of the i2c-scmi driver on basically all Kontron COMe based boards. The reason is the patch "ACPI / platform: Add SMB0001 HID to forbidden_id_list" submitted in November 2018 by Hans de Goede. The patch blacklists the SMB0001 HID that is also used by the COMe boards. This was due to issues with HP AMD based laptops according to the commit message. Ironically the commit message there states that it is OK to blacklist the HID as the device directly binds to the acpi_bus and therefore the platform_device is not needed anyway. This changed with this patch. As this affects all systems using this HID, applying a patch that whitelists specific boards again in the acpi-platform driver doesn't seem to be a good solution to me. Therefore I would request to remove this patch again, unless someone has a better idea. Thanks, Michael Brunner
