[PATCH 4/8] mmc: sdhci-of-aspeed: Get max clockk by using default api
Andrew Jeffery
andrew at codeconstruct.com.au
Tue Jun 17 19:39:09 PDT 2025
On Sun, 2025-06-15 at 11:57 +0800, Cool Lee wrote:
> Don't limit clock frequency by f_max.
>
> Signed-off-by: Cool Lee <cool_lee at aspeedtech.com>
> ---
> drivers/mmc/host/sdhci-of-aspeed.c | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci-of-aspeed.c b/drivers/mmc/host/sdhci-of-aspeed.c
> index 10160a706334..2bdd93a3f91f 100644
> --- a/drivers/mmc/host/sdhci-of-aspeed.c
> +++ b/drivers/mmc/host/sdhci-of-aspeed.c
> @@ -288,14 +288,6 @@ static void aspeed_sdhci_set_clock(struct sdhci_host *host, unsigned int clock)
> sdhci_enable_clk(host, clk);
> }
>
> -static unsigned int aspeed_sdhci_get_max_clock(struct sdhci_host *host)
> -{
> - if (host->mmc->f_max)
> - return host->mmc->f_max;
> -
> - return sdhci_pltfm_clk_get_max_clock(host);
> -}
> -
> static void aspeed_sdhci_set_bus_width(struct sdhci_host *host, int width)
> {
> struct sdhci_pltfm_host *pltfm_priv;
> @@ -446,7 +438,7 @@ static int aspeed_sdhci_execute_tuning(struct sdhci_host *host, u32 opcode)
> static const struct sdhci_ops aspeed_sdhci_ops = {
> .read_l = aspeed_sdhci_readl,
> .set_clock = aspeed_sdhci_set_clock,
> - .get_max_clock = aspeed_sdhci_get_max_clock,
> + .get_max_clock = sdhci_pltfm_clk_get_max_clock,
This was used to limit the maximum bus speed via the devicetree. See:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.16-rc2&id=0a0e8d7501cda79c9b20f6011814e2ec9b473ade
Why remove it? There's no discussion of the motivation in the commit
message.
Andrew
More information about the linux-arm-kernel
mailing list