lkml.org 
[lkml]   [2015]   [Jul]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC PATCH 1/1] Add a usbredir kernel module to remotely connect USB devices over IP.

>>
>> Well, the checkpatch.pl reports were all style (and mostly whitespace);
>> roughly 3000 of them against 3000 lines of code :-/. I did review the
>> code, looking for areas where I thought it would badly cram into the
>> kernel, and I adjusted the few I found (and sent changes upstream).
>
> style matters, as it's a thing with your brain. You learn patterns and
> if the patterns change, you have to do more work and don't see the real
> issues involved. So by ignoring our style you are saying you don't want
> anyone else in the kernel community to ever review or work on the code,
> which isn't ok.

Looks like I can't side step this unless Hans is willing to shift the
usbredir project entirely to using kernel style :-/.

I will plan to make changes so that checkpatch runs clean; I lay out my
concerns and my plan below to make sure I'm taking the best path.

My main concern with changing the ~2,500 lines of code from the upstream
usbredir project is that it will increase the odds that I will introduce
errors, both initially, and again later as I review and attempt to relay
patches from the upstream.

To summarize the checkpatch reports: the biggest issue is whitespace,
which shouldn't be a problem; I should be able to automate that without
error. There are also a fair number of one offs; FSF address, space
after '!', etc. I hope to persuade Hans to take a few style only
patches upstream to address those. That leaves a pack of about 60 brace
placement and line length issues.

I will plan to manually change those prior to submission. Any upstream
changes that affect the same code will be manually corrected as well,
prior to submission.

Make sense?

Cheers,

Jeremy


\
 
 \ /
  Last update: 2015-07-07 19:01    [W:0.085 / U:0.876 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site