Re: [cryptodev:master 144/146] drivers/crypto/nx/nx.h:153:39: warning: 'struct crypto_aead' declared inside parameter list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 17, 2015 at 05:52:31PM +0800, kbuild test robot wrote:
> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git master
> head:   58268e58e63550e24dee7d6125078ab2a2a22272
> commit: b0d955ba4688fcba8112884931aea1f1e6f50f03 [144/146] crypto: aead - Remove old AEAD interfaces
> config: powerpc-defconfig (attached as .config)
> reproduce:
>   wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>   chmod +x ~/bin/make.cross
>   git checkout b0d955ba4688fcba8112884931aea1f1e6f50f03
>   # save the attached .config to linux build tree
>   make.cross ARCH=powerpc 
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from drivers/crypto/nx/nx_debugfs.c:33:0:
> >> drivers/crypto/nx/nx.h:153:39: warning: 'struct crypto_aead' declared inside parameter list
>     int nx_crypto_ctx_aes_ccm_init(struct crypto_aead *tfm);
>                                           ^
> >> drivers/crypto/nx/nx.h:153:39: warning: its scope is only this definition or declaration, which is probably not what you want
>    drivers/crypto/nx/nx.h:154:39: warning: 'struct crypto_aead' declared inside parameter list
>     int nx_crypto_ctx_aes_gcm_init(struct crypto_aead *tfm);
>                                           ^
>    drivers/crypto/nx/nx.h:161:37: warning: 'struct crypto_aead' declared inside parameter list
>     void nx_crypto_ctx_aead_exit(struct crypto_aead *tfm);
>                                         ^

Oops, I don't know how I missed this warning.

---8<---
Subject: crypto: nx - Add forward declaration for struct crypto_aead

The file nx.h has function prototypes that use struct crypto_aead.
However, as crypto/aead.h is not included we don't have a definition
for it.  This patch adds a forward declaration to fix this.

Reported-by: kbuild test robot <fengguang.wu@xxxxxxxxx>
Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
---

 drivers/crypto/nx/nx.h |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/crypto/nx/nx.h b/drivers/crypto/nx/nx.h
index 591632d..9347878 100644
--- a/drivers/crypto/nx/nx.h
+++ b/drivers/crypto/nx/nx.h
@@ -149,6 +149,8 @@ struct nx_crypto_ctx {
 	} priv;
 };
 
+struct crypto_aead;
+
 /* prototypes */
 int nx_crypto_ctx_aes_ccm_init(struct crypto_aead *tfm);
 int nx_crypto_ctx_aes_gcm_init(struct crypto_aead *tfm);
-- 
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux