lkml.org 
[lkml]   [2015]   [Jan]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 1/5] usb: serial: add register map for F81232
On Thu, Jan 29, 2015 at 10:37:34AM +0800, Peter Hung wrote:
> Hello.
>
> 1. For retry Issue:
>
> These patches is referenced from our other usb serial product. That
> product maybe not ack the control ep command when It's in very heavily
> loading. Our workaround is to modify driver to retry more times if it
> timeout because it's f/w can't upgrade with usb protocol. I will remove
> the retry mechanic and test for it again.

That sounds good.

> 2. For some function/variable not used:
>
> The original driver is lacking a lot of feature. My first patch strategy
> is putting all needed function / variables in patch 1 and apply it in
> following patches. Sorry for my wrong strategy, I'll change it with more
> meaningful and logical patchset and resend it.

Perhaps you just need to break the patches up further.

Also if you get checkpatch warnings for code you haven't touched, you
can either ignore them or fix it separately in preparatory patch.

Just try to group related changes together.

> Thanks for your patient and advice.

No worries.

In the future when posting to the lists, please try to respond inline
(as I am above) rather than top-post, and remove context that isn't
relevant (e.g. code that isn't relevant) for the discussion at hand.

Thanks,
Johan


\
 
 \ /
  Last update: 2015-01-29 11:21    [W:0.070 / U:0.060 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site