2 comments:
File src/soc/intel/cannonlake/cnl_memcfg_init.c:
Patch Set #6, Line 144: No valid way to read mem info.
Just put this in die?
File src/soc/intel/cannonlake/include/soc/cnl_memcfg_init.h:
Patch Set #6, Line 130: * to indicate that the channel is populated. For example,
Are you saying we can just remove channel_empty[] here?
Furquan/Tim/Paul, if so, do we still need a strapping GPIO pin for single/dual channel memory skus?
What Patrick meant is that spd_info is being filled for each slot which means that it already provides information about which channel is populated and which is not. However, in order to fill spd_info, you would still need the strapping GPIO on hatch.
To view, visit change 32513. To unsubscribe, or for help writing mail filters, visit settings.