[adding linux-omap list] Vasiliy Kulikov <segooon@xxxxxxxxx> writes: > "ret" is unsigned, so check for (ret < 0) made no sense. > Made it signed. > > Signed-off-by: Vasiliy Kulikov <segooon@xxxxxxxxx> > --- > I cannot compile this driver, so it is not tested. Acked-by: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> Since the original driver is being queued via the omap tree, we can queue this one as well. Tony, can you add this one to the 2.6.37 queue? Thanks, Kevin > drivers/serial/omap-serial.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/serial/omap-serial.c b/drivers/serial/omap-serial.c > index 2ee1d32..14365f7 100644 > --- a/drivers/serial/omap-serial.c > +++ b/drivers/serial/omap-serial.c > @@ -1055,7 +1055,7 @@ static void serial_omap_rx_timeout(unsigned long uart_no) > { > struct uart_omap_port *up = ui[uart_no]; > unsigned int curr_dma_pos, curr_transmitted_size; > - unsigned int ret = 0; > + int ret = 0; > > curr_dma_pos = omap_get_dma_dst_pos(up->uart_dma.rx_dma_channel); > if ((curr_dma_pos == up->uart_dma.prev_rx_dma_pos) || -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html