On 03/18/2018 01:03 AM, Vagrant Cascadian wrote:
> On 2018-03-17, Heinrich Schuchardt wrote:
>> The Odroid C2 fails to read from mmc with U-Boot v2018.03.
>> The change avoids a division by zero.
>>
>> The fix was suggested by Jaehoon in
>> https://lists.denx.de/pipermail/u-boot/2018-January/318577.html
> 
> Thanks!
> 
> Works for me with odroid-c2 on u-boot 2018.03.
> 
> Without the patch, mmc access immediately hangs the board.
> 
> Tested-by: Vagrant Cascadian <vagr...@debian.org>
> 
> 
> live well,
>   vagrant
> 
>> Reported-by: Vagrant Cascadian <vagr...@debian.org>
>> Suggested-by: Jaehoon Chung <jh80.ch...@samsung.com>
>> Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
>> ---
>> Resent due to missing cc u-boot@lists.denx.de.
>> ---
>>  drivers/mmc/meson_gx_mmc.c | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/mmc/meson_gx_mmc.c b/drivers/mmc/meson_gx_mmc.c
>> index a2cd5d3a44..454593eec4 100644
>> --- a/drivers/mmc/meson_gx_mmc.c
>> +++ b/drivers/mmc/meson_gx_mmc.c
>> @@ -35,6 +35,9 @@ static void meson_mmc_config_clock(struct mmc *mmc)
>>      uint32_t meson_mmc_clk = 0;
>>      unsigned int clk, clk_src, clk_div;
>>  
>> +    if (!mmc->clock)
>> +            return;
>> +
>>      /* 1GHz / CLK_MAX_DIV = 15,9 MHz */
>>      if (mmc->clock > 16000000) {
>>              clk = SD_EMMC_CLKSRC_DIV2;
>> -- 
>> 2.14.2

Hello Jaehoon,

will you pick the patch?

Best regards

Heinrich
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to