[coreboot-gerrit] [XS] Change in coreboot[main]: soc/mediatek/mt8196: Set SPMI-P SCL/SDA SoC PD

[Yidi Lin (Code Review)]

Attention is currently required from: Hope Wang, Hung-Te Lin, Jarried Lin, Yu-Ping Wu.

Yidi Lin has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/85751?usp=email )

Change subject: soc/mediatek/mt8196: Set SPMI-P SCL/SDA SoC PD
......................................................................

Patch Set 6:

(1 comment)

File src/soc/mediatek/mt8196/pmif_spmi.c:

https://review.coreboot.org/c/coreboot/+/85751/comment/5fc42f17_4d1fb059?usp... :
PS1, Line 177: 	if (dev->slvid == SPMI_SLAVE_6) {
             : 		setbits32p(SPMI_SCL_P_PD_ADDR,
             : 			   BIT(SPMI_SCL_P_PD_OFFSET) | BIT(SPMI_SDA_P_PD_OFFSET));
             : 		printk(BIOS_INFO, "%s, Set SoC PD for SPMI-P SCL/SDA, [0x%x] = 0x%x\n",
             : 		       __func__, SPMI_SCL_P_PD_ADDR, read32p(SPMI_SCL_P_PD_ADDR));
             : 	}
> Because the 6316 with slave id 6 is the first PMIC in SPMI-P, just need to set one time.

If so, we can do that in `pmif_spmi_iocfg`. What do you think ?