Messages in this thread | | | Date | Fri, 30 Nov 2012 21:53:57 +0900 | Subject | Re: [PATCH] staging/serqt_usb2: Refactor qt_status_change_check() in serqt_usb2.c | From | YAMANE Toshiaki <> |
| |
On Fri, Nov 30, 2012 at 5:25 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote: > 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.
Thanks for your kindness reply.
But please discard this patch. Let me consider this patch again.
Regards,
YAMANE Toshiaki
| |