Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964928AbbEMSia (ORCPT ); Wed, 13 May 2015 14:38:30 -0400 Received: from bombadil.infradead.org ([198.137.202.9]:33071 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754809AbbEMSi0 (ORCPT ); Wed, 13 May 2015 14:38:26 -0400 Date: Wed, 13 May 2015 11:38:22 -0700 From: Darren Hart To: Pali =?iso-8859-1?Q?Roh=E1r?= Cc: Alex Hung , Matthew Garrett , "platform-driver-x86@vger.kernel.org" , linux-kernel@vger.kernel.org, Gabriele Mazzotta Subject: Re: [PATCH v2 0/3] Dell Airplane Mode Switch driver Message-ID: <20150513183822.GA7122@fury.dvhart.com> References: <1416755361-17357-1-git-send-email-pali.rohar@gmail.com> <1430301066-22870-1-git-send-email-pali.rohar@gmail.com> <201505031238.14772@pali> <20150506113119.GH30910@pali> <20150506215744.GA22096@fury.dvhart.com> <20150513114841.GF2920@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20150513114841.GF2920@pali> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2017 Lines: 41 On Wed, May 13, 2015 at 01:48:41PM +0200, Pali Rohár wrote: > Darren, can you review/comment this v2 series? It is OK for 4.2 now? > I thought I was waiting on an Ack from Alex but reviewing the various threads I think he's OK with this. Running the patches through checkpatch reports various issues about tabs (resolved by using proper multi-line comment formatting), line length, and not wrapping strings. I corrected these locally to move things along, but when building: CC [M] drivers/platform/x86/dell-rbtn.o drivers/platform/x86/dell-rbtn.c: In function ‘dell_rbtn_notifier_register’: drivers/platform/x86/dell-rbtn.c:273:2: warning: ignoring return value of ‘driver_for_each_device’, declared with attribute warn_unused_result [-Wunused-result] driver_for_each_device(&rbtn_driver.drv, NULL, &ret, rbtn_inc_count); ^ drivers/platform/x86/dell-rbtn.c:284:3: warning: ignoring return value of ‘driver_for_each_device’, declared with attribute warn_unused_result [-Wunused-result] driver_for_each_device(&rbtn_driver.drv, NULL, (void *)false, ^ drivers/platform/x86/dell-rbtn.c: In function ‘dell_rbtn_notifier_unregister’: drivers/platform/x86/dell-rbtn.c:300:3: warning: ignoring return value of ‘driver_for_each_device’, declared with attribute warn_unused_result [-Wunused-result] driver_for_each_device(&rbtn_driver.drv, NULL, (void *)true, We can't introduce new warnings to the build. I've pushed my fixes to the series to my dell-rbtn branch, please start from there and resolve the above, and resubmit. We can still make 4.2. But please make use of checkpatch and watch for warnings like the above in your builds tests so we can avoid unecessary churn. Thanks, -- Darren Hart Intel Open Source Technology Center -- 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/