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

From: Michael Ellerman <[email-masked]>
Date: Sun, 30 Jun 2019 18:37:26 +1000 (AEST)

On Wed, 2019-06-12 at 15:54:18 UTC, Greg Kroah-Hartman 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 <[email-masked]>
> Cc: Andrew Donnellan <[email-masked]>
> Cc: Arnd Bergmann <[email-masked]>
> Cc: [email-masked]
> Signed-off-by: Greg Kroah-Hartman <[email-masked]>
> Reviewed-by: Arnd Bergmann <[email-masked]>
> Reviewed-by: Frederic Barrat <[email-masked]>
> Acked-by: Andrew Donnellan <[email-masked]>

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/1b7de1df997a57e3e144bdb31ceaee9c16d6d284

cheers