On Tue, Jun 29, 2021 at 1:47 PM Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > > +static void mmci_hw_reset(struct mmc_host *mmc) > > +{ > > + struct mmci_host *host = mmc_priv(mmc); > > + > > + reset_control_assert(host->rst); > > + udelay(2); > > + reset_control_deassert(host->rst); > > +} > > Unfortunately, this is not the purpose of the ->hw_reset() ops. > Instead, it's supposed to be used to make a HW reset of the card, not > the controller. OK I see, let's skip this patch. I think patch 1 & 3 should be fine though? Yours, Linus Walleij