Messages in this thread | | | Date | Mon, 23 Aug 2021 11:54:56 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH 1/2] staging/rtl8192u: Avoid CamelCase in names of variables |
| |
On Sun, Aug 22, 2021 at 08:02:14AM -0700, Kees Cook wrote: > On Sun, Aug 22, 2021 at 04:28:19PM +0200, Len Baker wrote: > > Avoid CameCase in the names of all local variables inside the function > > rtl8192_phy_SwChnlStepByStep(). > > This mixes decamelization with some (minor) logic changes (moving > initializations earlier). I'd normally do this kind of thing with a sed > script and not touch anything else, so that the results can be compared > against the sed command. And I'd include the sed command in the commit > log.
Yep. Absolutely verboten on staging.
> > I'm actually not sure what the norm is in the kernel for doing > decamelization -- should the entire driver get decamelized at once, > instead of just one function at a time? Greg, do you have an opinion > here?
It doesn't matter. One function at a time is fine. So long as it's not too long to review.
regards, dan carpenter
| |