lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 18 Nov 2020 13:49:48 +0000 From: Lee Jones <lee.jones@...aro.org> To: Soham Biswas <sohambiswas41@...il.com> Cc: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>, thierry.reding@...il.com, linux-pwm@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v2] pwm: core: Use octal permission On Wed, 18 Nov 2020, Soham Biswas wrote: > Sure will do that. Sorry for the inconvenience, I am a bit new to the > process of emailing patches. Should I mark the next patch as v3? Make sure the text you are quoting does above your reply. This is called top-posting and is frowned upon. Yes, please bump the version number - it will make the tooling happy. > On Wed, 18 Nov 2020 at 18:13, Uwe Kleine-König > <u.kleine-koenig@...gutronix.de> wrote: > > > > [added "v2" to the subject, would have been better if you had already > > done that. I don't know if/how this confuses tools like b4 and patchwork] > > > > Hello, > > > > On Wed, Nov 18, 2020 at 04:17:30PM +0530, Soham Biswas wrote: > > > Fixes the following warning generated by checkpatch: > > > > > > drivers/pwm/core.c:1341: WARNING: Symbolic permissions 'S_IRUGO' are > > > not preferred. Consider using octal permissions '0444'. > > > > > > +debugfs_create_file("pwm", S_IFREG | S_IRUGO, NULL, NULL, > > > &pwm_debugfs_fops); > > > > something like: "Permission bits are easier readable in octal than with > > using the symbolic names." in the commit log would be good for those of > > us who missed why this was added to checkpatch. > > > > Best regards > > Uwe > > > > -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
Powered by blists - more mailing lists