Fixed the following checkpatch warning:

    WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider
    using octal permissions '0444'.

Signed-off-by: Enric Balletbo i Serra <[email protected]>
Suggested-by: Andy Shevchenko <[email protected]>
Reviewed-by: Andy Shevchenko <[email protected]>
---

Changes in v4: None
Changes in v3:
- [4/6] Add Reviewed-by Andy Shevchenko

Changes in v2:
- [4/6] Use octal permissions.

 drivers/platform/chrome/cros_ec_debugfs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/platform/chrome/cros_ec_debugfs.c 
b/drivers/platform/chrome/cros_ec_debugfs.c
index 0e88e18362c1..539403022568 100644
--- a/drivers/platform/chrome/cros_ec_debugfs.c
+++ b/drivers/platform/chrome/cros_ec_debugfs.c
@@ -288,7 +288,7 @@ static int cros_ec_create_console_log(struct 
cros_ec_debugfs *debug_info)
        init_waitqueue_head(&debug_info->log_wq);
 
        if (!debugfs_create_file("console_log",
-                                S_IFREG | S_IRUGO,
+                                S_IFREG | 0444,
                                 debug_info->dir,
                                 debug_info,
                                 &cros_ec_console_log_fops))
@@ -341,7 +341,7 @@ static int cros_ec_create_panicinfo(struct cros_ec_debugfs 
*debug_info)
        debug_info->panicinfo_blob.size = ret;
 
        if (!debugfs_create_blob("panicinfo",
-                                S_IFREG | S_IRUGO,
+                                S_IFREG | 0444,
                                 debug_info->dir,
                                 &debug_info->panicinfo_blob)) {
                ret = -ENOMEM;
-- 
2.16.2

Reply via email to