On 17.2.2017 5:56, Shawn Lin wrote:
> We need comment there to tell folks that we need this
> callback to return the timeout clock rate in KHz.
Good idea, except that currently in some cases it should return the rate
in MHz instead so the added comment is actually wrong.
Otherwise the patchset seems correct to me, and the timeout clock
situation would be better than before.
One option would be to just have the callback return Hz as that seems to
be the most obvious unit. The callback would probably have to be renamed
for that, though, to avoid any backporting etc. headaches later.
> Signed-off-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx>
> ---
>
> drivers/mmc/host/sdhci.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/mmc/host/sdhci.h b/drivers/mmc/host/sdhci.h
> index edf3adf..d5b4d57 100644
> --- a/drivers/mmc/host/sdhci.h
> +++ b/drivers/mmc/host/sdhci.h
> @@ -547,6 +547,7 @@ struct sdhci_ops {
> int (*enable_dma)(struct sdhci_host *host);
> unsigned int (*get_max_clock)(struct sdhci_host *host);
> unsigned int (*get_min_clock)(struct sdhci_host *host);
> + /* get_timeout_clock should return clk rate in KHz */
> unsigned int (*get_timeout_clock)(struct sdhci_host *host);
> unsigned int (*get_max_timeout_count)(struct sdhci_host *host);
> void (*set_timeout)(struct sdhci_host *host,
--
Anssi Hannula / Bitwise Oy
+358503803997
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
