Jiaxin Yu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32458 )
Change subject: mediatek/mt8183: Init audio related clock ......................................................................
Patch Set 2:
(4 comments)
Hi Hung-Te,please help to review again,thanks.
https://review.coreboot.org/#/c/32458/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32458/1//COMMIT_MSG@9 PS1, Line 9: and
replace with ','
Done
https://review.coreboot.org/#/c/32458/1//COMMIT_MSG@9 PS1, Line 9: hoping
is this a typo or correct name?
Done
https://review.coreboot.org/#/c/32458/1//COMMIT_MSG@10 PS1, Line 10: playback beep sound
audio playback
Done
https://review.coreboot.org/#/c/32458/1/src/soc/mediatek/mt8183/pll.c File src/soc/mediatek/mt8183/pll.c:
https://review.coreboot.org/#/c/32458/1/src/soc/mediatek/mt8183/pll.c@361 PS1, Line 361: setbits_le32
The comment gave 3 (hoping, intbus and infra) but here the settings only do 2 set (or 4 clocks). […]
Done