[coreboot-gerrit] Change in coreboot[master]: soc/intel: Move pch_enable_ioapic() to common code

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45787 )

Change subject: soc/intel: Move pch_enable_ioapic() to common code
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45787/8/src/soc/intel/common/block/... 
File src/soc/intel/common/block/lpc/lpc_lib.c:

https://review.coreboot.org/c/coreboot/+/45787/8/src/soc/intel/common/block/... 
PS8, Line 298: lpc_pch_enable_ioapic
> its due to pch related code that's why existed since BDW  https://github. […]

Another case of "it's always been done that way". It doesn't necessarily make it right though. `ioapic.c` could also work. Also just rename to `pch_ioapic_enable()` ? It does seem to live in the PCH/ICH/southbridge, as the default IOAPIC addresses decode to DMI.