Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31744 )
Change subject: x86/car: Fix incorrect config usage
......................................................................
Patch Set 1: Code-Review+2
I have had a closer look at this issue on mc_bdx1 (fsp_broadwell_de) and mc_tcu3 (fsp_baytrail) which are both fsp1.0 based. Currently (with this error still there) do_car_migrate_variables() is exited on line 127 (as Kyösti pointed out already). So there is no real work that is done here and "removing" it for fsp 1.0 based platforms should not harm.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/31744
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I74e0ed4d04471ee521ff5c69a74a6f4c949e5847
Gerrit-Change-Number: 31744
Gerrit-PatchSet: 1
Gerrit-Owner: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Piotr Król
piotr.krol@3mdeb.com
Gerrit-Reviewer: Werner Zeh
werner.zeh@siemens.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Tue, 05 Mar 2019 07:19:53 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment