No one checks the return value of debugfs_create_u64(), as it's not needed, so make the return value void, so that no one tries to do so in the future.
Showing the changes to include/linux (1 of 3 files). Show all
Unified · Split
| @@ -103,8 +103,8 @@ void debugfs_create_u16(const char *name, umode_t mode, struct dentry *parent, · 102 unmodified lines | ||
| 103 | 103 | u16 *value); |
| 104 | 104 | struct dentry *debugfs_create_u32(const char *name, umode_t mode, |
| 105 | 105 | struct dentry *parent, u32 *value); |
| 106 | −struct dentry *debugfs_create_u64(const char *name, umode_t mode, | |
| 107 | − struct dentry *parent, u64 *value); | |
| 106 | +void debugfs_create_u64(const char *name, umode_t mode, struct dentry *parent, | |
| 107 | + u64 *value); | |
| 108 | 108 | struct dentry *debugfs_create_ulong(const char *name, umode_t mode, |
| 109 | 109 | struct dentry *parent, unsigned long *value); |
| 110 | 110 | struct dentry *debugfs_create_x8(const char *name, umode_t mode, |
| @@ -257,12 +257,8 @@ static inline struct dentry *debugfs_create_u32(const char *name, umode_t mode, · 146 unmodified lines | ||
| 257 | 257 | return ERR_PTR(-ENODEV); |
| 258 | 258 | } |
| 259 | 259 | |
| 260 | −static inline struct dentry *debugfs_create_u64(const char *name, umode_t mode, | |
| 261 | − struct dentry *parent, | |
| 262 | − u64 *value) | |
| 263 | −{ | |
| 264 | − return ERR_PTR(-ENODEV); | |
| 265 | −} | |
| 260 | +static inline void debugfs_create_u64(const char *name, umode_t mode, | |
| 261 | + struct dentry *parent, u64 *value) { } | |
| 266 | 262 | |
| 267 | 263 | static inline struct dentry *debugfs_create_ulong(const char *name, |
| 268 | 264 | umode_t mode, |