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@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.
Thanks, Faiz