[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACRpkdYJAwDgpuTYuXuTAxu3RBHeATEpbxq_xE3LxrJ7D9CKBQ@mail.gmail.com>
Date: Thu, 24 Nov 2016 15:28:00 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: Shrirang Bagul <shrirang.bagul@...onical.com>
Cc: Jonathan Cameron <jic23@...nel.org>,
Hartmut Knaack <knaack.h@....de>,
Lars-Peter Clausen <lars@...afoo.de>,
Peter Meerwald <pmeerw@...erw.net>,
Denis CIOCCA <denis.ciocca@...com>,
Tiberiu Breana <tiberiu.a.breana@...el.com>,
Lorenzo Bianconi <lorenzo.bianconi83@...il.com>,
Gregor Boirie <gregor.boirie@...rot.com>,
"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 3/3] iio: st_pressure: Support i2c probe using acpi
On Thu, Nov 24, 2016 at 6:33 AM, Shrirang Bagul
<shrirang.bagul@...onical.com> wrote:
> Compatible strings are not available on ACPI based systems. This patch adds
> support to use DSDT information read from platform BIOS instead for probing
> st pressure sensors.
>
> Signed-off-by: Shrirang Bagul <shrirang.bagul@...onical.com>
(...)
> +#ifdef CONFIG_ACPI
> +static const struct acpi_device_id st_press_acpi_match[] = {
> + {"SNO9210", LPS22HB},
> + { },
> +};
Same comment. One sensor only supported by ACPI really?
Take a wider look.
Yours,
Linus Walleij
Powered by blists - more mailing lists