[XS] Change in coreboot[main]: soc/mediatek/mt8188: devapc: Allow APU to access BND_NORTH_APB2_S

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79348?usp=email )

(

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )Change subject: soc/mediatek/mt8188: devapc: Allow APU to access BND_NORTH_APB2_S
......................................................................

soc/mediatek/mt8188: devapc: Allow APU to access BND_NORTH_APB2_S

Update BND_NORTH_APB2_S's domain 5 permission to allow the access from
APU. The APU requires certain information saved in BND_NORTH_APB2_S for
voltage tuning. If this information cannot be retrieved, the APU may
operate at a high frequency with low voltage. Consequently, the APU may
not function as expected.

Change-Id: I967b138dc5517e54da7fbf94b9e502e478c991b5
Signed-off-by: Nina Wu 
Signed-off-by: Jason Chen 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79348
Reviewed-by: Yu-Ping Wu 
Reviewed-by: Yidi Lin 
Tested-by: build bot (Jenkins) 
---
M src/soc/mediatek/mt8188/devapc.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Yidi Lin: Looks good to me, approved
  Yu-Ping Wu: Looks good to me, approved
  build bot (Jenkins): Verified
  jason-ch chen: Looks good to me, but someone else must approve

diff --git a/src/soc/mediatek/mt8188/devapc.c b/src/soc/mediatek/mt8188/devapc.c
index 03e0dfd..6e47f705 100644
--- a/src/soc/mediatek/mt8188/devapc.c
+++ b/src/soc/mediatek/mt8188/devapc.c
@@ -1163,7 +1163,8 @@
    DAPC_PERI2_AO_SYS0_ATTR("BND_NORTH_APB1_S",
    			NO_PROTECTION, FORBIDDEN15),
    DAPC_PERI2_AO_SYS0_ATTR("BND_NORTH_APB2_S",
-				NO_PROTECTION, FORBIDDEN13, NO_PROTECTION, FORBIDDEN),
+				NO_PROTECTION, FORBIDDEN4, NO_PROTECTION, FORBIDDEN8,
+				NO_PROTECTION, FORBIDDEN),
    DAPC_PERI2_AO_SYS0_ATTR("BND_NORTH_APB3_S",
    			NO_PROTECTION, FORBIDDEN15),
    DAPC_PERI2_AO_SYS0_ATTR("BND_NORTH_APB4_S",