| Message ID | 20190822144138.75904-1-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Herbert Xu |
| Headers | show |
| Series | [-next] crypto: aegis128 - Fix -Wunused-const-variable warning | expand |
On Thu, Aug 22, 2019 at 10:41:38PM +0800, YueHaibing wrote: > crypto/aegis.h:27:32: warning: > crypto_aegis_const defined but not used [-Wunused-const-variable=] > > crypto_aegis_const is only used in aegis128-core.c, > just move the definition over there. > > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > crypto/aegis.h | 11 ----------- > crypto/aegis128-core.c | 11 +++++++++++ > 2 files changed, 11 insertions(+), 11 deletions(-) Patch applied. Thanks.
diff --git a/crypto/aegis.h b/crypto/aegis.h index 4d56a85..6920ebe 100644 --- a/crypto/aegis.h +++ b/crypto/aegis.h @@ -24,17 +24,6 @@ union aegis_block { #define AEGIS_BLOCK_ALIGN (__alignof__(union aegis_block)) #define AEGIS_ALIGNED(p) IS_ALIGNED((uintptr_t)p, AEGIS_BLOCK_ALIGN) -static const union aegis_block crypto_aegis_const[2] = { - { .words64 = { - cpu_to_le64(U64_C(0x0d08050302010100)), - cpu_to_le64(U64_C(0x6279e99059372215)), - } }, - { .words64 = { - cpu_to_le64(U64_C(0xf12fc26d55183ddb)), - cpu_to_le64(U64_C(0xdd28b57342311120)), - } }, -}; - static __always_inline void crypto_aegis_block_xor(union aegis_block *dst, const union aegis_block *src) { diff --git a/crypto/aegis128-core.c b/crypto/aegis128-core.c index fa69e99..80e7361 100644 --- a/crypto/aegis128-core.c +++ b/crypto/aegis128-core.c @@ -45,6 +45,17 @@ struct aegis128_ops { static bool have_simd; +static const union aegis_block crypto_aegis_const[2] = { + { .words64 = { + cpu_to_le64(U64_C(0x0d08050302010100)), + cpu_to_le64(U64_C(0x6279e99059372215)), + } }, + { .words64 = { + cpu_to_le64(U64_C(0xf12fc26d55183ddb)), + cpu_to_le64(U64_C(0xdd28b57342311120)), + } }, +}; + static bool aegis128_do_simd(void) { #ifdef CONFIG_CRYPTO_AEGIS128_SIMD
crypto/aegis.h:27:32: warning: crypto_aegis_const defined but not used [-Wunused-const-variable=] crypto_aegis_const is only used in aegis128-core.c, just move the definition over there. Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- crypto/aegis.h | 11 ----------- crypto/aegis128-core.c | 11 +++++++++++ 2 files changed, 11 insertions(+), 11 deletions(-)