From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:41185 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753582AbcK0LOy (ORCPT ); Sun, 27 Nov 2016 06:14:54 -0500 Subject: Re: [PATCH 2/3] iio: st_accel: Support sensor i2c probe using acpi To: Linus Walleij , Shrirang Bagul References: <20161124053345.11254-1-shrirang.bagul@canonical.com> <20161124053345.11254-3-shrirang.bagul@canonical.com> Cc: Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald , Denis CIOCCA , Tiberiu Breana , Lorenzo Bianconi , Gregor Boirie , "linux-iio@vger.kernel.org" , "linux-kernel@vger.kernel.org" From: Jonathan Cameron Message-ID: Date: Sun, 27 Nov 2016 11:14:47 +0000 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 24/11/16 14:26, Linus Walleij wrote: > On Thu, Nov 24, 2016 at 6:33 AM, Shrirang Bagul > 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 > (...) > >> +#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 >