[M] Change in coreboot[main]: soc/intel/common/systemagent: Improve systemagent

[Felix Held (Code Review)]

Attention is currently required from: Felix Singer, Jérémy Compostella, Shuo Liu, [e-mail].

Felix Held has posted comments on this change by [e-mail]. ( https://review.coreboot.org/c/coreboot/+/83318?usp=email )

Change subject: soc/intel/common/systemagent: Improve systemagent
......................................................................

Patch Set 12:

(1 comment)

File src/soc/intel/common/block/systemagent/systemagent.c:

https://review.coreboot.org/c/coreboot/+/83318/comment/a789092c_e44a187b?usp... :
PS7, Line 308: 		return;
> Removing the if statement causes reserving same address ranges multiple time, I guess it's ok but a  […]

i second that using the if_domain0() function in the if condition would be better here, since that one is used in all other places to do things only on domain0. CB:83643 removed the open-coded checks and used if_domain0 in all those places. also checking for the pci bus number instead of the domain number would add another way of things being done which might make this more confusing