1 comment:
File src/mainboard/google/dedede/variants/waddledoo/memory.c:
Patch Set #25, Line 41: 1, 3, 0, 2, 7, 5, 4, 6
I checked again in schematics the mapping is correct.
Synced up with Aamir offline. My understanding was that the index in the array corresponds to DRAM's DQS ID and the value in the array corresponds to CPU's DQS ID. This is documented that way for all the SoCs like CNL and TGL. Hence I came with that array definition. Also the comment above the array says so.
Aamir mentioned to me that the index in the array corresponds to CPU's DQS ID and the value in the array corresponds to DRAM's DQS ID. If so, then the array definition matches with the schematics. Probably the documentation shared for previous SoCs needs to be fixed?
To view, visit change 39136. To unsubscribe, or for help writing mail filters, visit settings.