On Monday 24 June 2024 13:15:12 Hans de Goede wrote: > Hans de Goede (6): > i2c: core: Setup i2c_adapter runtime-pm before calling device_add() > i2c: i801: Use a different adapter-name for IDF adapters > platform/x86: dell-smo8800: Move SMO88xx acpi_device_ids to > dell-smo8800-ids.h > platform/x86: dell-smo8800: Move instantiation of lis3lv02d i2c_client > from i2c-i801 to dell-lis3lv02d > platform/x86: dell-smo8800: Add a couple more models to > lis3lv02d_devices[] > platform/x86: dell-smo8800: Add support for probing for the > accelerometer i2c address Patches 1-5 looks good. There are just a few minor things, but you can add Reviewed-by: Pali Rohár <pali@xxxxxxxxxx> For patch 6 as I mentioned previously I'm strictly against this change until somebody goes and politely ask Dell about the current situation of the discovering of accelerometer's i2c address. And if there is no other option than start discussion if Dell can include this information into DMI / ACPI / WMI or other part of firmware data which they can send from BIOS/UEFI to operating system. > drivers/i2c/busses/i2c-i801.c | 133 +------- > drivers/i2c/i2c-core-base.c | 18 +- > drivers/platform/x86/dell/Makefile | 1 + > drivers/platform/x86/dell/dell-lis3lv02d.c | 331 +++++++++++++++++++ > drivers/platform/x86/dell/dell-smo8800-ids.h | 26 ++ > drivers/platform/x86/dell/dell-smo8800.c | 16 +- > 6 files changed, 379 insertions(+), 146 deletions(-) > create mode 100644 drivers/platform/x86/dell/dell-lis3lv02d.c > create mode 100644 drivers/platform/x86/dell/dell-smo8800-ids.h > > -- > 2.45.1 >
