[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Fix IPU and Vtd config

[Wonkyu Kim (Code Review)]

Wonkyu Kim has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44627 )

Change subject: soc/intel/tigerlake: Fix IPU and Vtd config
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44627/3/src/soc/intel/tigerlake/rom... 
File src/soc/intel/tigerlake/romstage/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/44627/3/src/soc/intel/tigerlake/rom... 
PS3, Line 176: 		if (m_cfg->InternalGfx) {
             : 			m_cfg->VtdIgdEnable = 0x1;
             : 			m_cfg->VtdBaseAddress[0] = GFXVT_BASE_ADDRESS;
             : 		}
             : 
             : 		if (m_cfg->SaIpuEnable) {
             : 			m_cfg->VtdIpuEnable = 0x1;
             : 			m_cfg->VtdBaseAddress[1] = IPUVT_BASE_ADDRESS;
             : 		}
This is minor but can we move this code up  after line 172?
Then we have sequential code for VtdBaseAddress array.