Attention is currently required from: Tim Wawrzynczak, Sridhar Siricilla, Nick Vaccaro, Balaji Manigandan, Krishna P Bhat D.
1 comment:
File util/spd_tools/src/spd_gen/lp5.go:
Patch Set #5, Line 18: DensityPerDieGb int
I'm not familiar with this. I'll look into it.
I assume you're talking about MEM_CH_SEL? E.g. https://review.coreboot.org/plugins/gitiles/coreboot/+/refs/heads/master/src/mainboard/google/brya/variants/baseboard/brya/gpio.c#186
My understanding of this still isn't great, so I could be completely wrong. But I think these are two different uses of the term channel:
1. What we're talking about here is the channels per die. This is a detail of the memory part's internal structure, i.e. which of the part's x8/x16 IO channels are connected to which dies internally.
2. MEM_CH_SEL is talking about the SoC's memory channels. E.g. ADL-P has 2 x64 channels A and B, and I think setting MEM_CH_SEL means that only one of these channels is used.
To view, visit change 58679. To unsubscribe, or for help writing mail filters, visit settings.