From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754758AbdBNRQj (ORCPT ); Tue, 14 Feb 2017 12:16:39 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:56638 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754408AbdBNRQb (ORCPT ); Tue, 14 Feb 2017 12:16:31 -0500 Date: Tue, 14 Feb 2017 09:16:31 -0800 From: Greg Kroah-Hartman To: Maksymilian Piechota Cc: devel@driverdev.osuosl.org, maksymilian.piechota@gmail.com, linux-kernel@vger.kernel.org Subject: Re: [PATCHv6 1/3] staging: wlan-ng: move logical continuations at the end of line Message-ID: <20170214171631.GA1518@kroah.com> References: <8b7698a301c1e755ab8a0f1efe8414e8f792948e.1486993631.git.maksymilianpiechota@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8b7698a301c1e755ab8a0f1efe8414e8f792948e.1486993631.git.maksymilianpiechota@gmail.com> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 13, 2017 at 09:25:49AM -0500, Maksymilian Piechota wrote: > move logical continuations at the end of line > > Signed-off-by: Maksymilian Piechota > --- > drivers/staging/wlan-ng/prism2mgmt.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/wlan-ng/prism2mgmt.c > index 16fb2d3..64a9ebc 100644 > --- a/drivers/staging/wlan-ng/prism2mgmt.c > +++ b/drivers/staging/wlan-ng/prism2mgmt.c > @@ -1308,9 +1308,8 @@ int prism2mgmt_wlansniff(struct wlandevice *wlandev, void *msgp) > hw->sniffhdr = 0; > wlandev->netdev->type = ARPHRD_IEEE80211_PRISM; > } else > - if ((msg->wlanheader.status == > - P80211ENUM_msgitem_status_data_ok) > - && (msg->wlanheader.data == P80211ENUM_truth_true)) { > + if ((msg->wlanheader.status == P80211ENUM_msgitem_status_data_ok) && > + (msg->wlanheader.data == P80211ENUM_truth_true)) { > hw->sniffhdr = 1; > wlandev->netdev->type = ARPHRD_IEEE80211_PRISM; > } else { This patch has checkpatch.pl errors. Please always use checkpatch so you don't get emails from grumpy maintainers telling you to fix the checkpatch errors :) Please fix up and resend the whole series. thanks, greg k-h