Messages in this thread | | | From | Andy Shevchenko <> | Date | Tue, 4 May 2021 16:09:51 +0300 | Subject | Re: [PATCH] platform/x86: samsung-laptop: set debugfs blobs to read only |
| |
On Tue, May 4, 2021 at 3:51 PM Wolfram Sang <wsa+renesas@sang-engineering.com> wrote: > > > Those blobs can only be read. So, don't confuse users with 'writable' > > > flags. > > > > Wouldn't it make sense at the same time to convert ugly namings to > > octal permissions? > > I think this is out-of-scpoe for this patch. While I also use octals > personally, I don't feel strongly about it and let others use what they > prefer.
Linus has (had?) a strong opinion about it. My point is to reduce the churn since two patches fixing the very same parameter in a row seems too much to me. https://lwn.net/Articles/696227/
-- With Best Regards, Andy Shevchenko
| |