[U-Boot] [PATCH] mmc: sdhci: don't clobber adjacent registers

Pantelis Antoniou panto at antoniou-consulting.com
Wed Mar 18 08:53:32 CET 2015


Hi Matt,

> On Feb 23, 2015, at 23:56 , Matt Reimer <mreimer at sdgsystems.com> wrote:
> 
> SDHCI_HOST_CONTROL is a byte-sized register, so don't write to it
> as if it were a long, as that would result in clobbering the three
> registers following.
> 
> Signed-off-by: Matt Reimer <mreimer at sdgsystems.com>
> ---
> drivers/mmc/sdhci.c |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
> index 82d7984..1f8917b 100644
> --- a/drivers/mmc/sdhci.c
> +++ b/drivers/mmc/sdhci.c
> @@ -412,7 +412,7 @@ static int sdhci_init(struct mmc *mmc)
> 	if (host->quirks & SDHCI_QUIRK_NO_CD) {
> 		unsigned int status;
> 
> -		sdhci_writel(host, SDHCI_CTRL_CD_TEST_INS | SDHCI_CTRL_CD_TEST,
> +		sdhci_writeb(host, SDHCI_CTRL_CD_TEST_INS | SDHCI_CTRL_CD_TEST,
> 			SDHCI_HOST_CONTROL);
> 
> 		status = sdhci_readl(host, SDHCI_PRESENT_STATE);
> -- 
> 1.7.9.5
> 

Thanks, applied.

— Pantelis



More information about the U-Boot mailing list