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

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Crystal Guo, Hung-Te Lin, Jarried Lin, Paul Menzel, Yidi Lin.

Yu-Ping Wu 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:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/85124/comment/80e70008_15f4b674?usp... :
PS17, Line 14: mc->num_entries / CHANNEL_MAX
> Is it like this? […]

Yes.

Alternatively, we can use `RANK_MAX` directly and remove `rank_count`.

```
for (i = 0; i < RANK_MAX; i++) {
    if (rank_size[i] == 0)
        continue;
    rank_size_sum += rank_size[i];
    bootmem_add_range(...);
}
```

https://review.coreboot.org/c/coreboot/+/85124/comment/6cfb9709_60d973cd?usp... :
PS17, Line 24:  
`* (e->channel_io_width / e->io_width)`

Also see CB:85312.