From: Bhavna Sarathy <bnagendr@redhat.com>
Date: Thu, 5 Aug 2010 18:04:35 -0400
Subject: [edac] amd64_edac: fix operator precedence error
Message-id: <20100805181015.6566.75639.sendpatchset@localhost.localdomain>
Patchwork-id: 27423
O-Subject: [RHEL5.6 PATCH 9/10] amd64_edac - Fix operator precedence error
Bugzilla: 568576
RH-Acked-by: Mauro Carvalho Chehab <mchehab@redhat.com>

>From 545d36aff8a830ea6bb6b1f6ad7eb61fe3f5b940 Mon Sep 17 00:00:00 2001
From: Borislav Petkov <borislav.petkov@amd.com>
Date: Wed, 4 Aug 2010 02:47:39 +0200
Subject: [PATCH] amd64_edac: Fix operator precendence error

The bitwise AND is higher prio.

Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>

diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c
index 5da611b..6edbe02 100644
--- a/drivers/edac/amd64_edac.c
+++ b/drivers/edac/amd64_edac.c
@@ -1446,7 +1446,7 @@ static void f10_map_sysaddr_to_csrow(struct mem_ctl_info *mci,
 	 * ganged. Otherwise @chan should already contain the channel at
 	 * this point.
 	 */
-	if (dct_ganging_enabled(pvt) && pvt->nbcfg & K8_NBCFG_CHIPKILL)
+	if (dct_ganging_enabled(pvt) && (pvt->nbcfg & K8_NBCFG_CHIPKILL))
 		chan = get_channel_from_ecc_syndrome(mci, syndrome);
 
 	if (chan >= 0)