[v4,0/5] ACPI: platform: Fix SMB0001 enumeration on Kontron devices

Message

Andy Shevchenko July 3, 2023, 12:14 p.m. UTC

After switching i2c-scmi driver to be a plaform one, it stopped
being enumerated on number of Kontron platforms, because it's
listed in the forbidden_id_list.

To resolve the situation, add a flag and check if there are resources
privided for the forbiden device. If it is the case, the device must
be skipped.

Changelog v4:
- dropped applied patch
- added kernel doc for new helper (Rafael)
- rewritten the commit message for the reason of a new helper (Rafael)
- added a couple of patches for ACPI scan.c that can be applied later on

Changelog v3:
- provided completely rewritten solution (Rafael)
- due to above, added two new patches
- due to above, dropped tags from patch 3

Andy Shevchenko (5):
  ACPI: bus: Introduce acpi_match_acpi_device() helper
  ACPI: platform: Ignore SMB0001 only when it has resources
  ACPI: platform: Move SMB0001 HID to the header and reuse
  ACPI: scan: Use the acpi_match_acpi_device() helper
  ACPI: scan: Provide symbol declarations

 drivers/acpi/acpi_platform.c  | 30 +++++++++++++++++++++++++++---
 drivers/acpi/bus.c            | 25 +++++++++++++++++++++----
 drivers/acpi/scan.c           | 22 +++++++++++++---------
 drivers/i2c/busses/i2c-scmi.c |  3 ---
 include/acpi/acpi_drivers.h   |  2 ++
 include/linux/acpi.h          |  9 +++++++++
 6 files changed, 72 insertions(+), 19 deletions(-)

Comments

Andy Shevchenko July 3, 2023, 1:09 p.m. UTC | #1

On Mon, Jul 03, 2023 at 03:14:06PM +0300, Andy Shevchenko wrote:
> After switching i2c-scmi driver to be a plaform one, it stopped
> being enumerated on number of Kontron platforms, because it's
> listed in the forbidden_id_list.
> 
> To resolve the situation, add a flag and check if there are resources
> privided for the forbiden device. If it is the case, the device must
> be skipped.

...

> Changelog v4:
> - dropped applied patch
> - added kernel doc for new helper (Rafael)
> - rewritten the commit message for the reason of a new helper (Rafael)
> - added a couple of patches for ACPI scan.c that can be applied later on

- converted to use boolean instead of integer (Rafael)
- added tag (Andi)

> Changelog v3:
> - provided completely rewritten solution (Rafael)
> - due to above, added two new patches
> - due to above, dropped tags from patch 3