Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1530860 > unrolled thread
| Started by | Jonathan Cameron <jic23@kernel.org> |
|---|---|
| First post | 2016-11-27 12:20 +0100 |
| Last post | 2016-11-29 05:30 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
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
| From | Jonathan Cameron <jic23@kernel.org> |
|---|---|
| Date | 2016-11-27 12:20 +0100 |
| Subject | Re: [PATCH 2/3] iio: st_accel: Support sensor i2c probe using acpi |
| Message-ID | <sI5F0-3AT-11@gated-at.bofh.it> |
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
>
[toc] | [next] | [standalone]
| From | Shrirang Bagul <shrirang.bagul@canonical.com> |
|---|---|
| Date | 2016-11-29 05:30 +0100 |
| Message-ID | <sIIdj-3os-11@gated-at.bofh.it> |
| In reply to | #1530860 |
On Sun, 2016-11-27 at 11:14 +0000, Jonathan Cameron wrote:
> 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
I'll take a look at the HW in our lab and send patches to identify more st_accel
sensors used mostly on Dell laptops.
Shrirang
> >
> > 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
> >
>
>
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web