Re: [PATCH 05/11] touchscreen: elants: Use octal permissions

From: Greg Kroah-Hartman
Date: Sun Jul 22 2018 - 05:40:12 EST


On Sun, Jul 22, 2018 at 12:39:45AM +0530, dev-harsh1998 wrote:
> WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'.
> +static DEVICE_ATTR(iap_mode, S_IRUGO, show_iap_mode, NULL);
>
> WARNING: Symbolic permissions 'S_IWUSR' are not preferred. Consider using octal permissions '0200'.
> +static DEVICE_ATTR(update_fw, S_IWUSR, NULL, write_update_fw)
>
> WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'.
> + .dattr = __ATTR(_field, S_IRUGO, \
>
> Signed-off-by: dev-harsh1998 <harshitjain6751@xxxxxxxxx>

You need to use your real name when creating Linux kernel patches.

thanks,

greg k-h