Re: [PATCH] staging/serqt_usb2: Refactor qt_status_change_check() inserqt_usb2.c

From: Dan Carpenter
Date: Fri Nov 30 2012 - 03:25:29 EST


On Thu, Nov 29, 2012 at 06:10:26PM -0800, Greg Kroah-Hartman wrote:
> On Thu, Nov 29, 2012 at 01:57:56PM +0900, YAMANE Toshiaki wrote:
> >
> > - ProcessLineStatus(qt_port, data[i + 3]);
> > -
> > i += 3;
> > + ProcessLineStatus(qt_port, data[i]);
>
> I think you just changed the logic in this function, didn't you?

This should be ok. The i += 3 was there in the original, it's just
moved up a line.

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/