lkml.org 
[lkml]   [2021]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] KVM: arm64: Remove unused TRNG_NO_ENTROPY
On 2021-10-26 22:09, Qian Cai wrote:
> TRNG_NO_ENTROPY was unused when introduced and now. This is also caught
> by
> GCC (W=2):
>
> arch/arm64/kvm/trng.c:17: warning: macro "TRNG_NO_ENTROPY" is not used
> [-Wunused-macros]
> 17 | #define TRNG_NO_ENTROPY ((unsigned long)-3)
>
> Signed-off-by: Qian Cai <quic_qiancai@quicinc.com>
> ---
> arch/arm64/kvm/trng.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/kvm/trng.c b/arch/arm64/kvm/trng.c
> index 99bdd7103c9c..6a46b3ac3970 100644
> --- a/arch/arm64/kvm/trng.c
> +++ b/arch/arm64/kvm/trng.c
> @@ -14,7 +14,6 @@
> #define TRNG_SUCCESS 0UL
> #define TRNG_NOT_SUPPORTED ((unsigned long)-1)
> #define TRNG_INVALID_PARAMETER ((unsigned long)-2)
> -#define TRNG_NO_ENTROPY ((unsigned long)-3)
>
> #define TRNG_MAX_BITS64 192

That's an architectural definition that doesn't cause any problem.
If anything, that's documentation.

We have *tons* of other definitions with no user all over the
arm64 code. They save the exact same purpose.

M.
--
Jazz is not dead. It just smells funny...

\
 
 \ /
  Last update: 2021-10-27 09:47    [W:0.077 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site