On 03/09/18 15:47, Ian Abbott wrote:
On 30/08/18 18:32, Ray Clinton wrote:
Using checkpatch.pl I was able to find a multiline dereference which goes
again the coding style for the kernel. I'm still working on my email
client so
the indentation looks bad here (in gmail) but the arguments for
comedi_check_trigger_arg_min should go just under the opening (
The patch description should describe what the patch does, not how bad
the indentation loos in your email client.
"loos" ==> "looks"
Also, you sent 4 patches with the same subject line (other than the
PATCH 1/4 part). They should be different. You can include the driver
name in the patch subject line to distinguish them, e.g.:
staging: comedi: dt3000: Correct multiline dereference
--
-=( Ian Abbott <[email protected]> || Web: www.mev.co.uk )=-
-=( MEV Ltd. is a company registered in England & Wales. )=-
-=( Registered number: 02862268. Registered address: )=-
-=( 15 West Park Road, Bramhall, STOCKPORT, SK7 3JZ, UK. )=-
