[-next PATCH 1/4] sysfs.h: Use octal permissions

2017-12-19 Thread Joe Perches
Use the more common and preferred octal directly.

Signed-off-by: Joe Perches 
---
 include/linux/sysfs.h | 14 ++
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/include/linux/sysfs.h b/include/linux/sysfs.h
index 40839c02d28c..b8bfdc173ec0 100644
--- a/include/linux/sysfs.h
+++ b/include/linux/sysfs.h
@@ -113,7 +113,7 @@ struct attribute_group {
 }
 
 #define __ATTR_RO(_name) { \
-   .attr   = { .name = __stringify(_name), .mode = S_IRUGO },  \
+   .attr   = { .name = __stringify(_name), .mode = 0444 }, \
.show   = _name##_show, \
 }
 
@@ -124,12 +124,11 @@ struct attribute_group {
 }
 
 #define __ATTR_WO(_name) { \
-   .attr   = { .name = __stringify(_name), .mode = S_IWUSR },  \
+   .attr   = { .name = __stringify(_name), .mode = 0200 }, \
.store  = _name##_store,\
 }
 
-#define __ATTR_RW(_name) __ATTR(_name, (S_IWUSR | S_IRUGO),\
-_name##_show, _name##_store)
+#define __ATTR_RW(_name) __ATTR(_name, 0644, _name##_show, _name##_store)
 
 #define __ATTR_NULL { .attr = { .name = NULL } }
 
@@ -192,14 +191,13 @@ struct bin_attribute {
 }
 
 #define __BIN_ATTR_RO(_name, _size) {  \
-   .attr   = { .name = __stringify(_name), .mode = S_IRUGO },  \
+   .attr   = { .name = __stringify(_name), .mode = 0444 }, \
.read   = _name##_read, \
.size   = _size,\
 }
 
-#define __BIN_ATTR_RW(_name, _size) __BIN_ATTR(_name,  \
-  (S_IWUSR | S_IRUGO), _name##_read,   \
-  _name##_write, _size)
+#define __BIN_ATTR_RW(_name, _size)\
+   __BIN_ATTR(_name, 0644, _name##_read, _name##_write, _size)
 
 #define __BIN_ATTR_NULL __ATTR_NULL
 
-- 
2.15.0



[-next PATCH 1/4] sysfs.h: Use octal permissions

2017-12-19 Thread Joe Perches
Use the more common and preferred octal directly.

Signed-off-by: Joe Perches 
---
 include/linux/sysfs.h | 14 ++
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/include/linux/sysfs.h b/include/linux/sysfs.h
index 40839c02d28c..b8bfdc173ec0 100644
--- a/include/linux/sysfs.h
+++ b/include/linux/sysfs.h
@@ -113,7 +113,7 @@ struct attribute_group {
 }
 
 #define __ATTR_RO(_name) { \
-   .attr   = { .name = __stringify(_name), .mode = S_IRUGO },  \
+   .attr   = { .name = __stringify(_name), .mode = 0444 }, \
.show   = _name##_show, \
 }
 
@@ -124,12 +124,11 @@ struct attribute_group {
 }
 
 #define __ATTR_WO(_name) { \
-   .attr   = { .name = __stringify(_name), .mode = S_IWUSR },  \
+   .attr   = { .name = __stringify(_name), .mode = 0200 }, \
.store  = _name##_store,\
 }
 
-#define __ATTR_RW(_name) __ATTR(_name, (S_IWUSR | S_IRUGO),\
-_name##_show, _name##_store)
+#define __ATTR_RW(_name) __ATTR(_name, 0644, _name##_show, _name##_store)
 
 #define __ATTR_NULL { .attr = { .name = NULL } }
 
@@ -192,14 +191,13 @@ struct bin_attribute {
 }
 
 #define __BIN_ATTR_RO(_name, _size) {  \
-   .attr   = { .name = __stringify(_name), .mode = S_IRUGO },  \
+   .attr   = { .name = __stringify(_name), .mode = 0444 }, \
.read   = _name##_read, \
.size   = _size,\
 }
 
-#define __BIN_ATTR_RW(_name, _size) __BIN_ATTR(_name,  \
-  (S_IWUSR | S_IRUGO), _name##_read,   \
-  _name##_write, _size)
+#define __BIN_ATTR_RW(_name, _size)\
+   __BIN_ATTR(_name, 0644, _name##_read, _name##_write, _size)
 
 #define __BIN_ATTR_NULL __ATTR_NULL
 
-- 
2.15.0