Re: [PATCH] memory: emif: Remove bogus debugfs error handling

santosh.shilimkar
2020-08-26T16:52:37Z

On 8/26/20 4:37 AM, Dan Carpenter wrote:
> Callers are generally not supposed to check the return values from
> debugfs functions. Debugfs functions never return NULL so this error
> handling will never trigger. (Historically debugfs functions used to
> return a mix of NULL and error pointers but it was eventually deemed too
> complicated for something which wasn't intended to be used in normal
> situations).
>
> Delete all the error handling.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> ---
Looks good to me !!

Acked-by: Santosh Shilimkar <ssantosh@kernel.org>