"HU TAO-TGHK48" <taohu@xxxxxxxxxxxx> writes:
> Resend to linux-omap
>
> -----Original Message-----
> From: HU TAO-TGHK48
> Sent: Monday, August 31, 2009 7:50 PM
> To: 'vimal singh'; linux-omap@xxxxxxxxxxxxxxx; LKML;
> linux-serial@xxxxxxxxxxxxxxx
> Cc: Ye Yuan.Bo-A22116; Chen Xiaolong-A21785
> Subject: RE: [RFC][PATCH]: Adding support for omap-serail driver
>
>
> 1. Shall we cleanup PM related stuff in arch/arm/mach-omap2/serial.c as
> well?
> Originally serail.c register UART IRQ to decide if UART idle for a
> while and is able to enter low power mode (e.g. retention).
> To work with original 8250 driver, it is probably the only way since
> 8250 is not aware of OMAP PM.
>
> However it would be more reasonable to merge PM stuff to
> omap-serial.c. since the new driver is already OMAP specific
>
> 2. There is an issue for DMA with current implementation in serial.c
> When Rx DMA is active NO Rx IRQ will be generated.
> So serial.c will easily set uart->can_sleep with "1" even there is
> Rx DMA ongoing
> + if ((iir & 0x4) && up->use_dma) {
> + up->ier &= ~UART_IER_RDI;
> + serial_out(up, UART_IER, up->ier
>
> In my view, the best way is to do the idle detection in
> omap_serial.c.
FWIW, as the author of much of the PM hacker in mach-omap2/serial.c, I
agree with Tao.
The only reason for the PM hackery in mach-omap2/serial.c is because
of the limitations of the 8250 driver.
Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
