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
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
