From ba363d3f185c30c51069cff711d8be8dccb3545f Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Fri, 23 May 2014 05:58:27 +1000 Subject: northbridge/amd/amdmct: Superfluous parenthesis in if-statements Remove superfluous parenthesis found in some if-statements, spotted by Clang. Change-Id: I98d2bf6b408caf320c5bcc8adb23d621b182976b Signed-off-by: Edward O'Callaghan Reviewed-on: http://review.coreboot.org/5817 Tested-by: build bot (Jenkins) Reviewed-by: Marc Jones --- src/northbridge/amd/amdmct/mct/mct_d.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/northbridge/amd/amdmct/mct') diff --git a/src/northbridge/amd/amdmct/mct/mct_d.c b/src/northbridge/amd/amdmct/mct/mct_d.c index 47b89b46d5..66eb88a7e9 100644 --- a/src/northbridge/amd/amdmct/mct/mct_d.c +++ b/src/northbridge/amd/amdmct/mct/mct_d.c @@ -1532,27 +1532,27 @@ static u8 AutoConfig_D(struct MCTStatStruc *pMCTstat, DramConfigLo |= 1 << 4; /* 75 Ohms ODT */ if (mctGet_NVbits(NV_MAX_DIMMS) == 8) { if (pDCTstat->Speed == 3) { - if ((pDCTstat->MAdimms[dct] == 4)) + if (pDCTstat->MAdimms[dct] == 4) DramConfigLo |= 1 << 5; /* 50 Ohms ODT */ } else if (pDCTstat->Speed == 4){ - if ((pDCTstat->MAdimms[dct] != 1)) + if (pDCTstat->MAdimms[dct] != 1) DramConfigLo |= 1 << 5; /* 50 Ohms ODT */ } } else { // FIXME: Skip for Ax versions - if ((pDCTstat->MAdimms[dct] == 4)) { + if (pDCTstat->MAdimms[dct] == 4) { if ( pDCTstat->DimmQRPresent != 0) { if ((pDCTstat->Speed == 3) || (pDCTstat->Speed == 4)) { DramConfigLo |= 1 << 5; /* 50 Ohms ODT */ } - } else if ((pDCTstat->MAdimms[dct] == 4)) { + } else if (pDCTstat->MAdimms[dct] == 4) { if (pDCTstat->Speed == 4) { if ( pDCTstat->DimmQRPresent != 0) { DramConfigLo |= 1 << 5; /* 50 Ohms ODT */ } } } - } else if ((pDCTstat->MAdimms[dct] == 2)) { + } else if (pDCTstat->MAdimms[dct] == 2) { DramConfigLo |= 1 << 5; /* 50 Ohms ODT */ } -- cgit v1.2.3