Message-ID: <20230129182441.082f29d0@jic23-huawei>
Date:   Sun, 29 Jan 2023 18:24:41 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Darrell Kavanagh <darrell.kavanagh@...il.com>
Cc:     lorenzo@...nel.org, lars@...afoo.de, linux-iio@...r.kernel.org,
        linux-kernel@...r.kernel.org, carnil@...ian.org
Subject: Re: Bug#1029850: linux: Driver not loaded for ST Microelectronics
 LSM6DS3TR-C accelerometer (acpi:SMO8B30:SMO8B30:)

On Sun, 29 Jan 2023 17:03:51 +0000
Darrell Kavanagh <darrell.kavanagh@...il.com> wrote:

> Hi,
> 
> I raised this bug in Debian, and have been asked to raise it upstream and
> was given your addresses to do so. Will this email be OK, or should I raise
> it in a bug tracking system somewhere?

Email is the right option.

> 
> Many thanks,
> Darrell
> 
> 
> 
> 
> ---------- Forwarded message ---------
> From: Salvatore Bonaccorso <carnil@...ian.org>
> Date: Sat, 28 Jan 2023 at 20:33
> Subject: Re: Bug#1029850: linux: Driver not loaded for ST Microelectronics
> LSM6DS3TR-C accelerometer (acpi:SMO8B30:SMO8B30:)
> To: Darrell Kavanagh <darrell.kavanagh@...il.com>, <1029850@...s.debian.org>
> 
> 
> Hi Darrell,
> 
> On Sat, Jan 28, 2023 at 08:13:16PM +0000, Darrell Kavanagh wrote:
> > Package: src:linux
> > Version: 6.1.4-1
> > Severity: normal
> > File: linux
> > X-Debbugs-Cc: darrell.kavanagh@...il.com
> >
> > Dear Maintainer,
> >
> > This is a convertable touchscreen tablet/laptop. The rotation sensor  
> device
> > ST Microelectronics LSM6DS3TR-C does not work. It is detected via ACPI  
> and the
> > sysfs trees are created at  
> devices/pci0000:00/0000:00:17.1/i2c_designware.3/i2c-4/i2c-SMO8B30:00
> > and devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:3c/SMO8B30:00 with
> > symlinks bus/acpi/devices/SMO8B30:00 and bus/i2c/devices/i2c-SMO8B30:00,  
> but
> > no driver is loaded.

At least this is using the ST PNP ID which is better than average
(long story!)

The driver in question (ultimately drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
does not currently have an ACPI support.  It should be straight forwards
to add though the driver first needs converting to use
device_get_match_data() with appropriate fallback so that it will match on
ACPI, OF or original spi_device_id tables

Completely untested but something like the following
(the offset in the enum is needed to allow us to tell if we got a result when
calling device_get_match_data() as it returns NULL on failure IIRC)

I'm not sure how sucessful the driver will be at finding any interrupts etc, but
it may get you basic functionality.

Good luck and others more familiar with the driver may well tell me what I forgot
when hacking the below ;)

diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
index 499fcf8875b4..2617ce236ddc 100644
--- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
+++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
@@ -39,7 +39,7 @@
 #define ST_ISM330IS_DEV_NAME   "ism330is"
 
 enum st_lsm6dsx_hw_id {
-       ST_LSM6DS3_ID,
+       ST_LSM6DS3_ID = 1,
        ST_LSM6DS3H_ID,
        ST_LSM6DSL_ID,
        ST_LSM6DSM_ID,
diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
index df5f60925260..ecfceb2fb3db 100644
--- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
+++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
@@ -23,10 +23,15 @@ static const struct regmap_config st_lsm6dsx_i2c_regmap_config = {
 
 static int st_lsm6dsx_i2c_probe(struct i2c_client *client)
 {
-       const struct i2c_device_id *id = i2c_client_get_device_id(client);
-       int hw_id = id->driver_data;
+       int hw_id;
        struct regmap *regmap;
 
+       hw_id = (kernel_ulong_t)device_get_match_data(&client->dev);
+       if (!hw_id)
+               hw_id = i2c_client_get_device_id(client)->driver_data;
+       if (!hw_id)
+               return -EINVAL;
+
        regmap = devm_regmap_init_i2c(client, &st_lsm6dsx_i2c_regmap_config);
        if (IS_ERR(regmap)) {
                dev_err(&client->dev, "Failed to register i2c regmap %ld\n", PTR_ERR(regmap));
@@ -129,6 +134,10 @@ static const struct of_device_id st_lsm6dsx_i2c_of_match[] = {
 };
 MODULE_DEVICE_TABLE(of, st_lsm6dsx_i2c_of_match);
 
+static const struct acpi_device_id st_lsm6dsx_i2c_acpi_match[] = {
+       { "SMO8B30", ST_LSM6DS3TRC_ID, },
+};
+
 static const struct i2c_device_id st_lsm6dsx_i2c_id_table[] = {
        { ST_LSM6DS3_DEV_NAME, ST_LSM6DS3_ID },
        { ST_LSM6DS3H_DEV_NAME, ST_LSM6DS3H_ID },
@@ -161,6 +170,7 @@ static struct i2c_driver st_lsm6dsx_driver = {
                .name = "st_lsm6dsx_i2c",
                .pm = pm_sleep_ptr(&st_lsm6dsx_pm_ops),
                .of_match_table = st_lsm6dsx_i2c_of_match,
+               .acpi_match_table = st_lsm6dsx_i2c_acpi_match,
        },
        .probe_new = st_lsm6dsx_i2c_probe,
        .id_table = st_lsm6dsx_i2c_id_table,


> >
> > The device is identifying itself to the kernel with PNP id SMO8B30:
> > physical_node:
> >       modalias=acpi:SMO8B30:SMO8B30:
> >       name=SMO8B30:00
> >       uevent=MODALIAS=acpi:SMO8B30:SMO8B30:
> >       waiting_for_supplier=0
> > firmware_node:
> >       hid=SMO8B30
> >       modalias=acpi:SMO8B30:SMO8B30:
> >       path=\_SB_.PCI0.I2C5.DEV_
> >       status=15
> >       uevent=MODALIAS=acpi:SMO8B30:SMO8B30:
> >       uid=0
> >
> > The kernel module for the appropriate driver (st_lsm6dsx_i2c) is not  
> loaded on boot.
> > Modprobing it does not associate it with the device, as I would expect as
> > the module does not provide an alias for the above acpi/pnp id.  
> 
> Can you report this issue upstream? Gues to reach out are according to
> get_maintainers.pl script:
> 
> Lorenzo Bianconi <lorenzo@...nel.org> (maintainer:ST LSM6DSx IMU IIO DRIVER)
> Jonathan Cameron <jic23@...nel.org> (maintainer:IIO SUBSYSTEM AND DRIVERS)
> Lars-Peter Clausen <lars@...afoo.de> (reviewer:IIO SUBSYSTEM AND DRIVERS)
> linux-iio@...r.kernel.org (open list:ST LSM6DSx IMU IIO DRIVER)
> linux-kernel@...r.kernel.org (open list)
> 
> Please keep us in the loop.
> 
> Regards,
> Salvatore

Powered by blists - more mailing lists