Re: [PATCH 2/3] iio: st_accel: Support sensor i2c probe using acpi

From	Jonathan Cameron <jic23@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/3] iio: st_accel: Support sensor i2c probe using acpi
Date	2016-11-27 12:20 +0100
Message-ID	<sI5F0-3AT-11@gated-at.bofh.it> (permalink)
References	<sGUVj-6Ie-3@gated-at.bofh.it> <sGUVj-6Ie-7@gated-at.bofh.it> <sH3ce-3Wy-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 24/11/16 14:26, Linus Walleij wrote:
> On Thu, Nov 24, 2016 at 6:33 AM, Shrirang Bagul
> <shrirang.bagul@canonical.com> wrote:
> 
>> Add support to probe st_accel sensors on i2c bus using ACPI. Compatible
>> strings are not avaialable on ACPI based systems.
>>
>> Signed-off-by: Shrirang Bagul <shrirang.bagul@canonical.com>
> (...)
> 
>> +#ifdef CONFIG_ACPI
>> +static const struct acpi_device_id st_accel_acpi_match[] = {
>> +       {"SMO8A90", LNG2DM},
>> +       { },
>> +};
> 
> Why is ACPI only supporting one out of 14 devices?
> 
> Surely there are some out-of-tree ACPI platforms using one
> or more of the others?
> 
> Apart from that it looks nice.
Applied to the togreg branch of iio.git and pushed out as testing
for the autobuilders to play with it.

Anyone know where the ACPI names come from? Is there some big
list somewhere to prevent clashes?

Jonathan
> 
> Yours,
> Linus Walleij
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/3] iio: st_accel: Support sensor i2c probe using acpi Jonathan Cameron <jic23@kernel.org> - 2016-11-27 12:20 +0100
  Re: [PATCH 2/3] iio: st_accel: Support sensor i2c probe using acpi Shrirang Bagul <shrirang.bagul@canonical.com> - 2016-11-29 05:30 +0100

csiph-web