[Linux-kernel-mentees] [PATCH] leds: convert symbolic permission bit macros to octal
dwaipayanray1 at gmail.com
Sat Dec 12 18:10:44 UTC 2020
On Sat, Dec 12, 2020 at 11:19 PM Greg KH <gregkh at linuxfoundation.org> wrote:
> On Sat, Dec 12, 2020 at 09:05:17PM +0530, Dwaipayan Ray wrote:
> > Symbolic macro names are hard to understand and should not be
> > used for permission bits.
> > Convert all bad symbolic permission bit macro uses in led to just use
> > the octal numbers.
> > Following macros were replaced:
> > S_IRUGO => 0444
> > S_IWUSR => 0200
> > S_IRUGO | S_IWUSR => 0644
> > Link: https://lore.kernel.org/lkml/CA+55aFw5v23T-zvDZp-MmD_EYxF8WbafwwB59934FV7g21uMGQ@mail.gmail.com/
> > Signed-off-by: Dwaipayan Ray <dwaipayanray1 at gmail.com>
> > ---
> > drivers/leds/leds-blinkm.c | 8 ++++----
> > drivers/leds/leds-lm355x.c | 2 +-
> > drivers/leds/leds-lm3642.c | 4 ++--
> > drivers/leds/leds-lp55xx-common.h | 6 +++---
> > 4 files changed, 10 insertions(+), 10 deletions(-)
> > diff --git a/drivers/leds/leds-blinkm.c b/drivers/leds/leds-blinkm.c
> > index e11fe1788242..6b9b13f58d8a 100644
> > --- a/drivers/leds/leds-blinkm.c
> > +++ b/drivers/leds/leds-blinkm.c
> > @@ -209,7 +209,7 @@ static ssize_t store_red(struct device *dev, struct device_attribute *attr,
> > return count;
> > }
> > -static DEVICE_ATTR(red, S_IRUGO | S_IWUSR, show_red, store_red);
> > +static DEVICE_ATTR(red, 0644, show_red, store_red);
> Why not use DEVICE_ATTR_RW() instead? THat would handle all of this
> automatically for you, which is why they are recommended to be used
> greg k-h
Thanks Greg, I will do that.
Seems like the led drivers are still using a lot of those old constructs.
leds$ git grep -P "DEVICE_ATTR\(" | wc -l
Will correct them all.
More information about the Linux-kernel-mentees