2009-09-02 08:48:16

by Govindraj

[permalink] [raw]
Subject: Re: [RFC][PATCH]: Adding support for omap-serail driver

Vikram,


> What about UART3 supporting IRDA and CIR modes?
> Is that planned to be added?
>

We do have an driver to support IrDA -- > drivers/net/irda/omap-ir.c


>>+static unsigned int check_modem_status(struct uart_omap_port *up)
>
> What is the use case for modem_status?

This is basically used to handle any change in uart line[cts,dcd],
like an change in status of cts line should be handled
which is done using the function by checking the MSR[modem status register].

>>+ ? ? ? ? ? ? ?if (jiffies_to_msecs(jiffies - up_activity) < 10000) {
>>+ ? ? ? ? ? ? ? ? ? ? ?mod_timer(&up->uart_dma.rx_timer, jiffies +
>>+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?usecs_to_jiffies(up->uart_dma.rx_timeout));
>
> Is this a 10 second timeout? Is this acceptable way?
> This has to be done in conjunction with the inactivity timer in mach-omap2/serial.c

This timeout is the period where we except some activity on rx line as
we dont the time period we receive data hence we keep rx dma channel
active for minimum of 10 secs, however this can be reduced.


-----
Regards,
Govindraj.R