Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752616AbbLRLWb (ORCPT ); Fri, 18 Dec 2015 06:22:31 -0500 Received: from faui40.informatik.uni-erlangen.de ([131.188.34.40]:38879 "EHLO faui40.informatik.uni-erlangen.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752035AbbLRLW2 (ORCPT ); Fri, 18 Dec 2015 06:22:28 -0500 From: Anton Wuerfel To: linux-serial@vger.kernel.org Cc: Greg Kroah-Hartman , Jiri Slaby , "James E.J. Bottomley" , Helge Deller , Peter Hurley , Heikki Krogerus , Andy Shevchenko , 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, Anton Wuerfel , Phillip Raffeck Subject: [PATCH v2 00/11] tty: serial: 8250: Fix checkpatch warnings Date: Fri, 18 Dec 2015 12:21:52 +0100 Message-Id: <1450437723-2978-1-git-send-email-anton.wuerfel@fau.de> X-Mailer: git-send-email 1.9.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3744 Lines: 88 Hello Peter, thanks for your feedback. We updated the patches according to your remarks. Regards, Phillip Raffeck Anton Wuerfel This patch set fixes several checkpatch warnings in tty/serial/8250. Patch 1/11: Adds missing spaces (mainly to function/macro headers) Patch 2/11: Replace spaces with tabs wherever possible Patch 3/11: Slight patch which moves an opening curly brace Patch 4/11: Fixes multiline comment style Patch 5/11: Removes else blocks after return statements Patch 6/11: Slight patch which moves EXPORT_SYMBOL macro to correct position Patch 7/11: Slight patch which removes an unneccessary line continuation Patch 8/11: Slight patch which adds parentheses to a macro definition Patch 9/11: Merges user-visible multiline strings to a single line Patch 10/11: Replaces printk by corresponding variant of pr_* Patch 11/11: Fixes code indentation 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 instead of 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 (11): Phillip Raffeck (11): 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: Replace printk by pr_* tty: serial: 8250: Fix indentation warnings drivers/tty/serial/8250/8250_accent.c | 2 +- drivers/tty/serial/8250/8250_acorn.c | 2 +- drivers/tty/serial/8250/8250_boca.c | 2 +- drivers/tty/serial/8250/8250_core.c | 15 ++--- drivers/tty/serial/8250/8250_exar_st16c554.c | 2 +- drivers/tty/serial/8250/8250_fourport.c | 2 +- drivers/tty/serial/8250/8250_gsc.c | 6 +- drivers/tty/serial/8250/8250_hp300.c | 25 ++++---- drivers/tty/serial/8250/8250_hub6.c | 2 +- drivers/tty/serial/8250/8250_ingenic.c | 12 ++-- drivers/tty/serial/8250/8250_pci.c | 49 +++++++-------- drivers/tty/serial/8250/8250_pnp.c | 16 ++--- drivers/tty/serial/8250/8250_port.c | 50 ++++++++-------- drivers/tty/serial/8250/serial_cs.c | 89 +++++++++++++++------------- 14 files changed, 142 insertions(+), 132 deletions(-) -- 1.9.1 -- 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/