Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932545AbbFVHOe (ORCPT ); Mon, 22 Jun 2015 03:14:34 -0400 Received: from mail-db3on0147.outbound.protection.outlook.com ([157.55.234.147]:49029 "EHLO emea01-db3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753790AbbFVHO1 convert rfc822-to-8bit (ORCPT ); Mon, 22 Jun 2015 03:14:27 -0400 From: "Abdul, Hussain (H.)" To: Sudip Mukherjee CC: "gregkh@linuxfoundation.org" , "devel@driverdev.osuosl.org" , "abbotti@mev.co.uk" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 3/6] Staging: comedi: daqboard2000: Simplify a trivial if-return sequence Thread-Topic: [PATCH 3/6] Staging: comedi: daqboard2000: Simplify a trivial if-return sequence Thread-Index: AQHQqlUwmEuxtO7bp0ixjgmBL4Wo7A== Date: Mon, 22 Jun 2015 07:14:24 +0000 Message-ID: References: <1434693514-10621-1-git-send-email-habdul@visteon.com> <1434693514-10621-3-git-send-email-habdul@visteon.com> <20150619140812.GB32112@sudip-PC> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: gmail.com; dkim=none (message not signed) header.d=none; x-originating-ip: [74.112.164.117] x-microsoft-exchange-diagnostics: 1;DB5PR06MB1111;5:VBdpYWZk2v5Avcwzdj/uC+A1jjWyF4+QcesKrJVl2ohBAttNBcaGuMcVHiEnNdr5LFn4qQ+Rkd7ZHT0FaSVH2VxD+RyH9CXl106YmTvuNuc1XndMnJBOd0qe4WSOGiaEGf6Uqap0i8svq50WtwlorQ==;24:TTB8jiFXaEbIoyImd0vf/6kEyqRjIeW8F2NoajZe54tpKzS7GsiB1ZLZg1hXEmZeKRT7e64aQ0n5X1qiX8qPCcTk6EgCV5A9lPZh3sBU6OU=;20:IMwO7rJNHs7sk3Hlj0gmBhn+OfQ81diQ7qEc66+yTNbeZbGNUbWzYzWJdVj6m7mcaI4x2RKL9MnEsyZn9OY2VA== x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB5PR06MB1111; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(601004)(5005006)(3002001);SRVR:DB5PR06MB1111;BCL:0;PCL:0;RULEID:;SRVR:DB5PR06MB1111; x-forefront-prvs: 06157D541C x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(6009001)(51704005)(24454002)(377454003)(164054003)(40100003)(77156002)(62966003)(33656002)(76576001)(19580395003)(19580405001)(2656002)(5003600100002)(102836002)(87936001)(74316001)(77096005)(122556002)(86362001)(2900100001)(5002640100001)(92566002)(110136002)(5001960100002)(5001920100001)(50986999)(54356999)(66066001)(106116001)(46102003)(76176999)(189998001);DIR:OUT;SFP:1102;SCL:1;SRVR:DB5PR06MB1111;H:DB5PR06MB1112.eurprd06.prod.outlook.com;FPR:;SPF:None;MLV:sfv;LANG:en; Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-OriginatorOrg: visteon.com X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Jun 2015 07:14:24.8107 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 7a147aaf-01ec-498c-80a1-e34a8c63c548 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB5PR06MB1111 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 752 Lines: 26 On Friday 19 June 2015 07:38 PM, Sudip Mukherjee wrote: > On Fri, Jun 19, 2015 at 06:00:06AM +0000, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch simplify a trivial if-return sequence. Possibly combine with >> a preceding function call. >> >> Signed-off-by: Abdul Hussain >> --- > >> - return 0; >> + return subdev_8255_init(dev, s, daqboard2000_8255_cb, dioP2ExpansionIO8Bit); > this is introducing new checkpatch warning about line over 80 characters > > regards > sudip > Sudip, I addressed the above issue and resent the patch set v2. Thanks, Abdul -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/