On Thu, Nov 1, 2018 at 2:25 AM Marek Vasut <marek.va...@gmail.com> wrote: > > Properly handle return values and abort operations when they are > non-zero. This is a minor improvement, which fixes two remaining > unchecked return values. > > Signed-off-by: Marek Vasut <marek.vasut+rene...@gmail.com> > Cc: Masahiro Yamada <yamada.masah...@socionext.com> > ---
Good catch! Thanks. > drivers/mmc/tmio-common.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/mmc/tmio-common.c b/drivers/mmc/tmio-common.c > index c8caa0fb18..8e83584e57 100644 > --- a/drivers/mmc/tmio-common.c > +++ b/drivers/mmc/tmio-common.c > @@ -498,6 +498,8 @@ int tmio_sd_send_cmd(struct udevice *dev, struct mmc_cmd > *cmd, > ret = tmio_sd_dma_xfer(dev, data); > else > ret = tmio_sd_pio_xfer(dev, data); > + if (ret) > + return ret; > > ret = tmio_sd_wait_for_irq(dev, TMIO_SD_INFO1, > TMIO_SD_INFO1_CMP); > @@ -505,9 +507,8 @@ int tmio_sd_send_cmd(struct udevice *dev, struct mmc_cmd > *cmd, > return ret; > } > > - tmio_sd_wait_for_irq(dev, TMIO_SD_INFO2, TMIO_SD_INFO2_SCLKDIVEN); > - > - return ret; > + return tmio_sd_wait_for_irq(dev, TMIO_SD_INFO2, > + TMIO_SD_INFO2_SCLKDIVEN); > } > > static int tmio_sd_set_bus_width(struct tmio_sd_priv *priv, > -- > 2.18.0 > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot -- Best Regards Masahiro Yamada _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
