cbfs: Replace more instances of cbfs_boot_locate() with newer APIs
In pursuit of the eventual goal of removing cbfs_boot_locate() (and
direct rdev access) from CBFS APIs, this patch replaces all remaining
"simple" uses of the function call that can easily be replaced by the
newer APIs (like cbfs_load() or cbfs_map()). Some cases of
cbfs_boot_locate() remain that will be more complicated to solve.
Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: Icd0f21e2fa49c7cc834523578b7b45b5482cb1a8
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50348
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
diff --git a/src/drivers/net/atl1e.c b/src/drivers/net/atl1e.c
index b533537..9b1b2ab 100644
--- a/src/drivers/net/atl1e.c
+++ b/src/drivers/net/atl1e.c
@@ -41,18 +41,11 @@
static enum cb_err fetch_mac_string_cbfs(u8 *macstrbuf)
{
- struct cbfsf fh;
- uint32_t matchraw = CBFS_TYPE_RAW;
-
- if (!cbfs_boot_locate(&fh, "atl1e-macaddress", &matchraw)) {
- /* check the cbfs for the mac address */
- if (rdev_readat(&fh.data, macstrbuf, 0, MACLEN) != MACLEN) {
- printk(BIOS_ERR, "atl1e: Error reading MAC from CBFS\n");
- return CB_ERR;
- }
- return CB_SUCCESS;
+ if (!cbfs_load("atl1e-macaddress", macstrbuf, MACLEN)) {
+ printk(BIOS_ERR, "atl1e: Error reading MAC from CBFS\n");
+ return CB_ERR;
}
- return CB_ERR;
+ return CB_SUCCESS;
}
static void get_mac_address(u8 *macaddr, const u8 *strbuf)