Hi Simon,
On 07/18/2016 07:55 PM, Simon Glass wrote:
Hi Kever,
On 18 July 2016 at 03:52, Kever Yang <kever.y...@rock-chips.com> wrote:
The devnum in is start from 0, so the device number should be
(devnum+1).
Signed-off-by: Kever Yang <kever.y...@rock-chips.com>
---
drivers/mmc/mmc-uclass.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c
index 38ced41..7c40d24 100644
--- a/drivers/mmc/mmc-uclass.c
+++ b/drivers/mmc/mmc-uclass.c
@@ -111,7 +111,7 @@ struct mmc *find_mmc_device(int dev_num)
int get_mmc_num(void)
{
- return max(blk_find_max_devnum(IF_TYPE_MMC), 0);
+ return max((blk_find_max_devnum(IF_TYPE_MMC)+1), 0);
Please add spaces around the +
Also can you add a comment to the function in mmc.h describing what it
should do? From what you are saying it should return the number of mmc
devices.
Also you will need to adjust mmc_get_next_devnum() which uses this function.
Sure, checkpatch script did not give me an error report for the format :(
For mmc_get_next_devnum(), not like legacy mmc, no one use it in mmc-uclass,
are we still need this function? blk_find_max_devnum() is used in
blk_create_device.
Thanks,
- Kever
}
int mmc_get_next_devnum(void)
--
1.9.1
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
