This patch that fixes coding style problem. Please note: the checkpatch.pl will be reports error: >ERROR: Use 4 digit octal (0777) not decimal permissions >#320: FILE: drivers/staging/iio/meter/ade7759.c:320: >+static IIO_DEV_ATTR_CH_OFF(1, >+ 0644, >+ ade7759_read_8bit, >+ ade7759_write_8bit, >+ ADE7759_CH1OS); >ERROR: Use 4 digit octal (0777) not decimal permissions >#325: FILE: drivers/staging/iio/meter/ade7759.c:325: >+static IIO_DEV_ATTR_CH_OFF(2, >+ 0644, >+ ade7759_read_8bit, >+ ade7759_write_8bit, >+ ADE7759_CH2OS); Because the script checks the first parameter as permissions in function like "IIO_DEV_ATTR_[A-Z_]+" and the IIO_DEV_ATTR_CH_OFF permissions parameter is second. So, I think this is a problem of checkpatch.pl. version 4: - fixes the IIO_DEV_ATTR_CH_OFF version 3: - update the title and patch scription - divide the patch into two parts version 2: - corrected spelling error Chen Guanqiao (1): iio:meter:ade7759: Removing use of deprecated macros (S_IRUGO, S_IWUSR) drivers/staging/iio/meter/ade7759.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.11.0 -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html