On Fri, May 01, 2020 at 02:28:12PM +0200, Christophe Leroy wrote: > > -extern int sha256_update(struct sha256_state *sctx, const u8 *input, > > - unsigned int length); > > -extern int sha256_final(struct sha256_state *sctx, u8 *hash); > > +extern void sha256_update(struct sha256_state *sctx, const u8 *data, > > + unsigned int len); > > +extern void sha256_final(struct sha256_state *sctx, u8 *out); > > The 'extern' keywork is useless in a function declaration. It should be > removed, as recommended by 'checkpatch --strict'. Sure I can do this, just keep in mind that a lot of kernel code still uses 'extern' (including the other function declarations in this file) and some people still disagree about the preferred style. That's why this warning hasn't been enabled in 'checkpatch' by default. - Eric
