> +     /* Reset HS400 mode */
> +     sd_ctrl_write16(host, CTL_SDIF_MODE, ~0x0001 &
> +                     sd_ctrl_read16(host, CTL_SDIF_MODE));
> +     sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_TMPPORT2,
> +                    ~(SH_MOBILE_SDHI_SCC_TMPPORT2_HS400EN |
> +                      SH_MOBILE_SDHI_SCC_TMPPORT2_HS400OSEL) &
> +                     sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_TMPPORT2));
> +

This looks like code duplication. Can't we simply call
renesas_sdhi_reset_hs400_mode()?

Attachment: signature.asc
Description: PGP signature

Reply via email to