Re: [PATCH V6 22/24] mmc: sdhci-uhs2: add post-mmc_attach_sd hook

From: Adrian Hunter
Date: Thu Jan 05 2023 - 16:32:01 EST


On 13/12/22 11:00, Victor Shih wrote:
> From: Ben Chuang <ben.chuang@xxxxxxxxxxxxxxxxxxx>
>
> This "post" hook for mmc_attach_sd(), uhs2_post_attach_sd, will be required
> to enable UHS-II support, at least, on GL9755.
>
> Signed-off-by: Ben Chuang <ben.chuang@xxxxxxxxxxxxxxxxxxx>
> Signed-off-by: AKASHI Takahiro <takahiro.akashi@xxxxxxxxxx>
> ---
> 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 e5f64b38e339..4aac22b58d79 100644
> --- a/drivers/mmc/host/sdhci.h
> +++ b/drivers/mmc/host/sdhci.h
> @@ -728,6 +728,7 @@ struct sdhci_ops {
> void (*dump_vendor_regs)(struct sdhci_host *host);
> void (*dump_uhs2_regs)(struct sdhci_host *host);
> void (*uhs2_pre_detect_init)(struct sdhci_host *host);
> + void (*uhs2_post_attach_sd)(struct sdhci_host *host);

This gets called already in "mmc: sdhci-uhs2: add uhs2_control()
to initialise the interface"

Either move that line to this patch or vice versa.

Please check compiling after each patch is applied.

> };
>
> #ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS