[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