From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Shevchenko, Andriy" Subject: Re: [PATCH v4 01/13] tty: serial: 8250: Fix whitespace errors Date: Mon, 11 Jan 2016 12:50:18 +0000 Message-ID: <1452516646.26146.22.camel@intel.com> References: <1452275039-18787-1-git-send-email-anton.wuerfel@fau.de> <1452275039-18787-2-git-send-email-anton.wuerfel@fau.de> <1452515294.26146.10.camel@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Cc: "linux-kernel@vger.kernel.org" , "mail@maciej.szmigiero.name" , "desmondl@broadcom.com" , "Zha, Qipeng" , "heikki.krogerus@linux.intel.com" , "long.wanglong@huawei.com" , "soeren.grunewald@desy.de" , "deller@gmx.de" , "jslaby@suse.com" , "phillip.raffeck@fau.de" , "paul.burton@imgtec.com" , "hpeter@gmail.com" , "ralf@linux-mips.org" , "linux-kernel@i4.cs.fau.de" , "mans@mansr.com" , "adam.lee@canonical.com" , "jejb@parisc-linux.org" , "linux-parisc@vger.kernel.org" , "anton.wuerfel@fau.de" Return-path: In-Reply-To: <1452515294.26146.10.camel@linux.intel.com> List-ID: List-Id: linux-parisc.vger.kernel.org T24gTW9uLCAyMDE2LTAxLTExIGF0IDE0OjI4ICswMjAwLCBBbmR5IFNoZXZjaGVua28gd3JvdGU6 DQo+IE9uIEZyaSwgMjAxNi0wMS0wOCBhdCAxODo0MyArMDEwMCwgQW50b24gV3VlcmZlbCB3cm90 ZToNCj4gPiBUaGlzIHBhdGNoIGZpeGVzIHdoaXRlc3BhY2UgZXJyb3JzIHJlcG9ydGVkIGJ5IGNo ZWNrcGF0Y2ggdG8NCj4gPiBpbmNyZWFzZQ0KPiA+IHJlYWRhYmlsaXR5LiBNYWluIGZvY3VzIGlz IG9uIG1pc3Npbmcgc3BhY2VzIGFmdGVyIGNvbW1hcyBpbg0KPiA+IGZ1bmN0aW9uIGhlYWRlcnMg YW5kIG1hY3JvcyAobGlrZSBmb28sYmFyIGVkaXRlZCB0byBmb28sIGJhcikuDQo+ID4gDQo+ID4g U2lnbmVkLW9mZi1ieTogQW50b24gV8O8cmZlbCA8YW50b24ud3VlcmZlbEBmYXUuZGU+DQo+ID4g U2lnbmVkLW9mZi1ieTogUGhpbGxpcCBSYWZmZWNrIDxwaGlsbGlwLnJhZmZlY2tAZmF1LmRlPg0K PiA+IENDOiBsaW51eC1rZXJuZWxAaTQuY3MuZmF1LmRlDQo+IA0KPiBDYzogbG9va3MgbW9yZSBz dGFuZGFyZC4NCj4gDQo+ID4gLS0tIGEvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9hY2Nl bnQuYw0KPiA+ICsrKyBiL2RyaXZlcnMvdHR5L3NlcmlhbC84MjUwLzgyNTBfYWNjZW50LmMNCj4g PiBAQCAtMTAsNyArMTAsNyBAQA0KPiA+IMKgI2luY2x1ZGUgPGxpbnV4L2luaXQuaD4NCj4gPiDC oCNpbmNsdWRlIDxsaW51eC9zZXJpYWxfODI1MC5oPg0KPiA+IMKgDQo+ID4gLSNkZWZpbmUgUE9S VChfYmFzZSxfaXJxKQkJCQlcDQo+ID4gKyNkZWZpbmUgUE9SVChfYmFzZSwgX2lycSkJCQkJXA0K PiA+IMKgCXsJCQkJCQlcDQo+ID4gwqAJCS5pb2Jhc2UJCT0gX2Jhc2UsCQlcDQo+ID4gwqAJCS5p cnEJCT0gX2lycSwJCQkNCj4gPiBcDQoNCjgyNTBfUE9SVCgpDQoNCj4gPiANCj4gDQo+IFdoaWxl IHRob3NlIGFyZSBva2F5LCBJIHdvdWxkIHN1Z2dlc3QgdG8gbWFrZSBhIG1hY3JvIGluDQo+IGRy aXZlcnMv4oCmLzgyNTAuaCBvbmUgZm9yIGFsbCwgc29tZXRoaW5nIGxpa2UgdGhpczoNCj4gDQo+ ICNkZWZpbmUgODI1MF9QT1JUX0ZMQUdTKF9iYXNlLCBfaXJxLCBfZmxhZ3MpDQo+IOKApg0KPiAJ LmZsYWdzwqDCoMKgwqDCoMKgwqDCoMKgwqA9IFVQRl9CT09UX0FVVE9DT05GIHwgKF9mbGFncykg XA0KPiANCj4gI2RlZmluZSA4MjUwX1BPUlQoX2Jhc2UsIF9pcnEpCTgyNTBfUE9SVF9GTEFHUyhf YmFzZSwgX2lycSwgMCkNCj4gDQo+IEFuZCB1c2UgaXQuDQoNCiNkZWZpbmUgODI1MF9GT1VSUE9S VChfYmFzZSwgX2lycSkJODI1MF9QT1JUX0ZMQUdTKF9iYXNlLCBfaXJxLA0KVVBGX0ZPVVJQT1JU KQ0KDQo+ID4gLS0tIGEvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9mb3VycG9ydC5jDQo+ ID4gKysrIGIvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9mb3VycG9ydC5jDQo+ID4gQEAg LTEwLDcgKzEwLDcgQEANCj4gPiDCoCNpbmNsdWRlIDxsaW51eC9pbml0Lmg+DQo+ID4gwqAjaW5j bHVkZSA8bGludXgvc2VyaWFsXzgyNTAuaD4NCj4gPiDCoA0KPiA+IC0jZGVmaW5lIFBPUlQoX2Jh c2UsX2lycSkJCQkJCQkNCj4gPiBcDQo+ID4gKyNkZWZpbmUgUE9SVChfYmFzZSwgX2lycSkJCQkJ CQkNCj4gPiBcDQo+ID4gwqAJewkJCQkJCQkJDQo+ID4gXA0KPiA+IMKgCQkuaW9iYXNlCQk9IF9i YXNlLAkJCQ0KPiA+IAlcDQo+ID4gwqAJCS5pcnEJCT0gX2lycSwJCQkNCj4gPiAJDQo+ID4gCVwN Cg0KODI1MF9GT1VSUE9SVCgpDQoNCi0tIA0KQW5keSBTaGV2Y2hlbmtvIDxhbmRyaXkuc2hldmNo ZW5rb0BpbnRlbC5jb20+DQpJbnRlbCBGaW5sYW5kIE95DQotLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KSW50ZWwgRmlu bGFuZCBPeQpSZWdpc3RlcmVkIEFkZHJlc3M6IFBMIDI4MSwgMDAxODEgSGVsc2lua2kgCkJ1c2lu ZXNzIElkZW50aXR5IENvZGU6IDAzNTc2MDYgLSA0IApEb21pY2lsZWQgaW4gSGVsc2lua2kgCgpU aGlzIGUtbWFpbCBhbmQgYW55IGF0dGFjaG1lbnRzIG1heSBjb250YWluIGNvbmZpZGVudGlhbCBt YXRlcmlhbCBmb3IKdGhlIHNvbGUgdXNlIG9mIHRoZSBpbnRlbmRlZCByZWNpcGllbnQocykuIEFu eSByZXZpZXcgb3IgZGlzdHJpYnV0aW9uCmJ5IG90aGVycyBpcyBzdHJpY3RseSBwcm9oaWJpdGVk LiBJZiB5b3UgYXJlIG5vdCB0aGUgaW50ZW5kZWQKcmVjaXBpZW50LCBwbGVhc2UgY29udGFjdCB0 aGUgc2VuZGVyIGFuZCBkZWxldGUgYWxsIGNvcGllcy4K From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932698AbcAKMu3 (ORCPT ); Mon, 11 Jan 2016 07:50:29 -0500 Received: from mga14.intel.com ([192.55.52.115]:40758 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932303AbcAKMu1 (ORCPT ); Mon, 11 Jan 2016 07:50:27 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,552,1444719600"; d="scan'208";a="878980410" From: "Shevchenko, Andriy" To: "gregkh@linuxfoundation.org" , "anton.wuerfel@fau.de" CC: "linux-kernel@vger.kernel.org" , "mail@maciej.szmigiero.name" , "desmondl@broadcom.com" , "Zha, Qipeng" , "heikki.krogerus@linux.intel.com" , "long.wanglong@huawei.com" , "soeren.grunewald@desy.de" , "deller@gmx.de" , "jslaby@suse.com" , "phillip.raffeck@fau.de" , "paul.burton@imgtec.com" , "hpeter@gmail.com" , "ralf@linux-mips.org" , "linux-kernel@i4.cs.fau.de" , "mans@mansr.com" , "adam.lee@canonical.com" , "jejb@parisc-linux.org" , "linux-parisc@vger.kernel.org" , "peter@hurleysoftware.com" , "matt.redfearn@imgtec.com" , "k.kozlowski@samsung.com" Subject: Re: [PATCH v4 01/13] tty: serial: 8250: Fix whitespace errors Thread-Topic: [PATCH v4 01/13] tty: serial: 8250: Fix whitespace errors Thread-Index: AQHRTG6g23xRPjsY5E6tMRrhwlK61Q== Date: Mon, 11 Jan 2016 12:50:18 +0000 Message-ID: <1452516646.26146.22.camel@intel.com> References: <1452275039-18787-1-git-send-email-anton.wuerfel@fau.de> <1452275039-18787-2-git-send-email-anton.wuerfel@fau.de> <1452515294.26146.10.camel@linux.intel.com> In-Reply-To: <1452515294.26146.10.camel@linux.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.237.72.86] Content-Type: text/plain; charset="utf-8" Content-ID: MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u0BCoXlL031584 On Mon, 2016-01-11 at 14:28 +0200, Andy Shevchenko wrote: > On Fri, 2016-01-08 at 18:43 +0100, Anton Wuerfel wrote: > > This patch fixes whitespace errors reported by checkpatch to > > increase > > readability. Main focus is on missing spaces after commas in > > function headers and macros (like foo,bar edited to foo, bar). > > > > Signed-off-by: Anton Würfel > > Signed-off-by: Phillip Raffeck > > CC: linux-kernel@i4.cs.fau.de > > Cc: looks more standard. > > > --- a/drivers/tty/serial/8250/8250_accent.c > > +++ b/drivers/tty/serial/8250/8250_accent.c > > @@ -10,7 +10,7 @@ > >  #include > >  #include > >   > > -#define PORT(_base,_irq) \ > > +#define PORT(_base, _irq) \ > >   { \ > >   .iobase = _base, \ > >   .irq = _irq, > > \ 8250_PORT() > > > > While those are okay, I would suggest to make a macro in > drivers/…/8250.h one for all, something like this: > > #define 8250_PORT_FLAGS(_base, _irq, _flags) > … > .flags          = UPF_BOOT_AUTOCONF | (_flags) \ > > #define 8250_PORT(_base, _irq) 8250_PORT_FLAGS(_base, _irq, 0) > > And use it. #define 8250_FOURPORT(_base, _irq) 8250_PORT_FLAGS(_base, _irq, UPF_FOURPORT) > > --- a/drivers/tty/serial/8250/8250_fourport.c > > +++ b/drivers/tty/serial/8250/8250_fourport.c > > @@ -10,7 +10,7 @@ > >  #include > >  #include > >   > > -#define PORT(_base,_irq) > > \ > > +#define PORT(_base, _irq) > > \ > >   { > > \ > >   .iobase = _base, > > \ > >   .irq = _irq, > > > > \ 8250_FOURPORT() -- Andy Shevchenko Intel Finland Oy --------------------------------------------------------------------- Intel Finland Oy Registered Address: PL 281, 00181 Helsinki Business Identity Code: 0357606 - 4 Domiciled in Helsinki This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.