From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.6 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9F6E4C43461 for ; Thu, 17 Sep 2020 18:26:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3EA88221EE for ; Thu, 17 Sep 2020 18:26:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1600367166; bh=q66+/z4O/ZxaWK3Of1p/Fwdf1GZ0+VZYTG0Hx7pDRLk=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=ruH8Mjv/Pgx1nf7p7/Ew1x92JMYS6W0wRwvtigRpFthd5HeVwtfABx5Xd9n7WeJii U0zMOoBJBb+4e22+T2jTv6kd8u3G2gqiv0mVhcwOvzQmNHVKqh4eFxlaiS5cA4TQzJ uOTEEmE5pa35YdzRconj5MAoHRHstsvhJVMlKpX8= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726479AbgIQSZs (ORCPT ); Thu, 17 Sep 2020 14:25:48 -0400 Received: from mail.kernel.org ([198.145.29.99]:60962 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726430AbgIQSZl (ORCPT ); Thu, 17 Sep 2020 14:25:41 -0400 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 349DF221EE; Thu, 17 Sep 2020 18:17:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1600366677; bh=q66+/z4O/ZxaWK3Of1p/Fwdf1GZ0+VZYTG0Hx7pDRLk=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=tmsXW7aZgRvH3uTIOHOOMcI3pGwDf7BlelO+jbxmI/Y3gY/Ytbz8StwYhVRTSQ3Sn hMeJ5ie5puNAGf+6wG/Gw74TsfyXBCtri50ON5PLIBnKGSjV8Z2AJic6rikuicpmZs HMprOV+CxhxzOOtpsebOvRcxNPF7pDj0aomHs1Q4= Date: Thu, 17 Sep 2020 19:17:52 +0100 From: Jonathan Cameron To: Andy Shevchenko Cc: linux-iio , Jonathan Cameron , Matt Ranostay Subject: Re: [PATCH 00/38] iio: remaining easy of_match_ptr removal and related. Message-ID: <20200917191752.4159a22e@archlinux> In-Reply-To: References: <20200910173242.621168-1-jic23@kernel.org> X-Mailer: Claws Mail 3.17.6 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Mon, 14 Sep 2020 11:04:43 +0300 Andy Shevchenko wrote: > On Thu, Sep 10, 2020 at 8:35 PM Jonathan Cameron wrote: > > > > From: Jonathan Cameron > > > > This bumper set removes all the remaining cases where of_match_ptr > > is mostly serving to prevent use of the device via ACPI PRP0001 and > > complicating the code. The main aim of this set is to reduce the chances > > of it being used in future drivers. > > It also includes some related refactorings, use of generic fw accessors, > > dropping of CONFIG_OF and lots of header changes inline with the above. > > > > For examples of how to use PRP0001 see https://github.com/westeri/meta-acpi > > which is an excellent resource from which to copy snippets of DSDT tables. > > > > It may well not be worth changing the remaining 10 IIO drivers and I > > certainly don't plan to do so anytime soon. > > > > I briefly looked at the series and didn't see anything wrong with it > (esp. taking into consideration that you are already familiar with the > topic). > Reviewed-by: Andy Shevchenko Thanks Andy + others that reviewed. I've fixed the typo's in descriptions Matt highlighted and applied the lot. If anyone want to add tags, it will be a few days before I push this out as a non rebasing tree. Thanks, Jonathan > > > Jonathan Cameron (38): > > iio:potentiometer:ad5272: Drop of_match_ptr and CONFIG_OF protections. > > iio:potentiometer:ds1803: Drop of_match_ptr and CONFIG_OF protections > > iio:potentiometer:max5432: Drop of_match_ptr and use generic fw > > accessors > > iio:potentiometer:max5481: Drop of_match_ptr and CONFIG_OF > > protections. > > iio:potentiometer:max5481: Drop invalid ACPI binding. > > iio:potentiometer:mcp4018: Drop of_match_ptr and CONFIG_OF > > protections. > > iio:potentiometer:mcp4131: Drop of_match_ptr and use generic fw > > interfaces. > > iio:potentiometer:mcp4531: Drop of_match_ptr and CONFIG_OF > > protections. > > iio:dac:ad5446: Drop of_match_ptr and CONFIG_OF protections > > iio:dac:ad5592r: Drop of_match_ptr and ACPI_PTR protections. > > iio:dac:ad5593r: Drop of_match_ptr and ACPI_PTR protections. > > iio:dac:ad7303: Drop of_match_ptr protection > > iio:dac:mcp4725: drop of_match_ptr and use generic fw properties > > iio:dac:ti-dac082s085: Drop of_match_ptr and CONFIG_OF protections > > iio:dac:ti-dac5571: Drop of_match_ptr and CONFIG_OF protections > > iio:potentiostat:lmp91000: Drop of_match_ptr and use generic fw > > accessors > > iio:pressure:icp10100: Drop of_match_ptr and CONFIG_OF protections > > iio:pressure:ms5611: Drop of_match_ptr and CONFIG_OF protections > > iio:pressure:ms5637: Drop of_match_ptr protection > > iio:pressure:zpa2326: Drop of_match_ptr protection > > iio:temperature:tsys01: Drop of_match_ptr protection > > iio:temperature:tmp007: Drop of_match_ptr protection > > iio:resolver:ad2s1200: Drop of_match_ptr protection > > iio:chemical:ams-iaq-core: Drop of_match_ptr protection > > iio:chemical:atlas-sensor: Drop of_match_ptr and use generic fw > > accessors > > iio:chemical:sgp30: Use local variable dev to simplify code > > iio:chemical:sgp30: Drop of_match_ptr and use generic fw accessors > > iio:chemical:vz89x: Introduce local struct device pointer. > > iio:chemical:vz89x: Drop of_match_ptr protection and use generic fw > > accessors > > iio:humidity:hdc100x: Drop of_match_ptr protection. > > iio:proximity:as3935: Use local struct device pointer to simplify > > code. > > iio:proximity:as3935: Drop of_match_ptr and use generic fw accessors > > iio:proximity:pulsedlight: Drop of_match_ptr protection > > iio:magn:ak8975: Drop of_match_ptr and ACPI_PTR protections. > > iio:magn:ak8974: Drop of_match_ptr protection > > iio:humidity:htu21: Drop of_match_ptr protection > > iio:humidity:si7020: Drop of_match_ptr protection > > iio:health:max30102: Drop of_match_ptr and use generic fw accessors > > > > drivers/iio/chemical/ams-iaq-core.c | 3 +- > > drivers/iio/chemical/atlas-sensor.c | 10 ++--- > > drivers/iio/chemical/sgp30.c | 28 ++++++------ > > drivers/iio/chemical/vz89x.c | 18 ++++---- > > drivers/iio/dac/ad5446.c | 5 +-- > > drivers/iio/dac/ad5592r.c | 7 ++- > > drivers/iio/dac/ad5593r.c | 7 ++- > > drivers/iio/dac/ad7303.c | 3 +- > > drivers/iio/dac/mcp4725.c | 29 +++--------- > > drivers/iio/dac/ti-dac082s085.c | 5 +-- > > drivers/iio/dac/ti-dac5571.c | 7 +-- > > drivers/iio/health/max30102.c | 11 +++-- > > drivers/iio/humidity/hdc100x.c | 3 +- > > drivers/iio/humidity/htu21.c | 3 +- > > drivers/iio/humidity/si7020.c | 3 +- > > drivers/iio/magnetometer/ak8974.c | 3 +- > > drivers/iio/magnetometer/ak8975.c | 8 ++-- > > drivers/iio/potentiometer/ad5272.c | 5 +-- > > drivers/iio/potentiometer/ds1803.c | 6 +-- > > drivers/iio/potentiometer/max5432.c | 8 ++-- > > drivers/iio/potentiometer/max5481.c | 23 ++-------- > > drivers/iio/potentiometer/mcp4018.c | 12 ++--- > > drivers/iio/potentiometer/mcp4131.c | 8 ++-- > > drivers/iio/potentiometer/mcp4531.c | 11 ++--- > > drivers/iio/potentiostat/lmp91000.c | 11 +++-- > > drivers/iio/pressure/icp10100.c | 3 +- > > drivers/iio/pressure/ms5611_i2c.c | 6 +-- > > drivers/iio/pressure/ms5611_spi.c | 6 +-- > > drivers/iio/pressure/ms5637.c | 3 +- > > drivers/iio/pressure/zpa2326_i2c.c | 6 +-- > > drivers/iio/pressure/zpa2326_spi.c | 6 +-- > > drivers/iio/proximity/as3935.c | 44 +++++++++---------- > > .../iio/proximity/pulsedlight-lidar-lite-v2.c | 3 +- > > drivers/iio/resolver/ad2s1200.c | 3 +- > > drivers/iio/temperature/tmp007.c | 4 +- > > drivers/iio/temperature/tsys01.c | 3 +- > > 36 files changed, 134 insertions(+), 190 deletions(-) > > > > -- > > 2.28.0 > > > >