[coreboot-gerrit] [S] Change in coreboot[master]: vendorcode/mediatek/mt8192: Fix set but unused variables

[Arthur Heymans (Code Review)]

Attention is currently required from: Xi Chen, Xixi Chen.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74543 )

Change subject: vendorcode/mediatek/mt8192: Fix set but unused variables
......................................................................

Patch Set 2:

(1 comment)

File src/vendorcode/mediatek/mt8192/include/dramc_common.h:

https://review.coreboot.org/c/coreboot/+/74543/comment/1d3090cc_13a15012 
PS2, Line 58: if (0)
> > one question, is this better?
>
> I guess BIOS_NEVER already statically makes it optimized out.

So without the if (0) the binary size does increase, so I think this is a good way to handle it.