On Fri, Jun 26, 2020 at 06:32:21PM +0900, Yoshihiro Shimoda wrote: > If regulator_is_enabled() of both vmmc and vqmmc returns false, > _mmc_suspend() should call mmc_poweroff_nofity() instead of > mmc_sleep(). This is possibly something it makes sense to do, if the power did suddenly vanish on the device then using a separate cleanup path for that seems sensible (it's probably something worth complaining loudly about). Registering for notification on power loss might also be sensible. > Note that this is possible to happen when the regulator-fixed driver > turns the vmmc and vqmmc off by firmware like PSCI while the system > is suspended. This is not a good interface, if there's a need to query the state over suspend then we should query the state over suspend rather than trying to somehow shoehorn it via the runtime enable state which is going to break any other users and relies on the regulator driver doing dodgy stuff representing the enable state.
Attachment:
signature.asc
Description: PGP signature
