[coreboot-gerrit] [XS] Change in coreboot[main]: FIXME remove asserts for mt8186 code

[Arthur Heymans (Code Review)]

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

Arthur Heymans has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/84208?usp=email )

Change subject: FIXME remove asserts for mt8186 code
......................................................................

Patch Set 2:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84208/comment/da2e4783_ce29bef3?usp... :
PS2, Line 7: remove
> Doesn't this patch "add" assertions?

It's commenting out existing ones.

https://review.coreboot.org/c/coreboot/+/84208/comment/4230d473_7e07c2ff?usp... :
PS2, Line 7: FIXME
> Why this tag?

Because I presume the asserts are there for a reason. They are somehow triggered so this means either the assert is wrong or the input is wrong. I don't know the details of this SOC to know which one is the case.