[U-Boot] [PATCH] mmc: fsl_esdhc: Fix i.MX53 eSDHCv3 clock

Stefano Babic sbabic at denx.de
Mon Jan 15 21:27:43 UTC 2018


Hi Benoît,

On 15/01/2018 22:11, Benoît Thébaudeau wrote:
> Hi Stefano,
> 
> On Mon, Jan 15, 2018 at 11:59 AM, Stefano Babic <sbabic at denx.de> wrote:
>> On 15/01/2018 00:46, Benoît Thébaudeau wrote:
>>> +     int pre_div = regs == (struct fsl_esdhc *)MMC_SDHC3_BASE_ADDR ? 2 : 1;
>>
>> It is surely a question of taste - but is it not cleared to surround the
>> expression with parenthesis ? (regs == .... )
> 
> I can send a v2 for that if you want. Which of the following do you
> prefer (personally, I'd say the last one)?
> 
> int pre_div = (regs == (struct fsl_esdhc *)MMC_SDHC3_BASE_ADDR ? 2 : 1);
> int pre_div = (regs == (struct fsl_esdhc *)MMC_SDHC3_BASE_ADDR) ? 2 : 1;
> 

I agree for last one - but as I said, it is just a question of taste.

> Note that this patch can be tested with the 2nd MMC on i.MX53 Loco if
> anyone has this board. Wladimir?

Best regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list