Commit 1b7de1df authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman Committed by Michael Ellerman

cxl: no need to check return value of debugfs_create functions

When calling debugfs functions, there is no need to ever check the
return value.  The function can work or not, but the code logic should
never do something different based on this.

Because there's no need to check, also make the return value of the
local debugfs_create_io_x64() call void, as no one ever did anything
with the return value (as they did not need to.)

And make the cxl_debugfs_* calls return void as no one was even checking
their return value at all.
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: default avatarArnd Bergmann <arnd@arndb.de>
Reviewed-by: default avatarFrederic Barrat <fbarrat@linux.ibm.com>
Acked-by: default avatarAndrew Donnellan <ajd@linux.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 0b1be03f
...@@ -912,11 +912,11 @@ void cxl_update_dedicated_ivtes_psl8(struct cxl_context *ctx); ...@@ -912,11 +912,11 @@ void cxl_update_dedicated_ivtes_psl8(struct cxl_context *ctx);
#ifdef CONFIG_DEBUG_FS #ifdef CONFIG_DEBUG_FS
int cxl_debugfs_init(void); void cxl_debugfs_init(void);
void cxl_debugfs_exit(void); void cxl_debugfs_exit(void);
int cxl_debugfs_adapter_add(struct cxl *adapter); void cxl_debugfs_adapter_add(struct cxl *adapter);
void cxl_debugfs_adapter_remove(struct cxl *adapter); void cxl_debugfs_adapter_remove(struct cxl *adapter);
int cxl_debugfs_afu_add(struct cxl_afu *afu); void cxl_debugfs_afu_add(struct cxl_afu *afu);
void cxl_debugfs_afu_remove(struct cxl_afu *afu); void cxl_debugfs_afu_remove(struct cxl_afu *afu);
void cxl_debugfs_add_adapter_regs_psl9(struct cxl *adapter, struct dentry *dir); void cxl_debugfs_add_adapter_regs_psl9(struct cxl *adapter, struct dentry *dir);
void cxl_debugfs_add_adapter_regs_psl8(struct cxl *adapter, struct dentry *dir); void cxl_debugfs_add_adapter_regs_psl8(struct cxl *adapter, struct dentry *dir);
...@@ -925,27 +925,24 @@ void cxl_debugfs_add_afu_regs_psl8(struct cxl_afu *afu, struct dentry *dir); ...@@ -925,27 +925,24 @@ void cxl_debugfs_add_afu_regs_psl8(struct cxl_afu *afu, struct dentry *dir);
#else /* CONFIG_DEBUG_FS */ #else /* CONFIG_DEBUG_FS */
static inline int __init cxl_debugfs_init(void) static inline void __init cxl_debugfs_init(void)
{ {
return 0;
} }
static inline void cxl_debugfs_exit(void) static inline void cxl_debugfs_exit(void)
{ {
} }
static inline int cxl_debugfs_adapter_add(struct cxl *adapter) static inline void cxl_debugfs_adapter_add(struct cxl *adapter)
{ {
return 0;
} }
static inline void cxl_debugfs_adapter_remove(struct cxl *adapter) static inline void cxl_debugfs_adapter_remove(struct cxl *adapter)
{ {
} }
static inline int cxl_debugfs_afu_add(struct cxl_afu *afu) static inline void cxl_debugfs_afu_add(struct cxl_afu *afu)
{ {
return 0;
} }
static inline void cxl_debugfs_afu_remove(struct cxl_afu *afu) static inline void cxl_debugfs_afu_remove(struct cxl_afu *afu)
......
...@@ -30,11 +30,11 @@ static int debugfs_io_u64_set(void *data, u64 val) ...@@ -30,11 +30,11 @@ static int debugfs_io_u64_set(void *data, u64 val)
DEFINE_DEBUGFS_ATTRIBUTE(fops_io_x64, debugfs_io_u64_get, debugfs_io_u64_set, DEFINE_DEBUGFS_ATTRIBUTE(fops_io_x64, debugfs_io_u64_get, debugfs_io_u64_set,
"0x%016llx\n"); "0x%016llx\n");
static struct dentry *debugfs_create_io_x64(const char *name, umode_t mode, static void debugfs_create_io_x64(const char *name, umode_t mode,
struct dentry *parent, u64 __iomem *value) struct dentry *parent, u64 __iomem *value)
{ {
return debugfs_create_file_unsafe(name, mode, parent, debugfs_create_file_unsafe(name, mode, parent, (void __force *)value,
(void __force *)value, &fops_io_x64); &fops_io_x64);
} }
void cxl_debugfs_add_adapter_regs_psl9(struct cxl *adapter, struct dentry *dir) void cxl_debugfs_add_adapter_regs_psl9(struct cxl *adapter, struct dentry *dir)
...@@ -58,25 +58,22 @@ void cxl_debugfs_add_adapter_regs_psl8(struct cxl *adapter, struct dentry *dir) ...@@ -58,25 +58,22 @@ void cxl_debugfs_add_adapter_regs_psl8(struct cxl *adapter, struct dentry *dir)
debugfs_create_io_x64("trace", S_IRUSR | S_IWUSR, dir, _cxl_p1_addr(adapter, CXL_PSL_TRACE)); debugfs_create_io_x64("trace", S_IRUSR | S_IWUSR, dir, _cxl_p1_addr(adapter, CXL_PSL_TRACE));
} }
int cxl_debugfs_adapter_add(struct cxl *adapter) void cxl_debugfs_adapter_add(struct cxl *adapter)
{ {
struct dentry *dir; struct dentry *dir;
char buf[32]; char buf[32];
if (!cxl_debugfs) if (!cxl_debugfs)
return -ENODEV; return;
snprintf(buf, 32, "card%i", adapter->adapter_num); snprintf(buf, 32, "card%i", adapter->adapter_num);
dir = debugfs_create_dir(buf, cxl_debugfs); dir = debugfs_create_dir(buf, cxl_debugfs);
if (IS_ERR(dir))
return PTR_ERR(dir);
adapter->debugfs = dir; adapter->debugfs = dir;
debugfs_create_io_x64("err_ivte", S_IRUSR, dir, _cxl_p1_addr(adapter, CXL_PSL_ErrIVTE)); debugfs_create_io_x64("err_ivte", S_IRUSR, dir, _cxl_p1_addr(adapter, CXL_PSL_ErrIVTE));
if (adapter->native->sl_ops->debugfs_add_adapter_regs) if (adapter->native->sl_ops->debugfs_add_adapter_regs)
adapter->native->sl_ops->debugfs_add_adapter_regs(adapter, dir); adapter->native->sl_ops->debugfs_add_adapter_regs(adapter, dir);
return 0;
} }
void cxl_debugfs_adapter_remove(struct cxl *adapter) void cxl_debugfs_adapter_remove(struct cxl *adapter)
...@@ -100,18 +97,16 @@ void cxl_debugfs_add_afu_regs_psl8(struct cxl_afu *afu, struct dentry *dir) ...@@ -100,18 +97,16 @@ void cxl_debugfs_add_afu_regs_psl8(struct cxl_afu *afu, struct dentry *dir)
debugfs_create_io_x64("trace", S_IRUSR | S_IWUSR, dir, _cxl_p1n_addr(afu, CXL_PSL_SLICE_TRACE)); debugfs_create_io_x64("trace", S_IRUSR | S_IWUSR, dir, _cxl_p1n_addr(afu, CXL_PSL_SLICE_TRACE));
} }
int cxl_debugfs_afu_add(struct cxl_afu *afu) void cxl_debugfs_afu_add(struct cxl_afu *afu)
{ {
struct dentry *dir; struct dentry *dir;
char buf[32]; char buf[32];
if (!afu->adapter->debugfs) if (!afu->adapter->debugfs)
return -ENODEV; return;
snprintf(buf, 32, "psl%i.%i", afu->adapter->adapter_num, afu->slice); snprintf(buf, 32, "psl%i.%i", afu->adapter->adapter_num, afu->slice);
dir = debugfs_create_dir(buf, afu->adapter->debugfs); dir = debugfs_create_dir(buf, afu->adapter->debugfs);
if (IS_ERR(dir))
return PTR_ERR(dir);
afu->debugfs = dir; afu->debugfs = dir;
debugfs_create_io_x64("sr", S_IRUSR, dir, _cxl_p1n_addr(afu, CXL_PSL_SR_An)); debugfs_create_io_x64("sr", S_IRUSR, dir, _cxl_p1n_addr(afu, CXL_PSL_SR_An));
...@@ -122,8 +117,6 @@ int cxl_debugfs_afu_add(struct cxl_afu *afu) ...@@ -122,8 +117,6 @@ int cxl_debugfs_afu_add(struct cxl_afu *afu)
if (afu->adapter->native->sl_ops->debugfs_add_afu_regs) if (afu->adapter->native->sl_ops->debugfs_add_afu_regs)
afu->adapter->native->sl_ops->debugfs_add_afu_regs(afu, dir); afu->adapter->native->sl_ops->debugfs_add_afu_regs(afu, dir);
return 0;
} }
void cxl_debugfs_afu_remove(struct cxl_afu *afu) void cxl_debugfs_afu_remove(struct cxl_afu *afu)
...@@ -131,19 +124,12 @@ void cxl_debugfs_afu_remove(struct cxl_afu *afu) ...@@ -131,19 +124,12 @@ void cxl_debugfs_afu_remove(struct cxl_afu *afu)
debugfs_remove_recursive(afu->debugfs); debugfs_remove_recursive(afu->debugfs);
} }
int __init cxl_debugfs_init(void) void __init cxl_debugfs_init(void)
{ {
struct dentry *ent;
if (!cpu_has_feature(CPU_FTR_HVMODE)) if (!cpu_has_feature(CPU_FTR_HVMODE))
return 0; return;
ent = debugfs_create_dir("cxl", NULL);
if (IS_ERR(ent))
return PTR_ERR(ent);
cxl_debugfs = ent;
return 0; cxl_debugfs = debugfs_create_dir("cxl", NULL);
} }
void cxl_debugfs_exit(void) void cxl_debugfs_exit(void)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment