From: "khand...@linux.vnet.ibm.com" <khand...@linux.vnet.ibm.com>
Value of 'valid' is zero when 'esid' is zero and it does not matter when 'esid' is non-zero. Hence the variable 'value' can be dropped from the conditional statement. This patch does that. Signed-off-by: Anshuman Khandual <khand...@linux.vnet.ibm.com> --- arch/powerpc/xmon/xmon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c index e599259..1798e21 100644 --- a/arch/powerpc/xmon/xmon.c +++ b/arch/powerpc/xmon/xmon.c @@ -2740,7 +2740,7 @@ void dump_segments(void) asm volatile("slbmfee %0,%1" : "=r" (esid) : "r" (i)); asm volatile("slbmfev %0,%1" : "=r" (vsid) : "r" (i)); valid = (esid & SLB_ESID_V); - if (valid | esid | vsid) { + if (esid | vsid) { printf("%02d %016lx %016lx", i, esid, vsid); if (valid) { llp = vsid & SLB_VSID_LLP; -- 2.1.0 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev