Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH 1/4] staging: comedi: Correct multiline dereference as per
 TODO
From:   Ian Abbott <abbotti@mev.co.uk>
To:     Ray Clinton <mr.ray.clinton@gmail.com>, greg@kroah.com
Cc:     hsweeten@visionengravers.com, linux-kernel@vger.kernel.org
References: <CAPMnawvndQUGj4ohrTCPrwcvZdS-3FrHGsNeN4N7OJ_YGK-QfQ@mail.gmail.com>
 <a7eeb3a5-ba6a-14c4-8ed4-7aa29d3840ce@mev.co.uk>
Organization: MEV Ltd.
Message-ID: <4b023c3d-eaeb-6466-5bc8-caed57fbd48b@mev.co.uk>
Date:   Mon, 3 Sep 2018 15:52:21 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <a7eeb3a5-ba6a-14c4-8ed4-7aa29d3840ce@mev.co.uk>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-GB
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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 <abbotti@mev.co.uk> || 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. )=-