> > +
> > + int (*start_signal_voltage_switch)(struct tmio_mmc_host *host,
> > + unsigned char signal_voltage);
>
> Do you really need to add a new tmio specific callback for this?
>
> Why can't you instead have the tmio variant driver assign the
> ->start_signal_voltage_switch() callback into the struct mmc_host_ops
> instead?
Just to see how it looks, I tried your suggestion. However, mmc_ops is
consted all the way into the core (rightfully, I'd say), so this makes
this approach quite clumsy. I kept the above callback now, just changed
the second paramater to pass the whole ios so I can use it with
mmc_regulator_set_vqmmc().
> > +static int tmio_mmc_card_busy(struct mmc_host *mmc)
> > +{
> > + struct tmio_mmc_host *host = mmc_priv(mmc);
> > + u32 status;
> > +
> > + pm_runtime_get_sync(mmc_dev(mmc));
>
> This isn't needed as the mmc core already deal with runtime PM of the
> host device via mmc_claim|release_host().
sdhci.c does it, too - missing cleanup?
Attachment:
signature.asc
Description: PGP signature
