flashrom.c: Fix 'internal'/'host' assumption of map_flash() failures
Unfortunately some, but not all, board chipset+spi chip combos
result in map_flash() failures while other configurations require
it. Previously the mapping was done unchecked and so a failure in
mapping was silient however now that maps are done via a dedicated
function from upstream and the return value is getting checked it
became apparent some board topologies fail to map_flash for some
For now we restore previous behaviour by not checking the return
value and instead proceed assuming the map_flash() was successful.
This results in a nasty looking msg in logs however the msg is
correct and useful as a data collection mechanism to find out how
wide spread this is and to dig down what is common, thus we leave
the msg in for now.
TEST=verified to fix Volteer && Trogdor.
Signed-off-by: Edward O'Callaghan <firstname.lastname@example.org>
Tested-by: Edward O'Callaghan <email@example.com>
Reviewed-by: Alex Levin <firstname.lastname@example.org>
Reviewed-by: Sam McNally <email@example.com>
Commit-Queue: Sam McNally <firstname.lastname@example.org>
Auto-Submit: Edward O'Callaghan <email@example.com>
1 file changed