[PATCH v2 02/10] mmc: Add init() API

Faiz Abbas faiz_abbas at ti.com
Mon Feb 3 11:48:04 CET 2020

Hi Peng,

On 01/02/20 6:43 pm, Peng Fan wrote:
>> Subject: Re: [PATCH v2 02/10] mmc: Add init() API
>> Hi Simon,
>> On 29/01/20 1:33 pm, Simon Goldschmidt wrote:
>>> On Fri, Jan 24, 2020 at 12:52 PM Faiz Abbas <faiz_abbas at ti.com> wrote:
>>>> Add an init() API for platform specific init() operations.
>>> Could you describe why this cannot be done in the probe callback? It's
>>> not easily visible as the function you changed (mmc_get_op_cond)
>>> doesn't even have a comment to describe what it does...
>> The reason is detailed in 06/10 patch description. probe() is always called for
>> all MMC instances. I only want to switch on power (by calling
>> sdhci_init()) and suffer the 1 second wait time when there is actually a card in
>> the slot and user wants to access it.
>>> In general, I think commit messages could be more detailed than one
>>> line. If only to make it easier in the future to recap why things have been
>> done.
>> You're right. I will add a more detailed patch description in v2.
> This patch is v2. Please v3.
> Could a quirk be added in probe and if card not there, just bypass?
> Or you want quick boot to avoid probe all the controllers?

The issue is that we need to wait for 1 second to detect the card
itself. I want to move that delay out from probe().

BTW did you receive this mail directly or through the list? I had CC'd
you but its not there in the patchwork headers.


More information about the U-Boot mailing list