Peichao Li has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32296 )
Change subject: mb/google/octopus: Add custom SAR values for Laser ......................................................................
Patch Set 7:
Patch Set 7:
Patch Set 7:
Patch Set 6:
Patch Set 4:
Looks like there is a common way to get sar file already and please refer to https://chromium-review.googlesource.com/c/chromiumos/overlays/chromiumos-ov...
So, laser requires the SAR file, but phaser does not?
Hi Furquan,
Yes, just Laser need customized SAR file. Got it let's take a look.
Thanks and best regards
Hi Furquan,
I have check config.phaser, but if modify IFWI, both phaser and Laser project will be impacted. I mean Whether should create a new config file for laser like config.laser. Thanks a lot!
Hi Marco and Furquan,
This is a special case. I think we should apply SAR for Laser according to sku id. sku id 5 is laser, so i think config.phaser would not add this SAR.
Thanks and best regards