Fixed following checkpatch.pl warning
WARNING: Symbolic permissions 'S_IRUGO | S_IWUSR' are not preferred.
Consider using octal permissions '0644'.

Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com>
---
 drivers/staging/media/bcm2048/radio-bcm2048.c | 40 +++++++++++++--------------
 1 file changed, 20 insertions(+), 20 deletions(-)

diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c 
b/drivers/staging/media/bcm2048/radio-bcm2048.c
index ea15cc6..5d2b702 100644
--- a/drivers/staging/media/bcm2048/radio-bcm2048.c
+++ b/drivers/staging/media/bcm2048/radio-bcm2048.c
@@ -2057,51 +2057,51 @@ property_signed_read(fm_rssi, int, "%d")
 DEFINE_SYSFS_PROPERTY(region, unsigned, int, "%u", 0)
 
 static struct device_attribute attrs[] = {
-       __ATTR(power_state, S_IRUGO | S_IWUSR, bcm2048_power_state_read,
+       __ATTR(power_state, 0644, bcm2048_power_state_read,
               bcm2048_power_state_write),
-       __ATTR(mute, S_IRUGO | S_IWUSR, bcm2048_mute_read,
+       __ATTR(mute, 0644, bcm2048_mute_read,
               bcm2048_mute_write),
-       __ATTR(audio_route, S_IRUGO | S_IWUSR, bcm2048_audio_route_read,
+       __ATTR(audio_route, 0644, bcm2048_audio_route_read,
               bcm2048_audio_route_write),
-       __ATTR(dac_output, S_IRUGO | S_IWUSR, bcm2048_dac_output_read,
+       __ATTR(dac_output, 0644, bcm2048_dac_output_read,
               bcm2048_dac_output_write),
-       __ATTR(fm_hi_lo_injection, S_IRUGO | S_IWUSR,
+       __ATTR(fm_hi_lo_injection, 0644,
               bcm2048_fm_hi_lo_injection_read,
               bcm2048_fm_hi_lo_injection_write),
-       __ATTR(fm_frequency, S_IRUGO | S_IWUSR, bcm2048_fm_frequency_read,
+       __ATTR(fm_frequency, 0644, bcm2048_fm_frequency_read,
               bcm2048_fm_frequency_write),
-       __ATTR(fm_af_frequency, S_IRUGO | S_IWUSR,
+       __ATTR(fm_af_frequency, 0644,
               bcm2048_fm_af_frequency_read,
               bcm2048_fm_af_frequency_write),
-       __ATTR(fm_deemphasis, S_IRUGO | S_IWUSR, bcm2048_fm_deemphasis_read,
+       __ATTR(fm_deemphasis, 0644, bcm2048_fm_deemphasis_read,
               bcm2048_fm_deemphasis_write),
-       __ATTR(fm_rds_mask, S_IRUGO | S_IWUSR, bcm2048_fm_rds_mask_read,
+       __ATTR(fm_rds_mask, 0644, bcm2048_fm_rds_mask_read,
               bcm2048_fm_rds_mask_write),
-       __ATTR(fm_best_tune_mode, S_IRUGO | S_IWUSR,
+       __ATTR(fm_best_tune_mode, 0644,
               bcm2048_fm_best_tune_mode_read,
               bcm2048_fm_best_tune_mode_write),
-       __ATTR(fm_search_rssi_threshold, S_IRUGO | S_IWUSR,
+       __ATTR(fm_search_rssi_threshold, 0644,
               bcm2048_fm_search_rssi_threshold_read,
               bcm2048_fm_search_rssi_threshold_write),
-       __ATTR(fm_search_mode_direction, S_IRUGO | S_IWUSR,
+       __ATTR(fm_search_mode_direction, 0644,
               bcm2048_fm_search_mode_direction_read,
               bcm2048_fm_search_mode_direction_write),
-       __ATTR(fm_search_tune_mode, S_IRUGO | S_IWUSR,
+       __ATTR(fm_search_tune_mode, 0644,
               bcm2048_fm_search_tune_mode_read,
               bcm2048_fm_search_tune_mode_write),
-       __ATTR(rds, S_IRUGO | S_IWUSR, bcm2048_rds_read,
+       __ATTR(rds, 0644, bcm2048_rds_read,
               bcm2048_rds_write),
-       __ATTR(rds_b_block_mask, S_IRUGO | S_IWUSR,
+       __ATTR(rds_b_block_mask, 0644,
               bcm2048_rds_b_block_mask_read,
               bcm2048_rds_b_block_mask_write),
-       __ATTR(rds_b_block_match, S_IRUGO | S_IWUSR,
+       __ATTR(rds_b_block_match, 0644,
               bcm2048_rds_b_block_match_read,
               bcm2048_rds_b_block_match_write),
-       __ATTR(rds_pi_mask, S_IRUGO | S_IWUSR, bcm2048_rds_pi_mask_read,
+       __ATTR(rds_pi_mask, 0644, bcm2048_rds_pi_mask_read,
               bcm2048_rds_pi_mask_write),
-       __ATTR(rds_pi_match, S_IRUGO | S_IWUSR, bcm2048_rds_pi_match_read,
+       __ATTR(rds_pi_match, 0644, bcm2048_rds_pi_match_read,
               bcm2048_rds_pi_match_write),
-       __ATTR(rds_wline, S_IRUGO | S_IWUSR, bcm2048_rds_wline_read,
+       __ATTR(rds_wline, 0644, bcm2048_rds_wline_read,
               bcm2048_rds_wline_write),
        __ATTR(rds_pi, S_IRUGO, bcm2048_rds_pi_read, NULL),
        __ATTR(rds_rt, S_IRUGO, bcm2048_rds_rt_read, NULL),
@@ -2115,7 +2115,7 @@ static struct device_attribute attrs[] = {
               bcm2048_fm_carrier_error_read, NULL),
        __ATTR(fm_rssi, S_IRUGO,
               bcm2048_fm_rssi_read, NULL),
-       __ATTR(region, S_IRUGO | S_IWUSR, bcm2048_region_read,
+       __ATTR(region, 0644, bcm2048_region_read,
               bcm2048_region_write),
        __ATTR(rds_data, S_IRUGO, bcm2048_rds_data_read, NULL),
 };
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to