[coreboot-gerrit] New patch to review for coreboot: b9e7799 google/panther: fix MAC address programming when VPD not present

Matt DeVillier (matt.devillier@gmail.com) gerrit at coreboot.org
Thu Jun 11 05:38:38 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 b9e7799b167279db358124d30cd821580cfc6624
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
    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