[L] Change in coreboot[main]: soc/intel/common/systemagent_server: Add server platform system agent

[Lean Sheng Tan (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Felix Held, Felix Singer, Jérémy Compostella, Paul Menzel, [e-mail].

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

Change subject: soc/intel/common/systemagent_server: Add server platform system agent
......................................................................

Patch Set 29:

(3 comments)

Patchset:

PS19:
> I've implemented a basic version of server system agent, you can review now.

Any response @[e-mail]? If not will proceed to resolve this comment by this week, thanks :)

Commit Message:

https://review.coreboot.org/c/coreboot/+/83318/comment/161b4d47_e40309f7?usp... :
PS13, Line 9: System agent in Intel common block (1) assumes TOLUD and TOUUD
            : registers hold the max available address plus 1, but on some SoC like
            : Snow Ridge, it holds the max available address; (2) aligns TOLUD, TOUUD
            : and TSEG registers to 1 MiB default, but some SoC may have different
            : alignments. This patch add a new weak function
            : `soc_systemagent_fixup_address()` to improve it.
> Now I'm using a new Kconfig item SA_FIXUP_ADDRESS to select if SoC need to fixup the address base an […]

Acknowledged

Commit Message:

https://review.coreboot.org/c/coreboot/+/83318/comment/31782979_d059ea3d?usp... :
PS19, Line 10: but on some SoC like
             : Snow Ridge, it holds the max available address
> No, and there's no reason to do run-time detection: the problems arise from trying to reuse common c […]

Acknowledged