[PATCH] amba-pl011: support hardware flow control

Russell King - ARM Linux linux at arm.linux.org.uk
Tue Feb 9 13:01:09 EST 2010


On Tue, Feb 09, 2010 at 08:00:56PM +0530, Rabin VINCENT wrote:
> On Mon, Feb 08, 2010 at 02:51:24PM +0100, Russell King - ARM Linux wrote:
> > On Mon, Feb 08, 2010 at 07:11:50PM +0530, Rabin Vincent wrote:
> > > Enable/disable hardware flow control as requested by the termios.
> > 
> > From what I remember, I didn't implement this because there were
> > corner cases to be dealt with.  I don't remember what they were
> > off hand though.
> 
> We haven't seen any problems with it thus far.  Nothing obvious strikes
> me on rechecking the pl011 spec; it would be helpful if you could
> elaborate.

Can't elaborate at the moment without re-reading the specs.

> Either way, this patch should be safe to apply since existing users of
> the driver shouldn't be affected.

Err, that really doesn't follow.  There are existing users, and existing
users will have CRTSCTS set by default.  Therefore, existing users are
going from having hardware flow control on the UART disabled to having
it enabled.

That's not "shouldn't be affected", that's a "this patch does change
the hardware behaviour for these users".



More information about the linux-arm-kernel mailing list