Attention is currently required from: Kapil Porwal, Paul Menzel, Subrata Banik, YH Lin.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/82084?usp=email )
Change subject: soc/intel/mtl: use different names per mtlrvp variants ......................................................................
Patch Set 4: Code-Review+1
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/82084/comment/8a079b3c_2f272547 : PS4, Line 7: soc/intel/mtl: ``mb/intel/mtlrvp``
https://review.coreboot.org/c/coreboot/+/82084/comment/7d387cec_aa0ba509 : PS4, Line 7: per nit: replace `per` with `for`
https://review.coreboot.org/c/coreboot/+/82084/comment/4f60826e_8b2d2cca : PS4, Line 14: TEST=Verified boot functionality on google/rex. Given that this change only affects `intel/mtlrvp`, I don't think testing on `google/rex` matters (it wouldn't be affected).
If you can get this tested on `intel/mtlrvp` (by you or anyone else), then please update the `TEST=` line accordingly. Otherwise, I would just remove it.