[PATCH 06/16] tty: serial: Add 8250-core based omap driver
fransklaver at gmail.com
Mon Sep 29 06:34:12 PDT 2014
On Mon, Sep 29, 2014 at 3:27 PM, Sebastian Andrzej Siewior
<bigeasy at linutronix.de> wrote:
> * Frans Klaver | 2014-09-29 11:38:23 [+0200]:
>>> + /*
>>> + * It claims to be 16C750 compatible however it is a little different.
>>> + * It has EFR and has no FCR7_64byte bit. The AFE (which it claims to
>>> + * have) is enabled via EFR instead of MCR. The type is set here 8250
>>> + * just to get things going. UNKNOWN does not work for a few reasons and
>>> + * we don't need our own type since we don't use 8250's set_termios()
>>> + * and our "bugs" are handeld via the bug member.
> replaced that last line with
> or pm callback.
> since there no bugs member anymore.
>>> + */
>>> + up.port.type = PORT_8250;
>>> + up.port.iotype = UPIO_MEM;
>>> + up.port.flags = UPF_FIXED_PORT | UPF_FIXED_TYPE | UPF_SOFT_FLOW |
>>> + UPF_HARD_FLOW;
>>> + up.port.private_data = priv;
>>> + up.port.regshift = 2;
>>> + up.port.fifosize = 64;
>>> + up.tx_loadsz = 64;
>>> + up.capabilities = UART_CAP_FIFO | UART_CAP_EFR | UART_CAP_SLEEP;
>>> +#ifdef CONFIG_PM_RUNTIME
>>> + /*
>>> + * PM_RUNTIME is mostly transparent. However to do it right we need to a
>>need to _do_ a ...?
> I think dropping that 'to' should fix it.
>>> + * TX empty interrupt before we can put the device to auto idle. So if
>>> + * PM_RUNTIME is not enabled we don't add that flag and can spare that
>>> + * one extra interrupt in the TX path.
>>> + */
>>> +config SERIAL_8250_OMAP
>>> + tristate "Support for OMAP internal UART (8250 based driver)"
>>> + depends on SERIAL_8250 && ARCH_OMAP2PLUS
>>> + help
>>> + If you have a machine based on an Texas Instruments OMAP CPU you
>>> + can enable its onboard serial ports by enabling this option.
>>> + This driver is in early stage and uses ttyS instead of ttyO.
>>I just wondered if this driver should be marked experimental?
> What did you have in mind? CONFIG_EXPERIMENTAL is gone. After all that
> debuging that I had in the meantime I was thinking about dropping that
> "early stage".
That was the other option. I'm good with that. Also, I never noticed
CONFIG_EXPERIMENTAL being gone, so that's down the drain ;).
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
More information about the linux-arm-kernel