[coreboot-gerrit] [S] Change in coreboot[main]: soc/mediatek/mt8196: Add booker driver

[Jarried Lin (Code Review)]

Attention is currently required from: Dehui Sun, Hung-Te Lin, Paul Menzel, Yidi Lin, Yu-Ping Wu.

Jarried Lin has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/85362?usp=email )

Change subject: soc/mediatek/mt8196: Add booker driver
......................................................................

Patch Set 11:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/85362/comment/d0a41f5f_2609382d?usp... :
PS10, Line 10: e(
> Okay, This will be modified in the next release.

Done

https://review.coreboot.org/c/coreboot/+/85362/comment/4d4faea0_2eb0d8cc?usp... :
PS10, Line 16:  set bit[3]
> Okay, This will be modified in the next release.

Done

File src/soc/mediatek/mt8196/booker.c:

https://review.coreboot.org/c/coreboot/+/85362/comment/60072edc_08017bf8?usp... :
PS10, Line 11: OFF_REG_POR_SBSX_CFG_CTL
> Okay, This will be modified in the next release.

Done

https://review.coreboot.org/c/coreboot/+/85362/comment/aa28e640_b5c82797?usp... :
PS10, Line 25: 64p
> NO. The register por_sbsx_cfg_ctl is 64bits width by checking CI-700 TRM.

Done