Greg Kroah-Hartman <gre...@linuxfoundation.org> 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" <lei...@debian.org> > Cc: Nayna Jain <na...@linux.ibm.com> > Cc: Paulo Flabiano Smorigo <pfsmor...@gmail.com> > Cc: Benjamin Herrenschmidt <b...@kernel.crashing.org> > Cc: Paul Mackerras <pau...@samba.org> > Cc: Michael Ellerman <m...@ellerman.id.au> > Cc: Herbert Xu <herb...@gondor.apana.org.au> > Cc: "David S. Miller" <da...@davemloft.net> > Cc: linux-crypto@vger.kernel.org > Cc: linuxppc-...@lists.ozlabs.org > Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org> > --- > 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 <herb...@gondor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
