linux-kernel - Re: [PATCH V3] 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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <126598808.1934339.1303910496698.JavaMail.root@sz0140a.emeryville.ca.mail.comcast.net>
Date:	Wed, 27 Apr 2011 13:21:36 +0000 (UTC)
From:	johnlinn@...cast.net
To:	Alan Cox <alan@...rguk.ukuu.org.uk>
Cc:	linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org,
	michal simek <michal.simek@...alogix.com>, joe@...ches.com,
	grant likely <grant.likely@...retlab.ca>, greg@...ah.com
Subject: Re: [PATCH V3] tty/serial: add support for Xilinx PS UART


----- "Alan Cox" <alan@...rguk.ukuu.org.uk> wrote:

> > I thought I was improving but now as I look at it further, that
> seems like it 
> > was better and I should have stuck to that method.
> > 
> > Thoughts?
> 
> Go for nearest is the right behaviour. The whole tty layer works on
> that
> basis as do soem other things. It means you don't have to have a set
> of
> calls to do things like "enumerate valid data rates" instead you ask
> and
> compare.

I wasn't very clear on that, sorry about that.

I meant we were not detecting if the error was too much and finding the 
nearest baud rate that had an acceptable amount of error. 

We were just finding the best baud rate generator values for the baud and
then using them without any kind of error.

Thanks,
John

> 
> > > > +	uart_update_timeout(port, termios->c_cflag, baud);
> > > 
> > > [which would also fix this timeout!]
> > 
> > Maybe you were meaning that if the baud rate was not really set
> because of the
> > error this timeout would be wrong also, and if so that makes sense.
> 
> Correct
> 
> Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/