[coreboot-gerrit] [S] Change in coreboot[main]: soc/mediatek/mt8196: Reserve DRAM buffers for HW TX TRACKING

[Crystal Guo (Code Review)]

Attention is currently required from: Hung-Te Lin, Jarried Lin, Paul Menzel, Yidi Lin, Yu-Ping Wu.

Crystal Guo has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/85124?usp=email )

Change subject: soc/mediatek/mt8196: Reserve DRAM buffers for HW TX TRACKING
......................................................................

Patch Set 17:

(1 comment)

File src/soc/mediatek/mt8196/dramc_info.c:

https://review.coreboot.org/c/coreboot/+/85124/comment/563bdc2d_1c874263?usp... :
PS17, Line 14: mc->num_entries / CHANNEL_MAX
> It's safer to get the number of ranks from `e->rank`.

Is it like this?
    for (i = 0; i < mc->num_entries; i++) {
    	const struct mem_chip_entry *e = &mc->entries[i];
    	...
    	rank_count = e->rank + 1;
    }