Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751857AbdFIJfk (ORCPT ); Fri, 9 Jun 2017 05:35:40 -0400 Received: from mail-qt0-f195.google.com ([209.85.216.195]:36838 "EHLO mail-qt0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751755AbdFIJfh (ORCPT ); Fri, 9 Jun 2017 05:35:37 -0400 MIME-Version: 1.0 In-Reply-To: <20170609090314.13991-9-gregkh@linuxfoundation.org> References: <20170609090314.13991-1-gregkh@linuxfoundation.org> <20170609090314.13991-9-gregkh@linuxfoundation.org> From: Andy Shevchenko Date: Fri, 9 Jun 2017 12:35:35 +0300 Message-ID: Subject: Re: [PATCH 09/11] platform: thinkpad_acpi: convert to use DRIVER_ATTR_RO/RW To: Greg Kroah-Hartman Cc: "linux-kernel@vger.kernel.org" , Henrique de Moraes Holschuh , Darren Hart , Andy Shevchenko , ibm-acpi-devel@lists.sourceforge.net, Platform Driver Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 9185 Lines: 229 On Fri, Jun 9, 2017 at 12:03 PM, Greg Kroah-Hartman wrote: > We are trying to get rid of DRIVER_ATTR(), and the thinkpad_acpi > driver's attributes can be trivially changed to use DRIVER_ATTR_RO() and > DRIVER_ATTR_RW(). Which tree is it supposed to go through? We might need an immutable tag / branch. P.S. Change is good to me, though let's give a chance to Darren and Henrique to comment. > Cc: Henrique de Moraes Holschuh > Cc: Darren Hart > Cc: Andy Shevchenko > Cc: > Cc: > Signed-off-by: Greg Kroah-Hartman > --- > drivers/platform/x86/thinkpad_acpi.c | 91 +++++++++++------------------------- > 1 file changed, 28 insertions(+), 63 deletions(-) > > diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c > index 7b6cb0c69b02..f6861b551178 100644 > --- a/drivers/platform/x86/thinkpad_acpi.c > +++ b/drivers/platform/x86/thinkpad_acpi.c > @@ -1438,25 +1438,20 @@ static int tpacpi_rfk_procfs_write(const enum tpacpi_rfk_id id, char *buf) > */ > > /* interface_version --------------------------------------------------- */ > -static ssize_t tpacpi_driver_interface_version_show( > - struct device_driver *drv, > - char *buf) > +static ssize_t interface_version_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "0x%08x\n", TPACPI_SYSFS_VERSION); > } > - > -static DRIVER_ATTR(interface_version, S_IRUGO, > - tpacpi_driver_interface_version_show, NULL); > +static DRIVER_ATTR_RO(interface_version); > > /* debug_level --------------------------------------------------------- */ > -static ssize_t tpacpi_driver_debug_show(struct device_driver *drv, > - char *buf) > +static ssize_t debug_level_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "0x%04x\n", dbg_level); > } > > -static ssize_t tpacpi_driver_debug_store(struct device_driver *drv, > - const char *buf, size_t count) > +static ssize_t debug_level_store(struct device_driver *drv, const char *buf, > + size_t count) > { > unsigned long t; > > @@ -1467,34 +1462,28 @@ static ssize_t tpacpi_driver_debug_store(struct device_driver *drv, > > return count; > } > - > -static DRIVER_ATTR(debug_level, S_IWUSR | S_IRUGO, > - tpacpi_driver_debug_show, tpacpi_driver_debug_store); > +static DRIVER_ATTR_RW(debug_level); > > /* version ------------------------------------------------------------- */ > -static ssize_t tpacpi_driver_version_show(struct device_driver *drv, > - char *buf) > +static ssize_t version_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "%s v%s\n", > TPACPI_DESC, TPACPI_VERSION); > } > - > -static DRIVER_ATTR(version, S_IRUGO, > - tpacpi_driver_version_show, NULL); > +static DRIVER_ATTR_RO(version); > > /* --------------------------------------------------------------------- */ > > #ifdef CONFIG_THINKPAD_ACPI_DEBUGFACILITIES > > /* wlsw_emulstate ------------------------------------------------------ */ > -static ssize_t tpacpi_driver_wlsw_emulstate_show(struct device_driver *drv, > - char *buf) > +static ssize_t wlsw_emulstate_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "%d\n", !!tpacpi_wlsw_emulstate); > } > > -static ssize_t tpacpi_driver_wlsw_emulstate_store(struct device_driver *drv, > - const char *buf, size_t count) > +static ssize_t wlsw_emulstate_store(struct device_driver *drv, const char *buf, > + size_t count) > { > unsigned long t; > > @@ -1508,22 +1497,16 @@ static ssize_t tpacpi_driver_wlsw_emulstate_store(struct device_driver *drv, > > return count; > } > - > -static DRIVER_ATTR(wlsw_emulstate, S_IWUSR | S_IRUGO, > - tpacpi_driver_wlsw_emulstate_show, > - tpacpi_driver_wlsw_emulstate_store); > +static DRIVER_ATTR_RW(wlsw_emulstate); > > /* bluetooth_emulstate ------------------------------------------------- */ > -static ssize_t tpacpi_driver_bluetooth_emulstate_show( > - struct device_driver *drv, > - char *buf) > +static ssize_t bluetooth_emulstate_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "%d\n", !!tpacpi_bluetooth_emulstate); > } > > -static ssize_t tpacpi_driver_bluetooth_emulstate_store( > - struct device_driver *drv, > - const char *buf, size_t count) > +static ssize_t bluetooth_emulstate_store(struct device_driver *drv, > + const char *buf, size_t count) > { > unsigned long t; > > @@ -1534,22 +1517,16 @@ static ssize_t tpacpi_driver_bluetooth_emulstate_store( > > return count; > } > - > -static DRIVER_ATTR(bluetooth_emulstate, S_IWUSR | S_IRUGO, > - tpacpi_driver_bluetooth_emulstate_show, > - tpacpi_driver_bluetooth_emulstate_store); > +static DRIVER_ATTR_RW(bluetooth_emulstate); > > /* wwan_emulstate ------------------------------------------------- */ > -static ssize_t tpacpi_driver_wwan_emulstate_show( > - struct device_driver *drv, > - char *buf) > +static ssize_t wwan_emulstate_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "%d\n", !!tpacpi_wwan_emulstate); > } > > -static ssize_t tpacpi_driver_wwan_emulstate_store( > - struct device_driver *drv, > - const char *buf, size_t count) > +static ssize_t wwan_emulstate_store(struct device_driver *drv, const char *buf, > + size_t count) > { > unsigned long t; > > @@ -1560,22 +1537,16 @@ static ssize_t tpacpi_driver_wwan_emulstate_store( > > return count; > } > - > -static DRIVER_ATTR(wwan_emulstate, S_IWUSR | S_IRUGO, > - tpacpi_driver_wwan_emulstate_show, > - tpacpi_driver_wwan_emulstate_store); > +static DRIVER_ATTR_RW(wwan_emulstate); > > /* uwb_emulstate ------------------------------------------------- */ > -static ssize_t tpacpi_driver_uwb_emulstate_show( > - struct device_driver *drv, > - char *buf) > +static ssize_t uwb_emulstate_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "%d\n", !!tpacpi_uwb_emulstate); > } > > -static ssize_t tpacpi_driver_uwb_emulstate_store( > - struct device_driver *drv, > - const char *buf, size_t count) > +static ssize_t uwb_emulstate_store(struct device_driver *drv, const char *buf, > + size_t count) > { > unsigned long t; > > @@ -1586,10 +1557,7 @@ static ssize_t tpacpi_driver_uwb_emulstate_store( > > return count; > } > - > -static DRIVER_ATTR(uwb_emulstate, S_IWUSR | S_IRUGO, > - tpacpi_driver_uwb_emulstate_show, > - tpacpi_driver_uwb_emulstate_store); > +static DRIVER_ATTR_RW(uwb_emulstate); > #endif > > /* --------------------------------------------------------------------- */ > @@ -8606,14 +8574,13 @@ static ssize_t fan_fan2_input_show(struct device *dev, > static DEVICE_ATTR(fan2_input, S_IRUGO, fan_fan2_input_show, NULL); > > /* sysfs fan fan_watchdog (hwmon driver) ------------------------------- */ > -static ssize_t fan_fan_watchdog_show(struct device_driver *drv, > - char *buf) > +static ssize_t fan_watchdog_show(struct device_driver *drv, char *buf) > { > return snprintf(buf, PAGE_SIZE, "%u\n", fan_watchdog_maxinterval); > } > > -static ssize_t fan_fan_watchdog_store(struct device_driver *drv, > - const char *buf, size_t count) > +static ssize_t fan_watchdog_store(struct device_driver *drv, const char *buf, > + size_t count) > { > unsigned long t; > > @@ -8630,9 +8597,7 @@ static ssize_t fan_fan_watchdog_store(struct device_driver *drv, > > return count; > } > - > -static DRIVER_ATTR(fan_watchdog, S_IWUSR | S_IRUGO, > - fan_fan_watchdog_show, fan_fan_watchdog_store); > +static DRIVER_ATTR_RW(fan_watchdog); > > /* --------------------------------------------------------------------- */ > static struct attribute *fan_attributes[] = { > -- > 2.13.1 > -- With Best Regards, Andy Shevchenko