On Fri, 26 Feb 2010 17:59:38 +0800 Masakazu Mokuno <mokuno@xxxxxxxxxxxxx> wrote: > Hi, > > > + u8 *pbuf, valid_str[M3110_RX_FIFO_DEPTH]; > > + int i, j; > > + > > + memset(obuf, 0, sizeof(obuf)); > > + memset(obuf, 0, sizeof(ibuf)); > > memset(ibuf, 0, sizeof(ibuf))? thanks for the catch, here is a follow on patch diff --git a/drivers/serial/max3110.c b/drivers/serial/max3110.c index 3283ba6..3dd2082 100644 --- a/drivers/serial/max3110.c +++ b/drivers/serial/max3110.c @@ -142,7 +142,7 @@ static int max3110_read_multi(struct uart_max3110 *max, u8 *buf) int i, j; memset(obuf, 0, sizeof(obuf)); - memset(obuf, 0, sizeof(ibuf)); + memset(ibuf, 0, sizeof(ibuf)); if (max3110_write_then_read(max, obuf, ibuf, M3110_RX_FIFO_DEPTH * 2, 1)) -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html