From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752936AbcERMAW (ORCPT ); Wed, 18 May 2016 08:00:22 -0400 Received: from smtp113.ord1c.emailsrvr.com ([108.166.43.113]:40167 "EHLO smtp113.ord1c.emailsrvr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752693AbcERMAU (ORCPT ); Wed, 18 May 2016 08:00:20 -0400 X-Auth-ID: abbotti@mev.co.uk X-Sender-Id: abbotti@mev.co.uk Subject: Re: [PATCH 00/14] staging: comedi: daqboard2000: checkpatch clean-ups To: Hartley Sweeten , "devel@driverdev.osuosl.org" References: <1463478773-5916-1-git-send-email-abbotti@mev.co.uk> Cc: Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" From: Ian Abbott Message-ID: <573C5951.10101@mev.co.uk> Date: Wed, 18 May 2016 13:00:17 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 17/05/16 18:46, Hartley Sweeten wrote: > On Tuesday, May 17, 2016 2:53 AM, Ian Abbott wrote: >> This series of patches to the daqboard2000 driver is mostly to fix the >> checkpatch.pl warnings. There is one warning remaining about one of the >> `udelay` calls with a parameter of 10 microseconds, but I decided to >> leave it alone, as converting it to `usleep_range` could increase >> firmware loading time. >> >> Patches 03 and 06 have checkpatch warnings themselves about CamelCase >> issues, but they are not "new" issues, and are resolved by the later >> patches in the series. >> >> 01) staging: comedi: daqboard2000: remove commented out code >> 02) staging: comedi: daqboard2000: use usual block comment style >> 03) staging: comedi: daqboard2000: CHECK: spaces preferred around that >> '*' >> 04) staging: comedi: daqboard2000: add blank line after struct >> declaration >> 05) staging: comedi: daqboard2000: rename serial EEPROM register macros >> 06) staging: comedi: daqboard2000: rename register offset macros >> 07) staging: comedi: daqboard2000: rename acquisition control register >> macros >> 08) staging: comedi: daqboard2000: rename acq status register macros >> 09) staging: comedi: daqboard2000: redo DAC control register macros >> 10) staging: comedi: daqboard2000: redo DAC status macros and fix busy >> 11) staging: comedi: daqboard2000: rename trigger control register >> macros >> 12) staging: comedi: daqboard2000: rename reference DACs register macros >> 13) staging: comedi: daqboard2000: rename CamelCase functions >> 14) staging: comedi: daqboard2000: prefer usleep_range() >> >> drivers/staging/comedi/drivers/daqboard2000.c | 376 +++++++++++++------------- >> 1 file changed, 188 insertions(+), 188 deletions(-) > > Ian, > > I had a couple comments on patches 1, 5, 6, and 14 but they are really just > nitpicks. If you prefer to leave this series as-is: > > Reviewed-by: H Hartley Sweeten Okay, I'll post a new series soon, taking your suggestions into account, apart from your comments on patch 5, which I'll leave until I modify the driver further. I'll edit in your `Reviewed-by` lines if that's okay by you, since the changes are just following your suggestions, and are pretty trivial. The least trivial change is changing patch 14 to change `udelay(10)` to `usleep_range(10, 20)` as you suggested. The other changes are just renaming macros and tagging on a couple of comments. -- -=( Ian Abbott @ MEV Ltd. E-mail: )=- -=( Web: http://www.mev.co.uk/ )=-