From mboxrd@z Thu Jan 1 00:00:00 1970 From: Asaf Vertz Subject: Re: [PATCH] input: edt-ft5x06: fixed a macro coding style issue Date: Tue, 09 Dec 2014 09:51:54 +0200 Message-ID: <5486AA1A.4060802@tandemg.com> References: <20141209063456.GA15121@ubuntu> <1418107460.15957.9.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1418107460.15957.9.camel@perches.com> Sender: linux-kernel-owner@vger.kernel.org To: Joe Perches Cc: dmitry.torokhov@gmail.com, LW@KARO-electronics.de, B38611@freescale.com, maksqwe1@ukr.net, jg1.han@samsung.com, linux@rasmusvillemoes.dk, fabf@skynet.be, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-input@vger.kernel.org On 12/09/2014 08:44 AM, Joe Perches wrote: > On Tue, 2014-12-09 at 08:34 +0200, Asaf Vertz wrote: >> Fixed a coding style error, macros with complex values should be >> enclosed in parentheses. >> >> -#define EDT_ATTR_CHECKSET(name, reg) \ >> +#define EDT_ATTR_CHECKSET(name, reg) { \ >> if (pdata->name >= edt_ft5x06_attr_##name.limit_low && \ >> pdata->name <= edt_ft5x06_attr_##name.limit_high) \ >> - edt_ft5x06_register_write(tsdata, reg, pdata->name) >> + edt_ft5x06_register_write(tsdata, reg, pdata->name); \ >> +} > > Any time you see a naked if (foo) bar in a macro > prefer to use a do while (0) like; > > #define EDT_ATTR_CHECKSET(name, reg) \ > do { \ > if (pdata->name >= edt_ft5x06_attr_##name.limit_low && \ > pdata->name <= edt_ft5x06_attr_##name.limit_high) \ > edt_ft5x06_register_write(tsdata, reg, pdata->name); \ > while (0) > thanks for the input, I will update and resend it