[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/xeon_sp: Remove PAM unlock operations

[Arthur Heymans (Code Review)]

Attention is currently required from: Shuo Liu.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81349?usp=email )

Change subject: soc/intel/xeon_sp: Remove PAM unlock operations
......................................................................

Patch Set 15:

(1 comment)

File src/soc/intel/xeon_sp/cpx/chip.c:

https://review.coreboot.org/c/coreboot/+/81349/comment/ca561531_393b1387 :
PS15, Line 164: /* Only call this code from socket0! */
              : static void unlock_pam_regions(void)
              : {
              : 	uint32_t pam0123_unlock_dram = 0x33333330;
              : 	uint32_t pam456_unlock_dram = 0x00333333;
              : 	/* Get UBOX(1) for socket0 */
              : 	uint32_t bus1 = socket0_get_ubox_busno(PCU_IIO_STACK);
              : 
              : 	/* Assume socket0 owns PCI segment 0 */
              : 	pci_io_write_config32(PCI_DEV(bus1, SAD_ALL_DEV, SAD_ALL_FUNC),
              : 		SAD_ALL_PAM0123_CSR, pam0123_unlock_dram);
              : 	pci_io_write_config32(PCI_DEV(bus1, SAD_ALL_DEV, SAD_ALL_FUNC),
              : 		SAD_ALL_PAM456_CSR, pam456_unlock_dram);
              : 
              : 	uint32_t reg1 = pci_io_read_config32(PCI_DEV(bus1, SAD_ALL_DEV,
              : 		SAD_ALL_FUNC), SAD_ALL_PAM0123_CSR);
              : 	uint32_t reg2 = pci_io_read_config32(PCI_DEV(bus1, SAD_ALL_DEV,
              : 		SAD_ALL_FUNC), SAD_ALL_PAM456_CSR);
              : 	printk(BIOS_DEBUG, "%s:%s pam0123_csr: 0x%x, pam456_csr: 0x%x\n",
              : 		__FILE__, __func__, reg1, reg2);
              : }
Why is this code duplicated in cpx and skx. It was also valid for SPR-SP but not used. Why not have the function definiton in common code but call it in SOC specific code?