Hi!

> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> 
> commit 183edc060e6969a3afe83f663b534f6324fb7e3a upstream.
> 
> We want to clear TAPEN in a software reset, too, to have a completely
> known state. Especially when we doing the initial reset during boot to
> clear previous firmware states.
> 

> +++ b/drivers/mmc/host/renesas_sdhi_core.c
> @@ -558,7 +558,7 @@ static void renesas_sdhi_reset(struct tmio_mmc_host *host)
>  	struct renesas_sdhi *priv = host_to_priv(host);
>  
>  	if (priv->scc_ctl) {
> -		renesas_sdhi_reset_scc(host, priv);
> +		renesas_sdhi_disable_scc(host->mmc);
>  		renesas_sdhi_reset_hs400_mode(host, priv);
>  		priv->needs_adjust_hs400 = false;

Having half of functions receive host, priv pointers and the other
half receiving host->mmc pointer is not very nice. Is there some logic
behind that / is it possible to somehow fix?

Best regards,
								Pavel
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany