From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752150AbbIQEzQ (ORCPT ); Thu, 17 Sep 2015 00:55:16 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:42793 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751653AbbIQEzO (ORCPT ); Thu, 17 Sep 2015 00:55:14 -0400 Date: Wed, 16 Sep 2015 21:55:13 -0700 From: Greg Kroah-Hartman To: =?iso-8859-1?Q?Rapha=EBl?= Beamonte Cc: Cristina Opriceana , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, Dan Carpenter Subject: Re: [PATCHv2 02/16] staging: rtl8192u: r8192U_core: add line breaks to keep lines under 80 characters Message-ID: <20150917045512.GA29493@kroah.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 11, 2015 at 03:29:10AM -0400, Raphaël Beamonte wrote: > Add line breaks in multiple lines to keep them under 80 characters, > as to follow the kernel code style. > > Signed-off-by: Raphaël Beamonte > --- > drivers/staging/rtl8192u/r8192U_core.c | 626 ++++++++++++++++++++++----------- > 1 file changed, 421 insertions(+), 205 deletions(-) > > diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r8192U_core.c > index 5e9d0ac..37c17eb 100644 > --- a/drivers/staging/rtl8192u/r8192U_core.c > +++ b/drivers/staging/rtl8192u/r8192U_core.c > @@ -144,25 +144,31 @@ struct CHANNEL_LIST { > > static struct CHANNEL_LIST ChannelPlan[] = { > /* FCC */ > - {{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 36, 40, 44, 48, 52, 56, 60, 64, 149, 153, 157, 161, 165}, 24}, > + {{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 36, 40, 44, 48, 52, 56, 60, 64, > + 149, 153, 157, 161, 165}, 24}, But now these look worse than before. I say just leave these long variables as-is. thanks, greg k-h