diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2020-03-26 16:47:55 +1100 |
|---|---|---|
| committer | Edward O'Callaghan <quasisec@chromium.org> | 2020-03-26 06:27:41 +0000 |
| commit | 2a0a02f98f742ff921ac3a9a8ccbf7fe47690509 (patch) | |
| tree | 9d79e5adeec241c0894611c45b6b47097f5a9e94 | |
| parent | 0e1380683fae3fd042ff30274add110a45720aa1 (diff) | |
| download | coreboot-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.c | 2 |
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"); } |