[M] Change in coreboot[main]: soc/intel/pantherlake: Add core scaling factors read support

[Subrata Banik (Code Review)]

Attention is currently required from: Jérémy Compostella.

Subrata Banik has posted comments on this change by Jérémy Compostella. ( https://review.coreboot.org/c/coreboot/+/85554?usp=email )

Change subject: soc/intel/pantherlake: Add core scaling factors read support
......................................................................

Patch Set 11:

(1 comment)

Patchset:

PS11:
> Initially, I shared similar concerns when assigned this task. However, after consulting with internal experts, I gained insights into the rationale behind the proposed dynamic approach. Recent experiences have revealed a disconnect between IA firmware, operating system, and other SoC firmware, resulting in certain issues.

So you're saying that after booting to the OS, the kernel can't override the static boot config scaling ratio parameters? And that it's always relied on BIOS-programmed static values? That's a problem. We don't expect the BIOS to be as intelligent or capable as the kernel, to dynamically handle things. The BIOS should just be there to allow the OS to launch, and the OS should be able to handle most trivial tasks. It sounds like there's a gap between who designed the architecture and whoever is heavily relying on the BIOS to be the only source of truth here.

Recognizing the minimal boot time impact of approximately 12 microseconds associated with reading these values, I concluded that this approach was prudent to mitigate long-term concerns. This recommendation extends to future generations as well.