[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Configure IPU based on devicetree

[Ravishankar Sarawadi (Code Review)]

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

Change subject: soc/intel/tigerlake: Configure IPU based on devicetree
......................................................................

Patch Set 2:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/44627/2/src/soc/intel/tigerlake/rom... 
PS2, Line 171: 		m_cfg->VtdIpuEnable = 0x1;
             : 		m_cfg->VtdBaseAddress[1] = IPUVT_BASE_ADDRESS;
> I think we also need to check IPU status for this.

John can you please confirm if check is needed?