Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752039AbbEZO2z (ORCPT ); Tue, 26 May 2015 10:28:55 -0400 Received: from mga01.intel.com ([192.55.52.88]:3527 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751851AbbEZO2x (ORCPT ); Tue, 26 May 2015 10:28:53 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,498,1427785200"; d="scan'208";a="731838705" Message-ID: <1432650518.13900.72.camel@linux.intel.com> Subject: Re: [PATCH v6] serial: 8250_uniphier: add UniPhier serial driver From: Alan Cox To: "Shevchenko, Andriy" Cc: "yamada.masahiro@socionext.com" , "manabian@gmail.com" , "bigeasy@linutronix.de" , "linux-kernel@vger.kernel.org" , "alan@lxorguk.ukuu.org.uk" , "ricardo.ribalda@gmail.com" , "blogic@openwrt.org" , "linux-serial@vger.kernel.org" , "gregkh@linuxfoundation.org" , "jslaby@suse.cz" , "matthias.bgg@gmail.com" , "peter@hurleysoftware.com" , "linux-arm-kernel@lists.infradead.org" Date: Tue, 26 May 2015 15:28:38 +0100 In-Reply-To: <1432545245.8736.5.camel@intel.com> References: <1432525477-14051-1-git-send-email-yamada.masahiro@socionext.com> <1432545245.8736.5.camel@intel.com> Organization: Intel Corporation Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.11 (3.12.11-1.fc21) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 975 Lines: 29 > > + > > +#define UNIPHIER_UART_CHAR_FCR 3 /* Character / FIFO Control Register */ > > +#define UNIPHIER_UART_LCR_MCR 4 /* Line/Modem Control Register */ > > +#define UNIPHIER_UART_LCR_SHIFT 8 > > Indentation problem, needs to be fixed. If you are going to review a patch set at least look at the previous reviews - the indentation was already discussed and is done that way to show (as many drivers do) which are fields for which registers > > +static unsigned int uniphier_serial_in(struct uart_port *p, int offset) > > +{ > > + int valshift = 0; > > Perhaps unsigned int? Why ? even if it mattered gcc is already realising that the value can only be 0 or 8 and will be generating whatever works best for that. Alan -- 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/