linux-kernel - RE: [PATCH V2] tty/serial: add support for Xilinx PS UART

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 22 Apr 2011 07:40:06 -0600
From:	John Linn <John.Linn@...inx.com>
To:	Joe Perches <joe@...ches.com>, <michal.simek@...alogix.com>
CC:	<linux-kernel@...r.kernel.org>, <linux-serial@...r.kernel.org>,
	<alan@...rguk.ukuu.org.uk>, <greg@...ah.com>,
	<grant.likely@...retlab.ca>
Subject: RE: [PATCH V2] tty/serial: add support for Xilinx PS UART

> -----Original Message-----
> From: Joe Perches [mailto:joe@...ches.com]
> Sent: Friday, April 22, 2011 3:21 AM
> To: michal.simek@...alogix.com
> Cc: John Linn; linux-kernel@...r.kernel.org; linux-
> serial@...r.kernel.org; alan@...rguk.ukuu.org.uk; greg@...ah.com;
> grant.likely@...retlab.ca
> Subject: Re: [PATCH V2] tty/serial: add support for Xilinx PS UART
> 
> 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.

Thanks Joe.  I'm surprised checkpatch didn't catch that.  Easy enough
to fix on the next spin of the patch.

I thought checkpatch did complain if the lines were longer than 80, but
maybe there are some exceptions I didn't realize.  

Thanks for your time and input, learning more everyday :)
John

> 
> 			dev_err(&pdev->dev,
> 				"uart_add_one_port() failed; err=%d\n", rc);
> 
> 


This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.