| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1452762780.2521.20.camel@linux.intel.com> Date: Thu, 14 Jan 2016 11:13:00 +0200 From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com> To: Anton Wuerfel <anton.wuerfel@....de>, Greg Kroah-Hartman <gregkh@...uxfoundation.org> Cc: Jiri Slaby <jslaby@...e.com>, "James E.J. Bottomley" <jejb@...isc-linux.org>, Helge Deller <deller@....de>, Joachim Eastwood <manabian@...il.com>, Matthias Brugger <matthias.bgg@...il.com>, Masahiro Yamada <yamada.masahiro@...ionext.com>, Peter Hurley <peter@...leysoftware.com>, Sebastian Andrzej Siewior <bigeasy@...utronix.de>, Phillip Raffeck <phillip.raffeck@....de>, Heikki Krogerus <heikki.krogerus@...ux.intel.com>, Peter Hung <hpeter@...il.com>, linux-serial@...r.kernel.org, linux-kernel@...r.kernel.org, linux-parisc@...r.kernel.org, linux-kernel@...cs.fau.de Subject: Re: [PATCH v6 00/15] tty: serial: 8250: Fix checkpatch warnings On Wed, 2016-01-13 at 22:29 +0100, Anton Wuerfel wrote: > Hello Andy, > > > Also, please thin out the Cc list, I would recommend remove just > > ordinary commiters (with maybe few exceptions) and leave > > maintainers. > > Moreover some addresses are bounced. > I thinned out the Cc list and left over only maintainers and major > contributors to the code. > > > Yep! > > However, you forgot to fix switch-case as well. > Phillip and me are kind of git magicians. We can make code disappear > during > rebase :-) The piece of code was part of a commit, but it never made > it into > the final patch file. > > I hope, there are no major issues with our patch set. There are comments regarding patch 12, otherwise FWIW: Reviewed-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com> > > Regards, > Anton Wuerfel > Phillip Raffeck > > This patch set fixes several checkpatch warnings in tty/serial/8250. > > Patch 1/15: Adds missing spaces (mainly to function/macro headers) > Patch 2/15: Replace spaces with tabs wherever possible > Patch 3/15: Slight patch which moves an opening curly brace > Patch 4/15: Fixes multiline comment style > Patch 5/15: Removes else blocks after return statements > Patch 6/15: Slight patch which moves EXPORT_SYMBOL macro to correct > position > Patch 7/15: Slight patch which removes an unneccessary line > continuation > Patch 8/15: Slight patch which adds parentheses to a macro > definition > Patch 9/15: Merges user-visible multiline strings to a single line > Patch 10/15: Replace printk by dev_* or pr_* where appropriate > Patch 11/15: Remove orphaned debug macro > Patch 12/15: Fix warnings in dev_dbg by adding two casts > Patch 13/15: Fixes code indentation > Patch 14/15: Add a generic port macro > Patch 15/15: Refactor a switch/case statement > > Remaining checkpatch warnings after applying this patch series: > > -line over 80 characters > This error mostly occurs in serial_cs.c, which contains long- > lined > macro calls. However, splitting these calls into multiple lines > would > not increase readability. > > -externs should be avoided in .c files > This occurs in 8250_hp300.c. There is no corresponding header > file > the extern statement could be moved to. It could be moved to > 8250.h > but this would affect other .c files. > > -Use #include <linux/*.h> instead of <asm/*.h> > This warning has been left open for more experienced kernel > hackers. > This patch series is about style issues. We do not intend to > alter > the code behavior. > -struct uart_ops should normally be const > This warning only occurs in 8250_core.c. The corresponding > struct > cannot be declared as const because it is altered in > serial8250_isa_init_ports(). Maybe a checkpatch exception > should > be added for this particular warning. > > -quoted string split across lines > These strings were ignored because they otherwise would exceed > 80 characters in a single line. These particular strings use > format specifiers, which break the ability to grep for them > anyway. > > Anton Wuerfel (15): > Phillip Raffeck (15): > tty: serial: 8250: Fix whitespace errors > tty: serial: 8250: Replace spaces with tabs > tty: serial: 8250: Fix braces after struct > tty: serial: 8250: Fix multiline comment style > tty: serial: 8250: Remove else after return > tty: serial: 8250: Move EXPORT_SYMBOL to function > tty: serial: 8250: Fix line continuation warning > tty: serial: 8250: Add parentheses to macro > tty: serial: 8250: Fix multi-line strings > tty: serial: 8250: Correct conversion specifiers > tty: serial: 8250: Merge duplicate conditions > tty: serial: 8250: Fix indentation warnings > tty: serial: 8250: Add generic port init macro > tty: serial: 8250: Suitably replace printk > tty: serial: 8250: Remove SERIAL_DEBUG_PNP macro > > drivers/tty/serial/8250/8250.h | 12 ++++ > drivers/tty/serial/8250/8250_accent.c | 13 +--- > drivers/tty/serial/8250/8250_acorn.c | 2 +- > drivers/tty/serial/8250/8250_boca.c | 41 ++++++------- > drivers/tty/serial/8250/8250_core.c | 15 ++--- > drivers/tty/serial/8250/8250_exar_st16c554.c | 17 ++---- > drivers/tty/serial/8250/8250_fourport.c | 28 ++++----- > drivers/tty/serial/8250/8250_gsc.c | 7 ++- > drivers/tty/serial/8250/8250_hp300.c | 27 +++++---- > drivers/tty/serial/8250/8250_hub6.c | 2 +- > drivers/tty/serial/8250/8250_ingenic.c | 12 ++-- > drivers/tty/serial/8250/8250_pci.c | 60 ++++++++++------ > --- > drivers/tty/serial/8250/8250_pnp.c | 28 +++++---- > drivers/tty/serial/8250/8250_port.c | 47 ++++++++------- > drivers/tty/serial/8250/serial_cs.c | 90 +++++++++++++++--- > ---------- > 15 files changed, 203 insertions(+), 198 deletions(-) > -- Andy Shevchenko <andriy.shevchenko@...ux.intel.com> Intel Finland Oy
Powered by blists - more mailing lists