mirror of
https://github.com/AsahiLinux/u-boot
synced 2024-11-24 13:43:28 +00:00
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>
This commit is contained in:
parent
df9e4cdabb
commit
24f5aec364
8 changed files with 15 additions and 62 deletions
|
@ -136,13 +136,8 @@ static int atmel_sdhci_probe(struct udevice *dev)
|
|||
static int atmel_sdhci_bind(struct udevice *dev)
|
||||
{
|
||||
struct atmel_sdhci_plat *plat = dev_get_platdata(dev);
|
||||
int ret;
|
||||
|
||||
ret = sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
return 0;
|
||||
return sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
}
|
||||
|
||||
static const struct udevice_id atmel_sdhci_ids[] = {
|
||||
|
|
|
@ -284,13 +284,8 @@ static int exynos_dwmmc_probe(struct udevice *dev)
|
|||
static int exynos_dwmmc_bind(struct udevice *dev)
|
||||
{
|
||||
struct exynos_mmc_plat *plat = dev_get_platdata(dev);
|
||||
int ret;
|
||||
|
||||
ret = dwmci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
return 0;
|
||||
return dwmci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
}
|
||||
|
||||
static const struct udevice_id exynos_dwmmc_ids[] = {
|
||||
|
|
|
@ -85,16 +85,10 @@ int mmc_boot_partition_size_change(struct mmc *mmc, unsigned long bootsize,
|
|||
*/
|
||||
int mmc_set_boot_bus_width(struct mmc *mmc, u8 width, u8 reset, u8 mode)
|
||||
{
|
||||
int err;
|
||||
|
||||
err = mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, EXT_CSD_BOOT_BUS_WIDTH,
|
||||
EXT_CSD_BOOT_BUS_WIDTH_MODE(mode) |
|
||||
EXT_CSD_BOOT_BUS_WIDTH_RESET(reset) |
|
||||
EXT_CSD_BOOT_BUS_WIDTH_WIDTH(width));
|
||||
|
||||
if (err)
|
||||
return err;
|
||||
return 0;
|
||||
return mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, EXT_CSD_BOOT_BUS_WIDTH,
|
||||
EXT_CSD_BOOT_BUS_WIDTH_MODE(mode) |
|
||||
EXT_CSD_BOOT_BUS_WIDTH_RESET(reset) |
|
||||
EXT_CSD_BOOT_BUS_WIDTH_WIDTH(width));
|
||||
}
|
||||
|
||||
/*
|
||||
|
@ -106,16 +100,10 @@ int mmc_set_boot_bus_width(struct mmc *mmc, u8 width, u8 reset, u8 mode)
|
|||
*/
|
||||
int mmc_set_part_conf(struct mmc *mmc, u8 ack, u8 part_num, u8 access)
|
||||
{
|
||||
int err;
|
||||
|
||||
err = mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, EXT_CSD_PART_CONF,
|
||||
EXT_CSD_BOOT_ACK(ack) |
|
||||
EXT_CSD_BOOT_PART_NUM(part_num) |
|
||||
EXT_CSD_PARTITION_ACCESS(access));
|
||||
|
||||
if (err)
|
||||
return err;
|
||||
return 0;
|
||||
return mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, EXT_CSD_PART_CONF,
|
||||
EXT_CSD_BOOT_ACK(ack) |
|
||||
EXT_CSD_BOOT_PART_NUM(part_num) |
|
||||
EXT_CSD_PARTITION_ACCESS(access));
|
||||
}
|
||||
|
||||
/*
|
||||
|
|
|
@ -190,13 +190,8 @@ static int msm_ofdata_to_platdata(struct udevice *dev)
|
|||
static int msm_sdc_bind(struct udevice *dev)
|
||||
{
|
||||
struct msm_sdhc_plat *plat = dev_get_platdata(dev);
|
||||
int ret;
|
||||
|
||||
ret = sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
return 0;
|
||||
return sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
}
|
||||
|
||||
static const struct udevice_id msm_mmc_ids[] = {
|
||||
|
|
|
@ -142,13 +142,8 @@ static int rockchip_dwmmc_probe(struct udevice *dev)
|
|||
static int rockchip_dwmmc_bind(struct udevice *dev)
|
||||
{
|
||||
struct rockchip_mmc_plat *plat = dev_get_platdata(dev);
|
||||
int ret;
|
||||
|
||||
ret = dwmci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
return 0;
|
||||
return dwmci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
}
|
||||
|
||||
static const struct udevice_id rockchip_dwmmc_ids[] = {
|
||||
|
|
|
@ -62,13 +62,8 @@ static int arasan_sdhci_ofdata_to_platdata(struct udevice *dev)
|
|||
static int rockchip_sdhci_bind(struct udevice *dev)
|
||||
{
|
||||
struct rockchip_sdhc_plat *plat = dev_get_platdata(dev);
|
||||
int ret;
|
||||
|
||||
ret = sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
return 0;
|
||||
return sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
}
|
||||
|
||||
static const struct udevice_id arasan_sdhci_ids[] = {
|
||||
|
|
|
@ -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)
|
||||
|
|
|
@ -63,13 +63,8 @@ static int arasan_sdhci_ofdata_to_platdata(struct udevice *dev)
|
|||
static int arasan_sdhci_bind(struct udevice *dev)
|
||||
{
|
||||
struct arasan_sdhci_plat *plat = dev_get_platdata(dev);
|
||||
int ret;
|
||||
|
||||
ret = sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
return 0;
|
||||
return sdhci_bind(dev, &plat->mmc, &plat->cfg);
|
||||
}
|
||||
|
||||
static const struct udevice_id arasan_sdhci_ids[] = {
|
||||
|
|
Loading…
Reference in a new issue