Attention is currently required from: Hung-Te Lin, jason-ch chen, Rex-BC Chen, Yidi Lin.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73375 )
Change subject: soc/mediatek/mt8188: Add bus protection for audio/video mtcmos
......................................................................
Patch Set 2:
(1 comment)
File src/soc/mediatek/mt8188/mtcmos.c:
https://review.coreboot.org/c/coreboot/+/73375/comment/c095067b_fcc8738d
PS2, Line 25: ADSP_PROT_STEP_1_MASK = 0x00100000,
: ADSP_PROT_STEP_2_MASK = 0x00001000,
: AUDIO_PROT_STEP_1_MASK = 0x00000F00,
: AUDIO_PROT_STEP_2_MASK = 0x00600000,
The order isn't consistent. `V*STEP_*` are sorted by decreasing steps.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/73375
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I14aa63c4934073a14552cef64f40657d0197bbe1
Gerrit-Change-Number: 73375
Gerrit-PatchSet: 2
Gerrit-Owner: jason-ch chen
Jason-ch.Chen@mediatek.com
Gerrit-Reviewer: Hung-Te Lin
hungte@chromium.org
Gerrit-Reviewer: Rex-BC Chen
rex-bc.chen@mediatek.com
Gerrit-Reviewer: Yidi Lin
yidilin@google.com
Gerrit-Reviewer: Yu-Ping Wu
yupingso@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Hung-Te Lin
hungte@chromium.org
Gerrit-Attention: jason-ch chen
Jason-ch.Chen@mediatek.com
Gerrit-Attention: Rex-BC Chen
rex-bc.chen@mediatek.com
Gerrit-Attention: Yidi Lin
yidilin@google.com
Gerrit-Comment-Date: Fri, 03 Mar 2023 02:59:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment