summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEdward O'Callaghan <quasisec@google.com>2020-03-26 16:47:55 +1100
committerEdward O'Callaghan <quasisec@chromium.org>2020-03-26 06:27:41 +0000
commit2a0a02f98f742ff921ac3a9a8ccbf7fe47690509 (patch)
tree9d79e5adeec241c0894611c45b6b47097f5a9e94
parent0e1380683fae3fd042ff30274add110a45720aa1 (diff)
downloadcoreboot-2a0a02f98f742ff921ac3a9a8ccbf7fe47690509.tar.xz
drivers/net/r8168: Fix extraneous space in "ethernet_mac "
Unfortunately this was noticed only after commit 0e1380683f merged, credit to Sam McNally for spotting it. Previously the legacy path replaced the space with a null byte and so the expected string here is precisely "ethernet_mac" and not "ethernet_mac ". BUG=b:152157720 BRANCH=none TEST=none Change-Id: I603fad4efd6d6c539137dd714329bcea1877abdd Signed-off-by: Edward O'Callaghan <quasisec@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/39856 Reviewed-by: Sam McNally <sammc@google.com> Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/drivers/net/r8168.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/net/r8168.c b/src/drivers/net/r8168.c
index eaf33f2978..a3e1e1b049 100644
--- a/src/drivers/net/r8168.c
+++ b/src/drivers/net/r8168.c
@@ -170,7 +170,7 @@ static void fetch_mac_string_vpd(struct drivers_net_config *config, u8 *macstrbu
return;
}
- if (fetch_mac_vpd_key(macstrbuf, "ethernet_mac ") != CB_SUCCESS)
+ if (fetch_mac_vpd_key(macstrbuf, "ethernet_mac") != CB_SUCCESS)
printk(BIOS_ERR, "r8168: mac address not found in VPD,"
" using default 00:e0:4c:00:c0:b0\n");
}