On Thursday 12 April 2018 07:46 PM, Tony Lindgren wrote: > Hi, > > * Keerthy <j-keerthy@xxxxxx> [180412 03:56]: >> From: Russ Dill <Russ.Dill@xxxxxxxxx> >> >> This adds a pair of context save/restore functions to save/restore the >> state of a set of pinctrl registers. This simplifies some of the AM33XX >> PM code as some of the pinctrl registers are lost when the per power >> domain loses power. The pincrtl code can perform the necessary >> save/restore. > > So where's the patch adding callers to this code? This series was to get all the additional save/restores needed for the rtc+ddr mode. Caller patch is in the main patch that implements that mode that i will be posting once the ground is set. Based on enable_off_mode being set this function was called only for rtc+ddr mode. > > If this cannot be done from regular driver suspend/resume calls, > this probably should be done from cpu_pm notifiers CPU_PM_ENTER and > CPU_PM_EXIT. We should also do the same for GPIO BTW. I will look into that. Thanks for suggestion. > > BTW, the subject line is wrong here, it should be "pinctrl" :) Oops yes that should be pinctrl. > > Regards, > > Tony > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html