Change in coreboot[master]: soc/intel/common: pmclib: make use of the new ETR address API

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36570 )

Change subject: soc/intel/common: pmclib: make use of the new ETR address API
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36570/4/src/soc/intel/common/block/... 
File src/soc/intel/common/block/pmc/pmclib.c:

https://review.coreboot.org/c/coreboot/+/36570/4/src/soc/intel/common/block/... 
PS4, Line 432: void pmc_global_reset_lock(void)
             : {
             : 	uint32_t *etr = soc_read_pmc_etr_addr();
             : 	uint32_t reg;
             : 
             : 	reg = read32(etr);
             : 	reg = (reg & ~CF9_GLB_RST) | CF9_LOCK;
             : 	write32(etr, reg);
             : }
Is it really worth it to have a function and callback for something that can be written down in 1 or 2 lines of code?