Greg Kroah-Hartman <[email protected]> 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. > > Also, there is no need to store the individual debugfs file names, > especially as the whole directiry is deleted at once, so remove the > unneeded structure entirely. > > Cc: "Breno Leitão" <[email protected]> > Cc: Nayna Jain <[email protected]> > Cc: Paulo Flabiano Smorigo <[email protected]> > Cc: Benjamin Herrenschmidt <[email protected]> > Cc: Paul Mackerras <[email protected]> > Cc: Michael Ellerman <[email protected]> > Cc: Herbert Xu <[email protected]> > Cc: "David S. Miller" <[email protected]> > Cc: [email protected] > Cc: [email protected] > Signed-off-by: Greg Kroah-Hartman <[email protected]> > --- > v2: fixed build error found by kbuild > > drivers/crypto/nx/nx.c | 4 +- > drivers/crypto/nx/nx.h | 12 +----- > drivers/crypto/nx/nx_debugfs.c | 71 +++++++++++----------------------- > 3 files changed, 26 insertions(+), 61 deletions(-)
Patch applied. Thanks. -- Email: Herbert Xu <[email protected]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
