diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2016-09-06 22:17:32 +0900 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2016-09-23 17:53:44 -0400 |
commit | 24f5aec3646f2a22d8ff8e02c07d6c9a92a5092a (patch) | |
tree | 53282ede51b26d1358e8e375d31f1b7eee271dab /drivers/mmc/sandbox_mmc.c | |
parent | df9e4cdabbb9ea2a1a1686d58bc389cb028a41f0 (diff) |
mmc: squash lines for immediate return
These functions can be much simpler by squashing lines for immediate
return.
For *_bind() callbacks, they will be a simple wrapper function of an
upper-level bind API.
For mmc_set_{boot_bus_width,part_conf}, they will be a wrapper of
mmc_switch().
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
Diffstat (limited to 'drivers/mmc/sandbox_mmc.c')
-rw-r--r-- | drivers/mmc/sandbox_mmc.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/mmc/sandbox_mmc.c b/drivers/mmc/sandbox_mmc.c index 5f1333b7480..fdb29a5505c 100644 --- a/drivers/mmc/sandbox_mmc.c +++ b/drivers/mmc/sandbox_mmc.c @@ -112,7 +112,6 @@ int sandbox_mmc_bind(struct udevice *dev) { struct sandbox_mmc_plat *plat = dev_get_platdata(dev); struct mmc_config *cfg = &plat->cfg; - int ret; cfg->name = dev->name; cfg->host_caps = MMC_MODE_HS_52MHz | MMC_MODE_HS | MMC_MODE_8BIT; @@ -121,11 +120,7 @@ int sandbox_mmc_bind(struct udevice *dev) cfg->f_max = 52000000; cfg->b_max = U32_MAX; - ret = mmc_bind(dev, &plat->mmc, cfg); - if (ret) - return ret; - - return 0; + return mmc_bind(dev, &plat->mmc, cfg); } int sandbox_mmc_unbind(struct udevice *dev) |