Don't negate the 'mci.*' operands and use the logical AND operator to simplify the return statement. No functional changes intended. Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxx> --- arch/s390/kernel/nmi.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/s390/kernel/nmi.c b/arch/s390/kernel/nmi.c index 11f33243a23f..cbefd6bde104 100644 --- a/arch/s390/kernel/nmi.c +++ b/arch/s390/kernel/nmi.c @@ -321,9 +321,7 @@ static bool notrace nmi_registers_valid(union mci mci) cr2.reg = get_lowcore()->cregs_save_area[2]; if (cr2.gse && !mci.gs && !test_cpu_flag(CIF_MCCK_GUEST)) return false; - if (!mci.ms || !mci.pm || !mci.ia) - return false; - return true; + return mci.ms && mci.pm && mci.ia; } NOKPROBE_SYMBOL(nmi_registers_valid); -- 2.51.0