Re: [PATCH] tty: serial: qcom_geni_serial: Don't slow all ports just for kgdb

2018-10-17 Thread msavaliy
On 2018-10-16 02:14, Douglas Anderson wrote: If you turn on CONFIG_KGDB then you'll get CONFIG_CONSOLE_POLL selected. If you have CONFIG_CONSOLE_POLL selected then the GENI serial driver was setting RX_BYTES_PW to 1 for _all_ UART ports. This doesn't seem like such a good idea. Let's only set

Re: [PATCH] tty: serial: qcom_geni_serial: Don't slow all ports just for kgdb

2018-10-17 Thread msavaliy
On 2018-10-16 02:14, Douglas Anderson wrote: If you turn on CONFIG_KGDB then you'll get CONFIG_CONSOLE_POLL selected. If you have CONFIG_CONSOLE_POLL selected then the GENI serial driver was setting RX_BYTES_PW to 1 for _all_ UART ports. This doesn't seem like such a good idea. Let's only set

Re: [PATCH] tty: serial: qcom_geni_serial: Don't slow all ports just for kgdb

2018-10-16 Thread Stephen Boyd
Quoting Douglas Anderson (2018-10-15 13:44:24) > If you turn on CONFIG_KGDB then you'll get CONFIG_CONSOLE_POLL > selected. > > If you have CONFIG_CONSOLE_POLL selected then the GENI serial driver > was setting RX_BYTES_PW to 1 for _all_ UART ports. > > This doesn't seem like such a good idea.

Re: [PATCH] tty: serial: qcom_geni_serial: Don't slow all ports just for kgdb

2018-10-16 Thread Stephen Boyd
Quoting Douglas Anderson (2018-10-15 13:44:24) > If you turn on CONFIG_KGDB then you'll get CONFIG_CONSOLE_POLL > selected. > > If you have CONFIG_CONSOLE_POLL selected then the GENI serial driver > was setting RX_BYTES_PW to 1 for _all_ UART ports. > > This doesn't seem like such a good idea.

[PATCH] tty: serial: qcom_geni_serial: Don't slow all ports just for kgdb

2018-10-15 Thread Douglas Anderson
If you turn on CONFIG_KGDB then you'll get CONFIG_CONSOLE_POLL selected. If you have CONFIG_CONSOLE_POLL selected then the GENI serial driver was setting RX_BYTES_PW to 1 for _all_ UART ports. This doesn't seem like such a good idea. Let's only set RX_BYTES_PW to 1 for the console port.

[PATCH] tty: serial: qcom_geni_serial: Don't slow all ports just for kgdb

2018-10-15 Thread Douglas Anderson
If you turn on CONFIG_KGDB then you'll get CONFIG_CONSOLE_POLL selected. If you have CONFIG_CONSOLE_POLL selected then the GENI serial driver was setting RX_BYTES_PW to 1 for _all_ UART ports. This doesn't seem like such a good idea. Let's only set RX_BYTES_PW to 1 for the console port.