From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [1/2] dmaengine: dmatest: change symbolic permissions to octal values From: Vinod Koul Message-Id: <20180130100444.GU18649@localhost> Date: Tue, 30 Jan 2018 15:34:45 +0530 To: "Yang, Shunyong" Cc: "dan.j.williams@intel.com" , "dmaengine@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-ID: T24gTW9uLCBKYW4gMjksIDIwMTggYXQgMDY6MjY6NDBBTSArMDAwMCwgWWFuZywgU2h1bnlvbmcg d3JvdGU6Cj4gSGksIFZpbm9kCj4gCj4gT24gTW9uLCAyMDE4LTAxLTI5IGF0IDEwOjE4ICswNTMw LCBWaW5vZCBLb3VsIHdyb3RlOgo+ID4gT24gTW9uLCBKYW4gMjIsIDIwMTggYXQgMDY6NDQ6NDBQ TSArMDgwMCwgWWFuZyBTaHVueW9uZyB3cm90ZToKPiA+ID4gCj4gPiA+IEN1cnJlbnQgY29kaW5n IHN0eWxlIHByZWZlcnMgb2N0YWwgcGVybWlzc2lvbnMgdmFsdWVzLiBUaGlzIHBhdGNoCj4gPiA+ IGNoYW5nZXMgc3ltYm9saWMgcGVybWlzc2lvbnMgdG8gb2N0YWwgdmFsdWVzLgo+ID4gSXMgdGhp cyBwcmVmZXJlbmNlIGRvY3VtZW50ZWQgYW55d2hlcmU/Cj4gPiAKPiAKPiBXaGVuIHVzaW5nIHN5 bWJvbGljIHBlcm1pc3Npb25zIGxpa2UgIlNfSVJVR08gfCBTX0lXVVNSIi4gVGhlCj4gY2hlY2tw YXRjaC5wbCB3aWxsIG91dHB1dCBzb21lIHdhcm5pbmdzIHRvIHN1Z2dlc3QgdG8gdXNlIG9jdGFs IHZhbHVlcy4KPiBJIHF1b3RlIGZvbGxvd2luZyBsaW5lcyBmcm9tIGNoZWNrcGF0Y2gucGwsCj4g Cj4gIyBjaGVjayBmb3IgdXNlcyBvZiBTXzxQRVJNUz4gdGhhdCBjb3VsZCBiZSBvY3RhbCBmb3Ig cmVhZGFiaWxpdHkKPiAuLi4KPiBpZiAoV0FSTigiU1lNQk9MSUNfUEVSTVMiLAo+IMKgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oCJTeW1ib2xpYyBwZXJtaXNzaW9ucyAnJG92YWwnIGFyZSBub3QKPiBwcmVmZXJyZWQuIENvbnNp ZGVyIHVzaW5nIG9jdGFsIHBlcm1pc3Npb25zICckb2N0YWwnLlxuIiAuICRoZXJlY3VycikKPiAm JgoKSXQgaXMgYSB3YXJuaW5nIGFuZCBhIHByZWZlcmVuY2UuIEFuZCBJIGRvbid0IHByZWZlciBj aGFuZ2luZyBjb2RlIGZvciBubwpiZW5lZml0LiBTb3JyeSBidXQgSSBhbSBub3QgY29uc2lkZXJp bmcgdGhpcyBjaGFuZ2UK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752117AbeA3KAP (ORCPT ); Tue, 30 Jan 2018 05:00:15 -0500 Received: from mga05.intel.com ([192.55.52.43]:7213 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751391AbeA3KAL (ORCPT ); Tue, 30 Jan 2018 05:00:11 -0500 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,434,1511856000"; d="scan'208";a="199748904" Date: Tue, 30 Jan 2018 15:34:45 +0530 From: Vinod Koul To: "Yang, Shunyong" Cc: "dan.j.williams@intel.com" , "dmaengine@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: Re: [PATCH 1/2] dmaengine: dmatest: change symbolic permissions to octal values Message-ID: <20180130100444.GU18649@localhost> References: <1516617881-30044-1-git-send-email-shunyong.yang@hxt-semitech.com> <20180129044801.GL18649@localhost> <1517207190.2845.6.camel@hxt-semitech.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1517207190.2845.6.camel@hxt-semitech.com> 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 On Mon, Jan 29, 2018 at 06:26:40AM +0000, Yang, Shunyong wrote: > Hi, Vinod > > On Mon, 2018-01-29 at 10:18 +0530, Vinod Koul wrote: > > On Mon, Jan 22, 2018 at 06:44:40PM +0800, Yang Shunyong wrote: > > > > > > Current coding style prefers octal permissions values. This patch > > > changes symbolic permissions to octal values. > > Is this preference documented anywhere? > > > > When using symbolic permissions like "S_IRUGO | S_IWUSR". The > checkpatch.pl will output some warnings to suggest to use octal values. > I quote following lines from checkpatch.pl, > > # check for uses of S_ that could be octal for readability > ... > if (WARN("SYMBOLIC_PERMS", >                                  "Symbolic permissions '$oval' are not > preferred. Consider using octal permissions '$octal'.\n" . $herecurr) > && It is a warning and a preference. And I don't prefer changing code for no benefit. Sorry but I am not considering this change -- ~Vinod