[PATCH] mmc: sdhci: Correct ADMA_DESC_LEN to 12

Jon Humphreys j-humphreys at ti.com
Wed May 1 22:06:49 CEST 2024


"A. Sverdlin" <alexander.sverdlin at siemens.com> writes:

> From: Alexander Sverdlin <alexander.sverdlin at siemens.com>
>
> Commit 37cb626da25d0d ("mmc: sdhci: Add Support for ADMA2") introduced
> ADMA_DESC_LEN == 16 (64 bit case), but it was never used before commit
> 74755c1fed1b0 ("mmc: sdhci: introduce adma_write_desc() hook to struct sdhci_ops").
>
> "sizeof(struct sdhci_adma_desc)" (== 12 for 64bit case) was used instead.
>
> Confusion probably originates from Linux commit 685e444bbaa0
> ("mmc: sdhci: Add ADMA2 64-bit addressing support for V4 mode"), but
> the latter "V4 mode" was never ported to U-Boot.
>
> Fixes: 74755c1fed1b0 ("mmc: sdhci: introduce adma_write_desc() hook to struct sdhci_ops")
> Signed-off-by: Alexander Sverdlin <alexander.sverdlin at siemens.com>
> ---
>  include/sdhci.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/sdhci.h b/include/sdhci.h
> index d73a725609be3..810ef56e4be66 100644
> --- a/include/sdhci.h
> +++ b/include/sdhci.h
> @@ -300,7 +300,7 @@ struct sdhci_ops {
>  
>  #define ADMA_MAX_LEN	65532
>  #ifdef CONFIG_DMA_ADDR_T_64BIT
> -#define ADMA_DESC_LEN	16
> +#define ADMA_DESC_LEN	12
>  #else
>  #define ADMA_DESC_LEN	8
>  #endif
> -- 
> 2.44.0

on TI AM64 and AM62p platforms:

Tested-by: Jonathan Humphreys <j-humphreys at ti.com>


More information about the U-Boot mailing list