From a26dbbd9d066d4276b2d090654c73f7c589b3114 Mon Sep 17 00:00:00 2001 From: Patrick Rudolph Date: Thu, 15 Jun 2017 08:43:11 +0200 Subject: soc/intel/common/opregion: Use enum cb_err as return value MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Return CB_SUCCESS and CB_ERR instead of some integer. Preparation to merge intel/soc and intel/nb opregion implementations. Change-Id: Ib99fcfe347b98736979fc82ab3de48bfc6fc7dcd Signed-off-by: Patrick Rudolph Reviewed-on: https://review.coreboot.org/20220 Tested-by: build bot (Jenkins) Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Martin Roth --- src/soc/intel/apollolake/graphics.c | 2 +- src/soc/intel/common/opregion.c | 10 +++++----- src/soc/intel/common/opregion.h | 2 +- src/soc/intel/skylake/opregion.c | 4 ++-- 4 files changed, 9 insertions(+), 9 deletions(-) (limited to 'src/soc/intel') diff --git a/src/soc/intel/apollolake/graphics.c b/src/soc/intel/apollolake/graphics.c index 7123e4f16f..0893d7dab2 100644 --- a/src/soc/intel/apollolake/graphics.c +++ b/src/soc/intel/apollolake/graphics.c @@ -44,7 +44,7 @@ static unsigned long igd_write_opregion(device_t dev, unsigned long current, printk(BIOS_DEBUG, "ACPI: * IGD OpRegion\n"); opregion = (igd_opregion_t *)current; - if (!init_igd_opregion(opregion)) + if (init_igd_opregion(opregion) != CB_SUCCESS) return current; current += sizeof(igd_opregion_t); diff --git a/src/soc/intel/common/opregion.c b/src/soc/intel/common/opregion.c index 25c0a58c6f..6cb388ce82 100644 --- a/src/soc/intel/common/opregion.c +++ b/src/soc/intel/common/opregion.c @@ -21,7 +21,7 @@ #include "opregion.h" #include "vbt.h" -int init_igd_opregion(igd_opregion_t *opregion) +enum cb_err init_igd_opregion(igd_opregion_t *opregion) { struct region_device vbt_rdev; optionrom_vbt_t *vbt; @@ -29,14 +29,14 @@ int init_igd_opregion(igd_opregion_t *opregion) if (locate_vbt(&vbt_rdev) == CB_ERR) { printk(BIOS_ERR, "VBT not found\n"); - return 0; + return CB_ERR; }; vbt = rdev_mmap_full(&vbt_rdev); if (!vbt) { printk(BIOS_ERR, "VBT couldn't be read\n"); - return 0; + return CB_ERR; } memset(opregion, 0, sizeof(igd_opregion_t)); @@ -51,7 +51,7 @@ int init_igd_opregion(igd_opregion_t *opregion) if (ext_vbt == NULL) { printk(BIOS_ERR, "Unable to add Ext VBT to cbmem!\n"); - return 0; + return CB_ERR; } memcpy(ext_vbt, vbt, vbt->hdr_vbt_size); @@ -71,5 +71,5 @@ int init_igd_opregion(igd_opregion_t *opregion) rdev_munmap(&vbt_rdev, vbt); - return 1; + return CB_SUCCESS; } diff --git a/src/soc/intel/common/opregion.h b/src/soc/intel/common/opregion.h index bcffe6e5f6..7945a49ace 100644 --- a/src/soc/intel/common/opregion.h +++ b/src/soc/intel/common/opregion.h @@ -19,6 +19,6 @@ #include /* Loads vbt and initializes opregion. Returns non-zero on success */ -int init_igd_opregion(igd_opregion_t *opregion); +enum cb_err init_igd_opregion(igd_opregion_t *opregion); #endif /* _COMMON_OPREGION_H_ */ diff --git a/src/soc/intel/skylake/opregion.c b/src/soc/intel/skylake/opregion.c index 4539560efd..ef7d4ab968 100644 --- a/src/soc/intel/skylake/opregion.c +++ b/src/soc/intel/skylake/opregion.c @@ -19,7 +19,7 @@ #include #include -int init_igd_opregion(igd_opregion_t *opregion) +enum cb_err init_igd_opregion(igd_opregion_t *opregion) { const optionrom_vbt_t *vbt; uint32_t vbt_len; @@ -46,5 +46,5 @@ int init_igd_opregion(igd_opregion_t *opregion) /* We just assume we're mobile for now */ opregion->header.mailboxes = MAILBOXES_MOBILE; - return 0; + return CB_SUCCESS; } -- cgit v1.2.3