From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 24 Apr 2018 11:51:06 +0100 From: Javier Arteaga To: Andy Shevchenko Cc: Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Dan O'Donovan , linux-iio@vger.kernel.org, Linux Kernel Mailing List , Nicola Lunghi Subject: Re: [PATCH v2 2/3] iio: adc128s052: add ACPI _HID AANT1280 Message-ID: <20180424105106.33efoxuhizcxcz5e@localhost> References: <20180423213805.12591-1-javier@emutex.com> <20180423213805.12591-3-javier@emutex.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: List-ID: Hi Andy, On Tue, Apr 24, 2018 at 01:08:31AM +0300, Andy Shevchenko wrote: > > + if (ACPI_COMPANION(&spi->dev)) { > > > + const struct acpi_device_id *ad_id; > > + > > + ad_id = acpi_match_device(spi->dev.driver->acpi_match_table, > > + &spi->dev); > > + if (!ad_id) > > + return -ENODEV; > > + > > + config = ad_id->driver_data; > > There is a new API to get it by one call. > https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git/commit/?id=b283f157611f129f5bbbf7d06f5af860d76797fd That looks much cleaner. Will do for a v3. Thank you!