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

From: santosh . shilimkar
Date: Wed Aug 26 2020 - 12:52:47 EST


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@xxxxxxxxxx>
---
Looks good to me !!

Acked-by: Santosh Shilimkar <ssantosh@xxxxxxxxxx>