[PATCH 1/5] serial: Introduce UPSTAT_SYNC_FIFO for synchronised FIFOs
Jeremy Kerr
jk at ozlabs.org
Tue Mar 27 12:38:04 AEDT 2018
Hi Greg,
>> --- a/drivers/tty/serial/serial_core.c
>> +++ b/drivers/tty/serial/serial_core.c
>> @@ -678,7 +678,7 @@ static void uart_throttle(struct tty_struct *tty)
>> if (C_CRTSCTS(tty))
>> mask |= UPSTAT_AUTORTS;
>>
>> - if (port->status & mask) {
>> + if (port->status & (mask | UPSTAT_SYNC_FIFO)) {
>> port->ops->throttle(port);
>> mask &= ~port->status;
>> }
>
> Why not just set mask to UPSTAT_SYNC_FIFO at the top of this function?
That was a bit of a line call - my thinking was that SYNC_FIFO is a
little different from the calculated bit values in mask, hence keeping
it separate and adding it explicitly at the check
However, adding it to the declaration of mask is a little simpler, so
I'll redo that in v2.
Cheers,
Jeremy
More information about the openbmc
mailing list