Thread (5 messages) 5 messages, 5 authors, 2019-06-30

Re: [PATCH v2] cxl: no need to check return value of debugfs_create functions

flat view

From: Arnd Bergmann <arnd@arndb.de>
Date: 2019-06-12 18:26:06

On Wed, Jun 12, 2019 at 5:54 PM Greg Kroah-Hartman
[off-list ref] wrote:
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.

Cc: Frederic Barrat <redacted>
Cc: Andrew Donnellan <redacted>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: linuxppc-dev@lists.ozlabs.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help