Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35481 )
Change subject: soc/mediatek/mt8183: Clean cached calibration results if mem test fails ......................................................................
Patch Set 47:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35481/45/src/soc/mediatek/mt8183/me... File src/soc/mediatek/mt8183/memory.c:
https://review.coreboot.org/c/coreboot/+/35481/45/src/soc/mediatek/mt8183/me... PS45, Line 160: 0
- If 'dparam' is not cleared now, is there a way to clear the flash?
I think my question is why do we need to clear it? What's the issue if we don't clear?
We're using 'fmap_overwrite_area()' now and I didn't find an API to directly clear fmap. I thought this memset was only for clearing the flash.
After full-k it should rewrite the area with expected value, isn't it?
- If we use a special STATUS to prevent running full-k if it failed previously, those failed devices would always run partial-k (until next version). If the mem test somehow passed, it would be even harder for us to know that full-k failed on those devices, right?
Then what's the alternative? I can't see how clearing dparam would help in this case?