[coreboot-gerrit] Patch set updated for coreboot: 6a7e242 google/panther: fix MAC address programming when VPD not present
Matt DeVillier (matt.devillier@gmail.com)
gerrit at coreboot.org
Thu Jun 11 17:04:34 CEST 2015
Matt DeVillier (matt.devillier at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10509
-gerrit
commit 6a7e242a4f70065274fa8784bab49cb00a91dd18
Author: Matt DeVillier <matt.devillier at gmail.com>
Date: Wed Jun 10 22:31:28 2015 -0500
google/panther: fix MAC address programming when VPD not present
Commit 899d13d (cbfs: new API and better program loading) broke
panther's lan init when no vpd.bin present from which to read
the MAC address. Fix this by checking the validity of the search
address pointer, rather than the search length.
Change-Id: I8c7ca410d8ce5c5d92242a21c4c2ff4c001a68bd
Signed-off-by: Matt DeVillier <matt.devillier at gmail.com>
---
src/mainboard/google/panther/lan.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mainboard/google/panther/lan.c b/src/mainboard/google/panther/lan.c
index 4b34c06..5d806db 100644
--- a/src/mainboard/google/panther/lan.c
+++ b/src/mainboard/google/panther/lan.c
@@ -132,7 +132,7 @@ static void program_mac_address(u16 io_base)
&search_length);
}
- if (search_length <= 0)
+ if (search_address == NULL)
printk(BIOS_ERR, "LAN: VPD not found.\n");
else
get_mac_address(&high_dword, &low_dword, search_address,
More information about the coreboot-gerrit
mailing list