From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH 05/11] touchscreen: elants: Use octal permissions Date: Mon, 23 Jul 2018 10:18:31 -0700 Message-ID: <20180723171831.GC100814@dtor-ws> References: <20180723131954.30739-1-harshitjain6751@gmail.com> <20180723131954.30739-4-harshitjain6751@gmail.com> <20180723133200.GA1167@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180723133200.GA1167@kroah.com> Sender: linux-kernel-owner@vger.kernel.org To: Greg Kroah-Hartman Cc: dev-harsh1998 , trivial@kernel.org, Simon Budig , Andi Shyti , Luca Ceresoli , Joe Perches , Guenter Roeck , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-input@vger.kernel.org On Mon, Jul 23, 2018 at 03:32:00PM +0200, Greg Kroah-Hartman wrote: > On Mon, Jul 23, 2018 at 06:49:20PM +0530, dev-harsh1998 wrote: > > WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'. > > +static DEVICE_ATTR(iap_mode, S_IRUGO, show_iap_mode, NULL); > > > > WARNING: Symbolic permissions 'S_IWUSR' are not preferred. Consider using octal permissions '0200'. > > +static DEVICE_ATTR(update_fw, S_IWUSR, NULL, write_update_fw) > > > > WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'. > > + .dattr = __ATTR(_field, S_IRUGO, \ > > > > Signed-off-by: Harshit Jain > > This name doesn't match up with the From: line above :( > > Please fix up and try again. dtor@dtor-ws:~/kernel/linux-next$ git grep S_IRU | wc -l 7605 We either need to run a tree-wide script or leave this alone. FWIW I am perfectly fine with either octals or symbolic names so I do not see benefit of doing conversion for code that is not known to be broken. Thanks. -- Dmitry