> > + host->ops.prepare_hs400_tuning = renesas_sdhi_prepare_hs400_tuning; > > + host->ops.hs400_downgrade = renesas_sdhi_disable_scc; > > Would it make sens to rename renesas_sdhi_disable_scc() to > renesas_sdhi_hs400_downgrade() to fit the pattern that it's called from > the mmc_ops? I also thought about it and came to the conclusion that this is more readable. To downgrade from HS400, we need to disable SCC. But no big deal to change it if you/others think the other pattern...