summaryrefslogtreecommitdiff
path: root/src/device
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2020-07-29 17:39:21 -0700
committerPatrick Georgi <pgeorgi@google.com>2020-08-03 05:15:15 +0000
commit3e034b6e9aaddcca1996773b25d2ee88940d1144 (patch)
treeefb1a415a83fa68fed21d14f4f407ced7b1d545f /src/device
parentc435d3daa71005d190373ee00a6491520b542eaa (diff)
Change all assert(0) to BUG()
I would like to make assertions evaluate at compile time where possible, but sometimes people used a literal assert(0) to force an assertion in a certain code path. We already have BUG() for that so let's just replace those instances with that. Signed-off-by: Julius Werner <jwerner@chromium.org> Change-Id: I674e5f8ec7f5fe8b92b1c7c95d9f9202d422ce32 Reviewed-on: https://review.coreboot.org/c/coreboot/+/44047 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Furquan Shaikh <furquan@google.com> Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/device')
-rw-r--r--src/device/device_const.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/device/device_const.c b/src/device/device_const.c
index 79f025da97..2dc583ca27 100644
--- a/src/device/device_const.c
+++ b/src/device/device_const.c
@@ -184,7 +184,7 @@ DEVTREE_CONST struct device *find_dev_path(
DEVTREE_CONST struct device *child;
if (!parent) {
- assert(0);
+ BUG();
/* Return NULL in case asserts are considered non-fatal. */
return NULL;
}
@@ -282,7 +282,7 @@ DEVTREE_CONST struct device *pcidev_path_behind_pci2pci_bridge(
pci_devfn_t devfn)
{
if (!bridge || (bridge->path.type != DEVICE_PATH_PCI)) {
- assert(0);
+ BUG();
/* Return NULL in case asserts are non-fatal. */
return NULL;
}