[coreboot-gerrit] Change in coreboot[master]: vendorcode/google: load sar config from CBFS first then VPD

[Justin TerAvest (Code Review)]

Justin TerAvest has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33661 )

Change subject: vendorcode/google: load sar config from CBFS first then VPD
......................................................................

Patch Set 1:
> Patch Set 1:
> > +1. With support for CBFS SAR, I would like to completely deprecate the use of VPD for SAR and thus remove it from coreboot as well.
>
> +1000, and that's exactly what I proposed in email. In fact, I thought (or hoped, since it seemed like the sane option) we already *had* removed support until that thread came up.
>
> I see the rest as a feature, not a bug. A new model is a new model. We don't try to pretend it's not just to make it easier for the ODM to screw everything up^W^W^Witerate more quickly. (Or IOW: what Hung-Te said.)

We don't want to define new models just to allow for differences in SAR values.
We do want to allow for flexibility in devices without incurring tons of deployment and testing overhead.