[coreboot-gerrit] Change in coreboot[master]: soc/intel/broadwell/iobp: Log success in `pch_iobp_write()`

[Angel Pons (Code Review)]

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

Change subject: soc/intel/broadwell/iobp: Log success in `pch_iobp_write()`
......................................................................

soc/intel/broadwell/iobp: Log success in `pch_iobp_write()`

This reduces the differences between Lynxpoint and Broadwell.

Change-Id: I759aa98b80c70c5024213bd8795375061bdbbf10
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42622
Reviewed-by: Arthur Heymans 
Reviewed-by: Tim Wawrzynczak 
Reviewed-by: Paul Menzel 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/broadwell/iobp.c
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Paul Menzel: Looks good to me, but someone else must approve
  Arthur Heymans: Looks good to me, approved
  Tim Wawrzynczak: Looks good to me, approved

diff --git a/src/soc/intel/broadwell/iobp.c b/src/soc/intel/broadwell/iobp.c
index 76a3e58..ffa61fb 100644
--- a/src/soc/intel/broadwell/iobp.c
+++ b/src/soc/intel/broadwell/iobp.c
@@ -91,8 +91,12 @@

/* Check for successful transaction */
    status = RCBA16(IOBPS);
-	if (status & IOBPS_TX_MASK)
+	if (status & IOBPS_TX_MASK) {
    	printk(BIOS_ERR, "IOBP: write 0x%08x failed\n", address);
+		return;
+	}
+
+	printk(BIOS_INFO, "IOBP: set 0x%08x to 0x%08x\n", address, data);
 }

void pch_iobp_update(u32 address, u32 andvalue, u32 orvalue)