diff options
| author | Moritz Fischer <moritzf@google.com> | 2021-02-28 19:26:37 -0800 |
|---|---|---|
| committer | Julius Werner <jwerner@chromium.org> | 2021-03-04 01:22:03 +0000 |
| commit | 1e192138d19f0d6d049d06a7a799095c200f04de (patch) | |
| tree | 814d3098879eb435028be5d062caa875a456069d /src/soc/rockchip | |
| parent | a2e5746c811eaf29f77af4ba7dafb4b3b0ed8ca6 (diff) | |
| download | coreboot-1e192138d19f0d6d049d06a7a799095c200f04de.tar.xz | |
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>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51126
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Diffstat (limited to 'src/soc/rockchip')
| -rw-r--r-- | src/soc/rockchip/rk3399/sdram.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/rockchip/rk3399/sdram.c b/src/soc/rockchip/rk3399/sdram.c index 0fdef715c0..6d58b6a171 100644 --- a/src/soc/rockchip/rk3399/sdram.c +++ b/src/soc/rockchip/rk3399/sdram.c @@ -1133,7 +1133,7 @@ void sdram_init(const struct rk3399_sdram_params *params) * 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(); } |