[coreboot-gerrit] [S] Change in coreboot[main]: soc/mt6366: Work around GCC LTO build

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Arthur Heymans, Cliff Huang, Elyes Haouas, Hung-Te Lin, Julius Werner, Jérémy Compostella, Lance Zhao, Tim Wawrzynczak, Yidi Lin.

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

Change subject: soc/mt6366: Work around GCC LTO build
......................................................................

Patch Set 8:

(1 comment)

File src/soc/mediatek/mt8186/mt6366.c:

https://review.coreboot.org/c/coreboot/+/84208/comment/ff98e867_7a3f1266?usp... :
PS8, Line 10: #define NO_BUILDTIME_ASSERT
> I still feel like that's a property of a few specific functions that happen to be grouped in this file

Okay, fine. I didn't search for all assertions in this file. I'm fine with per-statement solution, if we use the workaround macro for both assertions below:

```
assertX(vddq_uv >= 530000);
assertX(vddq_uv <= 680000);
```

and all the other similar assertions in this file. I really don't prefer code like this

```
assert(vddq_uv >= 530000);
assertX(vddq_uv <= 680000);
```

which is inconsistent, confusing, and easy to break (as explained above).