[coreboot-gerrit] Change in coreboot[master]: soc/mediatek: PCI: Assert PERST# at bootblock stage

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Hung-Te Lin, Shelley Chen, Paul Menzel, Angel Pons, Jianjun Wang.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62359 )

Change subject: soc/mediatek: PCI: Assert PERST# at bootblock stage
......................................................................

Patch Set 11:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/62359/comment/650efffc_d7d0421f 
PS1, Line 28:
> Updated, thanks.

Ack

Commit Message:

https://review.coreboot.org/c/coreboot/+/62359/comment/52831687_3f5ae97c 
PS9, Line 14: 100ms
> I meant to say that the `timestamp_get()` function has a __weak definition, which might have to be overriden. However, if `HAVE_MONOTONIC_TIMER` is enabled, it should work.

Right... I misread your previous comment.
> As for sharing data across stages, I think the CBMEM_INIT hook stuff is the only common thing we have.

For that to work we'll also need to introduce a new CBMEM_ID_*. However I'm not sure this case is general enough to be worth a new entry there. So I think I'd prefer the simpler approach here -- to enable COLLECT_TIMESTAMPS.