[S] Change in coreboot[master]: mb/google/brask/var/kuldax: Set power limit values for RPL SKUs

[Sumeet R Pawnikar (Code Review)]

Attention is currently required from: Bob Moragues, David Wu, Nick Vaccaro, Paul Menzel, Sumeet R Pawnikar, Tarun Tuli, Zhuohao Lee.

Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76769?usp=email )

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

Change subject: mb/google/brask/var/kuldax: Set power limit values for RPL SKUs
......................................................................

Patch Set 3:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/76769/comment/391176eb_5fa90c53 :
PS3, Line 7: brask
This should be brya, isn't it?

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

https://review.coreboot.org/c/coreboot/+/76769/comment/2e42ffa3_a6ad8f03 :
PS3, Line 19: 100000
Add comment for this PL4 value suggested by thermal team which is different from the reference document 686872.

https://review.coreboot.org/c/coreboot/+/76769/comment/2a11af7e_7bfd9801 :
PS3, Line 43: { PCI_DID_INTEL_RPL_P_ID_5, 15, 138 },
            : 	{ PCI_DID_INTEL_RPL_P_ID_4, 15, 138 },
            : 	{ PCI_DID_INTEL_RPL_P_ID_3, 15, 138 },
I'm not clear why are we maintaining two separate structures to update the same values. If someone miss to update either one, it might create an issue in the future. We might need to come up with better approach for adding these new values. 
It seems originally this revise was added to differentiate for 65W BJ power at line 78.