On Wed, Mar 22, 2017 at 08:24:32AM +0100, Linus Walleij wrote: > On Mon, Mar 20, 2017 at 12:51 PM, Charles Keepax > <ckeepax@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > > On Mon, Mar 20, 2017 at 10:13:52AM +0100, Linus Walleij wrote: > > >> Mark: I was thinking about adding runtime PM for disabling > >> these regulators when unused, but I'm uncertain about the > >> interaction with DAPM in that regard. This atleast gives us > >> control over the supplies. > > > > DAPM will hold a pm_runtime reference whilst the chip is active > > so usually there isn't really much interaction to worry about. > > OK, so since the codec is registered on i2c's client->dev I take > it I could use the runtime PM / suspend/resume callbacks on > that device, and it should just work. > Yeah should do, wm8962 should provide a reasonably similar model if you are looking for a handy example and feel free to drop me a line if I can be of any help. Thanks, Charles -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html