1 comment:
File src/soc/mediatek/mt8183/include/soc/emi.h:
DRAMC_PARAM_SOURCE_SDRAM_CONFIG = 0,
DRAMC_PARAM_SOURCE_FLASH,
I've thought about it, but then we would have to deal with the _invalid_ case every time. […]
Since you are declaring this as u16, you already need to consider any unexpected values.
In fact I think we don't need to care about invalid case every time, it should be fine to only handle that in where the param is loaded, and the entry of some functions.
And it helps to make sure we won't fail when adding new sources, if we do a switch-case (and default: assert) when different logic is needed.
To view, visit change 35164. To unsubscribe, or for help writing mail filters, visit settings.