[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Don't add memory resource twice

[Marc Jones (Code Review)]

Marc Jones has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47173 )

Change subject: soc/intel/xeon_sp: Don't add memory resource twice
......................................................................

Patch Set 2:

(1 comment)

I updated the commit message. I hope it clarifies the change. The resource map is the same for all cpus. Don't add the same map multiple times.

https://review.coreboot.org/c/coreboot/+/47173/1/src/soc/intel/xeon_sp/uncor... 
File src/soc/intel/xeon_sp/uncore.c:

https://review.coreboot.org/c/coreboot/+/47173/1/src/soc/intel/xeon_sp/uncor... 
PS1, Line 66: 	value |= (uint64_t)pci_read_config32(dev, entry->reg);
> This looks like a hardware access to me.

ack