Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751668AbaF1HAr (ORCPT ); Sat, 28 Jun 2014 03:00:47 -0400 Received: from mail-ie0-f179.google.com ([209.85.223.179]:64803 "EHLO mail-ie0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751001AbaF1HAp (ORCPT ); Sat, 28 Jun 2014 03:00:45 -0400 Date: Sat, 28 Jun 2014 02:00:41 -0500 From: Michael Welling To: Greg Kroah-Hartman , Guillaume Morin , Jonathan Cameron , Sachin Kamat , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/1] staging: iio: ad9850.c: fix checkpatch.pl error Message-ID: <20140628070041.GA3846@deathray> References: <4d0e6b74dcc10ef5d60c9cab4f2df886782fb3c3.1403919775.git.guillaume@morinfr.org> <20140628020935.GA14333@kroah.com> <20140628023009.GA13273@bender.morinfr.org> <20140628023718.GB3579@kroah.com> <20140628031045.GA13985@bender.morinfr.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140628031045.GA13985@bender.morinfr.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Looking at the driver the define you are fixing isn't even used. So it could be removed. While you are at it you may want to remove the peice of completey redundant code. . . if (ret) goto error_ret; error_ret: . . Wow. On Sat, Jun 28, 2014 at 05:10:46AM +0200, Guillaume Morin wrote: > On 27 Jun 22:37, Greg Kroah-Hartman wrote: > > Put that below the --- line. > > Will do. > > > > > And what checkpatch error did you fix? And are you sure it needs to be > > > > fixed? > > > > > > That's what I changed: > > > > > > $ scripts/checkpatch.pl -f drivers/staging/iio/frequency/ad9850.c > > > ERROR: Macros with complex values should be enclosed in parenthesis > > > > Then why didn't you say that :) > > Well it was not totally clear to me if that was obvious or not. Anyway, > I'll mention it in the future. > > > > > > I assumed that if it was reported as an error, it needed to be fixed... > > > > Use your judgement, checkpatch is a tool, it isn't always correct. > > Right, I guess it's borderline. Should I resend the patch or just drop > it? > > Guillaume. > > -- > Guillaume Morin > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/