Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752666AbdFSWBf (ORCPT ); Mon, 19 Jun 2017 18:01:35 -0400 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:34173 "EHLO out4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752167AbdFSWBd (ORCPT ); Mon, 19 Jun 2017 18:01:33 -0400 X-ME-Sender: X-Sasl-enc: rLGIg/eH+eXnzVcCYPW8kYntw4DZmhn/uoLTL6PuNjCh 1497909681 Date: Tue, 20 Jun 2017 08:01:19 +1000 From: "Tobin C. Harding" To: "Kershner, David A" Cc: Derek Robson , "gregkh@linuxfoundation.org" , "Sell, Timothy C" , "Binder, David Anthony" , "Wadgaonkar, Sameer Laxmikant" , "marcos.souza.org@gmail.com" , "devel@driverdev.osuosl.org" , *S-Par-Maintainer , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH V2] staging: unisys: visorhba - style fix Message-ID: <20170619220119.GC32305@eros> References: <20170617031255.14575-1-robsonde@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Mailer: Mutt 1.5.24 (2015-08-30) User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2238 Lines: 63 On Mon, Jun 19, 2017 at 03:28:19PM +0000, Kershner, David A wrote: > > > -----Original Message----- > > From: Derek Robson [mailto:robsonde@gmail.com] > > Sent: Friday, June 16, 2017 11:13 PM > > To: Kershner, David A ; > > gregkh@linuxfoundation.org; Sell, Timothy C ; > > Binder, David Anthony ; Wadgaonkar, Sameer > > Laxmikant ; > > marcos.souza.org@gmail.com; robsonde@gmail.com > > Cc: *S-Par-Maintainer ; > > devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org > > Subject: [PATCH V2] staging: unisys: visorhba - style fix > > > > Fixed style of permissions to octal. > > Found using checkpatch > > > > Signed-off-by: Derek Robson > > I applied it and tested it. Though when I applied it, the V1 comment was included in the patch commit. Derek, for clarification what David means is that the version notes should go below the dotted line so as not to be included in the patch commit. You may also like to read Documentation/process/submitting-patches.rst for tips on writing your commit log (specifically section 2). > > > > > > V1 has vauge subject > > --- v1 -> v2 - fix vague subject > > drivers/staging/unisys/visorhba/visorhba_main.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/staging/unisys/visorhba/visorhba_main.c > > b/drivers/staging/unisys/visorhba/visorhba_main.c > > index 2fd31c9762c6..a6e7a6bbc428 100644 > > --- a/drivers/staging/unisys/visorhba/visorhba_main.c > > +++ b/drivers/staging/unisys/visorhba/visorhba_main.c > > @@ -1090,7 +1090,7 @@ static int visorhba_probe(struct visor_device *dev) > > goto err_scsi_remove_host; > > } > > devdata->debugfs_info = > > - debugfs_create_file("info", S_IRUSR | S_IRGRP, > > + debugfs_create_file("info", 0440, > > devdata->debugfs_dir, devdata, > > &info_debugfs_fops); > > if (!devdata->debugfs_info) { > > -- > > 2.13.0 > > _______________________________________________ > devel mailing list > devel@linuxdriverproject.org > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel Good luck, Tobin.