Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47186 )
Change subject: mb/purism/librem_mini: Increase TDP/PL2 setting ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47186/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/47186/1//COMMIT_MSG@11 PS1, Line 11: which works with both Librem Mini variants
well, when one of both support a higher value, why not use a overridetree?
Both variants can use the same value. It's just that the fan on the WHL minis originally wasn't working with coreboot (was fixed with an EC FW update) and the PL2 value was lowered to avoid literal meltdown. Now that the fan is working, there's no reason to use the lower value anymore.
Since both devices use very similar boards (main difference is the SoC they come with), there's no need to use overridetrees at all. I've even seen WHL and CML chips which have the same CPUID.