On Wed, Jun 26, 2019 at 10:46 AM Bartosz Golaszewski <[email protected]> wrote:
> From: Greg Kroah-Hartman <[email protected]> > > 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. > > Cc: Bamvor Jian Zhang <[email protected]> > Cc: Linus Walleij <[email protected]> > Cc: Bartosz Golaszewski <[email protected]> > Cc: [email protected] > Signed-off-by: Greg Kroah-Hartman <[email protected]> > [Bartosz: removed one more check for debugfs return value] > Signed-off-by: Bartosz Golaszewski <[email protected]> > --- > v1 -> v2: > - fix build warning found by kbuild > - fix build error found by kbuild > > v2 -> v3: > - remove one more unnecessary ifdef Looks good Reviewed-by: Linus Walleij <[email protected]> Will you send me this in a pull request with the rest of the stuff you have queued? Yours, Linus Walleij
