On 3/25/21 4:18 PM, Aswath Govindraju wrote:
> Use dev_seq() to read aliases node's index and pass it as device number
> for creating bulk device.
> 
> Suggested-by: Grygorii Strashko <grygorii.stras...@ti.com>
> Signed-off-by: Aswath Govindraju <a-govindr...@ti.com>

Reviewed-by: Jaehoon Chung <jh80.ch...@samsung.com>

Best Regards,
Jaehoon Chung

> ---
>  drivers/mmc/mmc-uclass.c | 12 +++++-------
>  1 file changed, 5 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c
> index 53eabc9e612d..d36aae367e7c 100644
> --- a/drivers/mmc/mmc-uclass.c
> +++ b/drivers/mmc/mmc-uclass.c
> @@ -383,18 +383,16 @@ int mmc_bind(struct udevice *dev, struct mmc *mmc, 
> const struct mmc_config *cfg)
>  {
>       struct blk_desc *bdesc;
>       struct udevice *bdev;
> -     int ret, devnum = -1;
> +     int ret;
>  
>       if (!mmc_get_ops(dev))
>               return -ENOSYS;
> -#ifndef CONFIG_SPL_BUILD
> -     /* Use the fixed index with aliase node's index */
> -     ret = dev_read_alias_seq(dev, &devnum);
> -     debug("%s: alias ret=%d, devnum=%d\n", __func__, ret, devnum);
> -#endif
> +
> +     /* Use the fixed index with aliases node's index */
> +     debug("%s: alias devnum=%d\n", __func__, dev_seq(dev));
>  
>       ret = blk_create_devicef(dev, "mmc_blk", "blk", IF_TYPE_MMC,
> -                     devnum, 512, 0, &bdev);
> +                     dev_seq(dev), 512, 0, &bdev);
>       if (ret) {
>               debug("Cannot create block device\n");
>               return ret;
> 

Reply via email to