Moritz Fischer has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/51126 )
Change subject: soc/rockchip/rk3399/sdram: Simplify error condition ......................................................................
soc/rockchip/rk3399/sdram: Simplify error condition
There is no need for explicit 0 comparison, any return value not equal to 0 is treated as error.
Change-Id: I72612af4108a616b6247ee68c8ac2a53242b0853 Signed-off-by: Moritz Fischer moritzf@google.com --- M src/soc/rockchip/rk3399/sdram.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/51126/1
diff --git a/src/soc/rockchip/rk3399/sdram.c b/src/soc/rockchip/rk3399/sdram.c index 0fdef71..6d58b6a 100644 --- a/src/soc/rockchip/rk3399/sdram.c +++ b/src/soc/rockchip/rk3399/sdram.c @@ -1133,7 +1133,7 @@ * step may fail, before that, we just reset the * system, and start again. */ - if (pctl_cfg(channel, params) != 0) { + if (pctl_cfg(channel, params)) { printk(BIOS_ERR, "pctl_cfg fail, reset\n"); board_reset(); }