All docs I checked do say LCR should NOT be 0xBF (16c850, 16c854, 16v2750). Fix it according to that, since the only usage of the register currently implements the revised access method. If somebody later finds a derivate which actually does need 0xBF to enable that register, we can update the comment again with precise references. Signed-off-by: Wolfram Sang <w.sang@xxxxxxxxxxxxxx> --- include/linux/serial_reg.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/linux/serial_reg.h b/include/linux/serial_reg.h index 0267bab6..0a1beca 100644 --- a/include/linux/serial_reg.h +++ b/include/linux/serial_reg.h @@ -211,7 +211,7 @@ #define UART_FCTR_TX 0x80 /* Programmable trigger mode select */ /* - * LCR=0xBF, FCTR[6]=1 + * LCR!=0xBF (some docs say =0), FCTR[6]=1, XR16C85x */ #define UART_EMSR 7 /* Extended Mode Select Register */ #define UART_EMSR_FIFO_COUNT 0x01 /* Rx/Tx select */ -- 1.7.7.3 -- 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
