Larry Finger <Larry.Finger@xxxxxxxxxxxx> writes: > This set of patches fix all instances of CamelCase for all the drivers. > When the symbol refers to a macro, it is converted to upper case. If a > variable or a function, then it is converted to lower case. > > As noted in the comments, some of the other checkpatch exceptions are > addressed. I wish you had separated the camel case fixes from other cleanups, and otherwise tried to simplify the patches, now you made the patchset painful to review as all the changes are mixed and the size of patches is just too large. Remember that the rule of thumb is one logical change per patch. Of course that's not a hard rule, and can be ignored with small and trivial patches, but most of the patches here are not small. No need to resend because of this but please take care with the size of patches and patchsets in the future. -- Kalle Valo
