From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755134AbcAMRHk (ORCPT ); Wed, 13 Jan 2016 12:07:40 -0500 Received: from mga14.intel.com ([192.55.52.115]:29886 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755024AbcAMRHb (ORCPT ); Wed, 13 Jan 2016 12:07:31 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,290,1449561600"; d="scan'208";a="880653291" Message-ID: <1452704613.26146.70.camel@linux.intel.com> Subject: Re: [PATCH v5 04/15] tty: serial: 8250: Fix multiline comment style From: Andy Shevchenko To: Anton Wuerfel , Greg Kroah-Hartman Cc: Jiri Slaby , "James E.J. Bottomley" , Helge Deller , Peter Hurley , Heikki Krogerus , Qipeng Zha , Desmond Liu , Wang Long , Matt Redfearn , Paul Burton , Ralf Baechle , Krzysztof Kozlowski , Peter Hung , Soeren Grunewald , Adam Lee , "Maciej S. Szmigiero" , Mans Rullgard , linux-kernel@vger.kernel.org, linux-parisc@vger.kernel.org, linux-kernel@i4.cs.fau.de, Phillip Raffeck Date: Wed, 13 Jan 2016 19:03:33 +0200 In-Reply-To: <1452703171-5632-5-git-send-email-anton.wuerfel@fau.de> References: <1452703171-5632-1-git-send-email-anton.wuerfel@fau.de> <1452703171-5632-5-git-send-email-anton.wuerfel@fau.de> Organization: Intel Finland Oy Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.3-1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2016-01-13 at 17:39 +0100, Anton Wuerfel wrote: > Checkpatch outputs some warnings about incorrect comment style, > which is fixed by this patch. > > --- a/drivers/tty/serial/8250/8250_ingenic.c > +++ b/drivers/tty/serial/8250/8250_ingenic.c > @@ -152,14 +152,18 @@ static void ingenic_uart_serial_out(struct > uart_port *p, int offset, int value) >   break; >   >   case UART_IER: > - /* Enable receive timeout interrupt with the > -  * receive line status interrupt */ > + /* > +  * Enable receive timeout interrupt with the > +  * receive line status interrupt I'm pretty sure that at least one word could fit previous line. Can you re-check your patches with sane editor setting for line breaking (like 76 characters)? > +  */ >   value |= (value & 0x4) << 2; >   break; >   >   case UART_MCR: > - /* If we have enabled modem status IRQs we should > enable modem > -  * mode. */ > + /* > +  * If we have enabled modem status IRQs we should > enable modem > +  * mode. To check. > +  */ > > @@ -367,8 +367,10 @@ static const struct pnp_device_id > pnp_dev_table[] = { >   { "PNPCXXX", UNKNOWN_DEV }, >   /* More unknown PnP modems */ >   { "PNPDXXX", UNKNOWN_DEV }, > - /* Winbond CIR port, should not be probed. We should keep > track > -    of it to prevent the legacy serial driver from probing it > */ > + /* > +  * Winbond CIR port, should not be probed. We should keep > track > +  * of it to prevent the legacy serial driver from probing it I would suggest to add dot at the end of sentences. Here and in the rest of the places. > --- a/drivers/tty/serial/8250/8250_port.c > +++ b/drivers/tty/serial/8250/8250_port.c > @@ -250,9 +250,11 @@ static const struct serial8250_config > uart_config[] = { >   .fcr = UART_FCR_ENABLE_FIFO | > UART_FCR_R_TRIG_10, >   .flags = UART_CAP_FIFO | > UART_CAP_AFE, >   }, > -/* tx_loadsz is set to 63-bytes instead of 64-bytes to implement > -workaround of errata A-008006 which states that tx_loadsz should  be > -configured less than Maximum supported fifo bytes */ > + /* > +  * tx_loadsz is set to 63-bytes instead of 64-bytes to > implement > +  * workaround of errata A-008006 which states that tx_loadsz > should be > +  * configured less than Maximum supported fifo bytes Ditto. > --- a/drivers/tty/serial/8250/serial_cs.c > +++ b/drivers/tty/serial/8250/serial_cs.c > @@ -441,16 +441,20 @@ static int simple_config(struct pcmcia_device > *link) >   struct serial_info *info = link->priv; >   int i = -ENODEV, try; >   > - /* First pass: look for a config entry that looks normal. > -  * Two tries: without IO aliases, then with aliases */ > + /* > +  * First pass: look for a config entry that looks normal. > +  * Two tries: without IO aliases, then with aliases Ditto. > @@ -480,8 +484,10 @@ static int multi_config_check(struct > pcmcia_device *p_dev, void *priv_data) >   if (p_dev->resource[1]->end) >   return -EINVAL; >   > - /* The quad port cards have bad CIS's, so just look for a > -    window larger than 8 ports and assume it will be right */ > + /* > +  * The quad port cards have bad CIS's, so just look for a > +  * window larger than 8 ports and assume it will be right Ditto. > - /* Another check for dual-serial cards: look for either > serial or > -    multifunction cards that ask for appropriate IO port > ranges */ > + /* > +  * Another check for dual-serial cards: look for either > serial or > +  * multifunction cards that ask for appropriate IO port > ranges Ditto. -- Andy Shevchenko Intel Finland Oy