Change in coreboot[master]: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update

[Angel Pons (Code Review)]

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/49180 )

Change subject: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update
......................................................................

[UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update

This one IOBP register only needs to be updated if its bit 20 is zero.

Change-Id: I7c9e268f9fdf0ae57c6f453f8bce160b177889a4
Signed-off-by: Angel Pons 
---
M src/southbridge/intel/bd82x6x/early_pch.c
1 file changed, 2 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/49180/1

diff --git a/src/southbridge/intel/bd82x6x/early_pch.c b/src/southbridge/intel/bd82x6x/early_pch.c
index e86297b..251ff86 100644
--- a/src/southbridge/intel/bd82x6x/early_pch.c
+++ b/src/southbridge/intel/bd82x6x/early_pch.c
@@ -181,8 +181,8 @@
    pch_iobp_update(0xea0000a9, ~(0x3ff << 3), 0x29f << 3);
    pch_iobp_update(SATA_IOBP_SP1G3IR, ~(0xff << 16 | 0x3f << 8), 0x88 << 16 | 3 << 8);

-	/* TODO: Needs special handling */
-	pch_iobp_update(0xea007f05, 0, 0x00010642);
+	if ((pch_iobp_read(0xea007f05) & 1 << 20) == 0)
+		pch_iobp_update(0xea007f05, ~(7 << 8), 6 << 8);

pch_iobp_update(0xea0040b7, ~0xffff, 0xc91c);
    pch_iobp_update(0xea0040b8, ~0xffff, 0xc91c);

[XS] Change in coreboot[master]: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update

[Stefan Reinauer (Code Review)]

Stefan Reinauer has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/49180?usp=email )

Change subject: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update
......................................................................

Abandoned

[XS] Change in coreboot[master]: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update

[Angel Pons (Code Review)]

Angel Pons has restored this change. ( https://review.coreboot.org/c/coreboot/+/49180?usp=email )

Change subject: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update
......................................................................

Restored

[XS] Change in coreboot[main]: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update

[Angel Pons (Code Review)]

Angel Pons has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/49180?usp=email )

Change subject: [UNTESTED] sb/intel/bd82x6x: Correct conditional IOBP update
......................................................................

Abandoned

No interest in pursuing this