Messages in this thread | | | Date | Tue, 07 Jul 2015 11:47:01 -0500 | From | Jeremy White <> | Subject | Re: [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
| |