staging:most/hdm-i2c: Replace symbolic permissions with octal permissions
authorZhengyi Shen <shenzhengyi@gmail.com>
Fri, 3 Feb 2017 14:54:06 +0000 (22:54 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 5 Feb 2017 10:41:38 +0000 (11:41 +0100)
WARNING: Symbolic permissions 'S_IRUGO' are not preferred.
Consider using octal permissions '0444'.
This warning was detected by checkpatch.pl for hdm_i2c.c.

Signed-off-by: Zhengyi Shen <shenzhengyi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/hdm-i2c/hdm_i2c.c

index ba0263bb3d12685972669e7d886f4358cb2aac4d..1d5b22927bcd9317032076f121503531f7954603 100644 (file)
@@ -37,7 +37,7 @@ enum { CH_RX, CH_TX, NUM_CHANNELS };
 
 /* IRQ / Polling option */
 static bool polling_req;
-module_param(polling_req, bool, S_IRUGO);
+module_param(polling_req, bool, 0444);
 MODULE_PARM_DESC(polling_req, "Request Polling. Default = 0 (use irq)");
 
 /* Polling Rate */