Thread (7 messages) read the whole thread 7 messages, 4 authors, 2017-02-23

Re: [PATCH v2] staging: wilc1000: renames struct tstrRSSI and its members u8Index, u8Full

From: Joe Perches <joe@perches.com>
Date: 2017-02-23 06:26:49
Also in: lkml

On Wed, 2017-02-22 at 20:50 +0100, Arend Van Spriel wrote:
On 22-2-2017 18:14, Tahia Khan wrote:
quoted
Fixes multiple camel case checks on struct tstrRSSI from checkpatch.pl:
[]
Just a generic remark that may help you with other changes you will be
making in the linux kernel. Warnings from checkpatch.pl and other tools
are useful, but try to look further than just fixing a warning.
Understand what the code is doing is just as important.
I'd assert understanding what the code is doing is
_more_ important.  Style consistency simply helps
improve the speed of a new reader's understanding.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help