There's already 'ret' availible, no need to declare another return
value variable.
Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
---
drivers/mci/imx-esdhc.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/mci/imx-esdhc.c b/drivers/mci/imx-esdhc.c
index 7d12df2b8..b6dd75d2c 100644
--- a/drivers/mci/imx-esdhc.c
+++ b/drivers/mci/imx-esdhc.c
@@ -236,8 +236,6 @@ esdhc_send_cmd(struct mci_host *mci, struct mci_cmd *cmd, struct mci_data *data)
/* Set up for a data transfer if we have one */
if (data) {
- int err;
-
if (!IS_ENABLED(CONFIG_MCI_IMX_ESDHC_PIO)) {
num_bytes = data->blocks * data->blocksize;
@@ -254,9 +252,9 @@ esdhc_send_cmd(struct mci_host *mci, struct mci_cmd *cmd, struct mci_data *data)
return -EFAULT;
}
- err = esdhc_setup_data(host, data, dma);
- if(err)
- return err;
+ ret = esdhc_setup_data(host, data, dma);
+ if (ret)
+ return ret;
}
sdhci_set_cmd_xfer_mode(&host->sdhci, cmd, data,
--
2.21.0
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
