[M] Change in coreboot[main]: soc/mediatek/mt8196: Add booker driver

[Dehui Sun (Code Review)]

Attention is currently required from: Hung-Te Lin, Jarried Lin, Yidi Lin, Yu-Ping Wu.

Dehui Sun 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 6:

(3 comments)

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

https://review.coreboot.org/c/coreboot/+/85362/comment/5d2af5de_40f30702?usp... :
PS1, Line 23: 	"ldr	x0, =0x0a450a00\n"
            : 	"ldr	x1, [x0]\n"
            : 	"ldr	x2, =0x8\n"
            : 	"bic	x1, x1, x2\n"
            : 	"str	x1, [x0]\n"
            : 
            : 	"ldr	x0, =0x0ac50a00\n"
            : 	"ldr	x1, [x0]\n"
            : 	"ldr	x2, =0x8\n"
            : 	"bic	x1, x1, x2\n"
            : 	"str	x1, [x0]\n"
            : 
            : 	"ldr	x0, =0x0b450a00\n"
            : 	"ldr	x1, [x0]\n"
            : 	"ldr	x2, =0x8\n"
            : 	"bic	x1, x1, x2\n"
            : 	"str	x1, [x0]\n"
            : 
            : 	"ldr	x0, =0x0bc50a00\n"
            : 	"ldr	x1, [x0]\n"
            : 	"ldr	x2, =0x8\n"
            : 	"bic	x1, x1, x2\n"
            : 	"str	x1, [x0]\n"
> Thanks. Using `clrbits32` would be clear and readable.

Okay, it's better to use clrbits32().

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

https://review.coreboot.org/c/coreboot/+/85362/comment/99f88e23_446b29d1?usp... :
PS5, Line 56: __func__
> Align with BIOS_DEBUG.

Okay

File src/soc/mediatek/mt8196/include/soc/booker.h:

https://review.coreboot.org/c/coreboot/+/85362/comment/76b4a35b_c37551d9?usp... :
PS5, Line 6: extern
> remove

Okay, This can be modified in the next release.