ACK/cmnt: [B][PATCH 1/1] iio: st_sensors: remap SMO8840 to LIS2DH12

[Kleber Souza]

On 30.03.20 12:49, Wen-chien Jesse Sung wrote:
> BugLink: https://launchpad.net/bugs/1869694
> 
> According to ST, the HID is for LIS2DH12.
> 
> Fixes: 3d56e19815b3 ("iio: accel: st_accel: Add support for the SMO8840 ACPI id")
> Signed-off-by: Wen-chien Jesse Sung <jesse.sung at canonical.com>
> Tested-by: Hans de Goede <hdegoede at redhat.com>
> Reviewed-by: Hans de Goede <hdegoede at redhat.com>
> Cc: <Stable at vger.kernel.org>
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron at huawei.com>
> (backported from commit e43d110cdc206b6df4dd438cd10c81d1da910aad)
> Signed-off-by: Wen-chien Jesse Sung <jesse.sung at canonical.com>

Hi Jesse,

The backport looks good to me. However, as the code in Bionic is very
different from the newer series, could you please provide a small
explanation about the backport?

Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>

> ---
>  drivers/iio/accel/st_accel_i2c.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/iio/accel/st_accel_i2c.c b/drivers/iio/accel/st_accel_i2c.c
> index 363429b5686c..90d7b4bebb74 100644
> --- a/drivers/iio/accel/st_accel_i2c.c
> +++ b/drivers/iio/accel/st_accel_i2c.c
> @@ -107,6 +107,7 @@ MODULE_DEVICE_TABLE(of, st_accel_of_match);
>  
>  #ifdef CONFIG_ACPI
>  static const struct acpi_device_id st_accel_acpi_match[] = {
> +	{"SMO8840", LIS2DH12},
>  	{"SMO8A90", LNG2DM},
>  	{ },
>  };
>