[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