[M] Change in coreboot[main]: mb/google/brya/var/xol: Configure power limits by battery status

[SH Kim (Code Review)]

Attention is currently required from: Dinesh Gehlot, Eric Lai, Jamie Chen, Kapil Porwal, Nick Vaccaro, Paul Menzel, Subrata Banik, Sumeet R Pawnikar.

SH Kim has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81614?usp=email )

Change subject: mb/google/brya/var/xol: Configure power limits by battery status
......................................................................

Patch Set 3:

(5 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81614/comment/8b8fc9fd_25725058 :
PS2, Line 10: platform
            : instabilities
> What instabilities occur? Please elaborate? Doesn’t the battery just drain faster?

Done

https://review.coreboot.org/c/coreboot/+/81614/comment/54574e55_50adca65 :
PS2, Line 18: MSR PL2: 55W, MSR PL4: 114W
> Please document the command you used to get these.

I mentioned I used PTAT.

File src/mainboard/google/brya/variants/xol/ramstage.c:

https://review.coreboot.org/c/coreboot/+/81614/comment/dfb9ff84_ca71fa31 :
PS2, Line 24:
> Log the non-zero return value?

Done

https://review.coreboot.org/c/coreboot/+/81614/comment/92086335_f751206a :
PS2, Line 28: get_soc_power_limit_config
> this is already present inside baseboard/brya. why not leveraging the same ? […]

Done

https://review.coreboot.org/c/coreboot/+/81614/comment/fa41df4c_d1f30ae3 :
PS2, Line 64: performance
> The commit message says *power*?

Done