Change in coreboot[master]: soc/amd/common/block/data_fabric: fix data_fabric_write32 broadcast case

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/50826 )

Change subject: soc/amd/common/block/data_fabric: fix data_fabric_write32 broadcast case
......................................................................

soc/amd/common/block/data_fabric: fix data_fabric_write32 broadcast case

Calling data_fabric_write32 with BROADCAST_FABRIC_ID as instance_id
would have caused an infinite recursion, so call the right function
data_fabric_broadcast_write32 for that case instead.

Signed-off-by: Felix Held 
Change-Id: If7f0a80f0430e8bfb29ee510ef86c278e3a42063
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50826
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
Reviewed-by: Raul Rangel 
---
M src/soc/amd/common/block/data_fabric/data_fabric_helper.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Raul Rangel: Looks good to me, approved
  Angel Pons: Looks good to me, approved

diff --git a/src/soc/amd/common/block/data_fabric/data_fabric_helper.c b/src/soc/amd/common/block/data_fabric/data_fabric_helper.c
index 7bbdc7f..27fc03d 100644
--- a/src/soc/amd/common/block/data_fabric/data_fabric_helper.c
+++ b/src/soc/amd/common/block/data_fabric/data_fabric_helper.c
@@ -34,7 +34,7 @@
 void data_fabric_write32(uint8_t function, uint16_t reg, uint8_t instance_id, uint32_t data)
 {
    if (instance_id == BROADCAST_FABRIC_ID) {
-		data_fabric_write32(function, reg, BROADCAST_FABRIC_ID, data);
+		data_fabric_broadcast_write32(function, reg, data);
    	return;
    }