[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Replace and-zero with assignment

[Angel Pons (Code Review)]

Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47608 )

Change subject: nb/intel/sandybridge: Replace and-zero with assignment
......................................................................

nb/intel/sandybridge: Replace and-zero with assignment

The intent here is to clear the register, so a simple write will work.

Change-Id: I547805059e911942ac2cac7bd2165af23d926a2b
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47608
Tested-by: build bot (Jenkins) 
Reviewed-by: Arthur Heymans 
---
M src/northbridge/intel/sandybridge/raminit_common.c
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved

diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index 0eb966d..971d692 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -2026,7 +2026,7 @@
    printram("CPF\n");

FOR_ALL_CHANNELS FOR_ALL_POPULATED_RANKS FOR_ALL_LANES {
-		MCHBAR32_AND(IOSAV_By_BW_MASK_ch(channel, lane), 0);
+		MCHBAR32(IOSAV_By_BW_MASK_ch(channel, lane)) = 0;
    }

FOR_ALL_POPULATED_CHANNELS {
@@ -2049,7 +2049,7 @@
    	program_timings(ctrl, channel);

FOR_ALL_CHANNELS FOR_ALL_POPULATED_RANKS FOR_ALL_LANES {
-		MCHBAR32_AND(IOSAV_By_BW_MASK_ch(channel, lane), 0);
+		MCHBAR32(IOSAV_By_BW_MASK_ch(channel, lane)) = 0;
    }
    return 0;
 }