From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH V2] tty/serial: add support for Xilinx PS UART Date: Fri, 22 Apr 2011 02:20:33 -0700 Message-ID: <1303464033.26896.11.camel@Joe-Laptop> References: <90e49570-bc82-41a9-ac79-2f5007e93a2a@VA3EHSMHS018.ehs.local> <4DB14626.8090002@petalogix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mail.perches.com ([173.55.12.10]:4912 "EHLO mail.perches.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752891Ab1DVJUf (ORCPT ); Fri, 22 Apr 2011 05:20:35 -0400 In-Reply-To: <4DB14626.8090002@petalogix.com> Sender: linux-serial-owner@vger.kernel.org List-Id: linux-serial@vger.kernel.org To: michal.simek@petalogix.com Cc: John Linn , linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, alan@lxorguk.ukuu.org.uk, greg@kroah.com, Grant Likely On Fri, 2011-04-22 at 11:11 +0200, Michal Simek wrote: > John Linn wrote: > > The Xilinx PS Uart is used on the new ARM based SoC. This > > UART is not compatible with others such that a seperate > > driver is required. [] > > +++ b/drivers/tty/serial/xilinx_uartps.c [] > > + rc = uart_add_one_port(&xuartps_uart_driver, port); > > + if (rc) { > > + dev_err(&pdev->dev, "uart_add_one_port() failed; \ > > + err=%i\n", rc); That's a whitespace error. Please don't be concerned about 80 column lines used for formatting. You could use something like this as well. dev_err(&pdev->dev, "uart_add_one_port() failed; err=%d\n", rc);