From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757427AbcGKA4u (ORCPT ); Sun, 10 Jul 2016 20:56:50 -0400 Received: from bh-25.webhostbox.net ([208.91.199.152]:52541 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751406AbcGKA4q (ORCPT ); Sun, 10 Jul 2016 20:56:46 -0400 Subject: Re: [PATCH 0/5] hwmon: New hwmon registration API To: Jonathan Cameron , Jean Delvare References: <1466911590-26296-1-git-send-email-linux@roeck-us.net> Cc: Zhang Rui , Eduardo Valentin , linux-pm@vger.kernel.org, linux-iio@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org From: Guenter Roeck Message-ID: <5782EEBB.1040209@roeck-us.net> Date: Sun, 10 Jul 2016 17:56:27 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Authenticated_sender: linux@roeck-us.net X-OutGoing-Spam-Status: No, score=-1.0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: authenticated_id: linux@roeck-us.net X-Authenticated-Sender: bh-25.webhostbox.net: linux@roeck-us.net X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jonathan, On 07/10/2016 09:00 AM, Jonathan Cameron wrote: > On 26/06/16 04:26, Guenter Roeck wrote: >> Up to now, each hwmon driver has to implement its own sysfs attributes. >> This requires a lot of template code, and distracts from the driver's >> core function to read and write chip registers. >> >> To be able to reduce driver complexity, move sensor attribute handling >> and thermal zone registration into the hwmon core. By using the new API, >> driver size is typically reduced by 20-50% depending on driver complexity >> and the number of sysfs attributes supported. >> >> The first patch of the series introduces the API as well as support >> for temperature sensors. Subsequent patches introduce support for >> voltage, current, power, energy, humidity, and fan speed sensors. >> >> The series was tested by converting several drivers (lm75, lm90, tmp102, >> tmp421, ltc4245) to the new API. Testing was done with with real chips >> as well as with the hwmon driver module test code available at >> https://github.com/groeck/module-tests. > Some cool stuff hiding in there :) > > I've only reviewed 1 and 7 and the intermediate ones are really either > correct or they aren't and they look fine to me. > Thanks a lot for your feedback - it is reassuring that it looks sane to you. > > So what's next? (beyond presumably a lot of driver conversions). > Next steps will be to address your comments, re-test, submit a new series, add to -next after v4.8-rc1 is out, submit the conversions I have done and add to -next. Then, obviously, only accept new drivers using the new API. Not sure if there is anything else we can or want to do do at this time. Do you have anything in mind ? Thanks, Guenter