[PATCH 28/92] ram: rk3399: Handle pctl_cfg return type

Jagan Teki
2019-06-11T14:50:31Z

Add proper return type handling of pctl_cfg with
meaningful print statement.

Signed-off-by: Jagan Teki <jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org>
---
drivers/ram/rockchip/sdram_rk3399.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/ram/rockchip/sdram_rk3399.c b/drivers/ram/rockchip/sdram_rk3399.c
index 6bf8dce6e0..fab7515b2d 100644
--- a/drivers/ram/rockchip/sdram_rk3399.c
+++ b/drivers/ram/rockchip/sdram_rk3399.c
@@ -1120,6 +1120,7 @@ static int sdram_init(struct dram_info *dram,
unsigned char dramtype = sdram_params->base.dramtype;
unsigned int ddr_freq = sdram_params->base.ddr_freq;
int channel;
+ int ret;

debug("Starting SDRAM initialization...\n");

@@ -1139,9 +1140,10 @@ static int sdram_init(struct dram_info *dram,
if (channel >= sdram_params->base.num_channels)
continue;

- if (pctl_cfg(chan, channel, sdram_params) != 0) {
- printf("pctl_cfg fail, reset\n");
- return -EIO;
+ ret = pctl_cfg(chan, channel, sdram_params);
+ if (ret < 0) {
+ printf("%s: pctl config failed\n", __func__);
+ return ret;
}

/* LPDDR2/LPDDR3 need to wait DAI complete, max 10us */
--
2.18.0.321.gffc6fa0e3