On Thu, Mar 19, 2015 at 11:30 AM, Thomas Petazzoni <[email protected]> wrote:
> This commit adds suspend/resume support to the Armada XP pinctrl > driver, by simply saving and restoring the MPP registers. > > Signed-off-by: Thomas Petazzoni <[email protected]> Patch applied, but: > static void __iomem *mpp_base; > +static u32 *mpp_saved_regs; I didn't notice this singleton design pattern in this driver before :( I see the legacy Dove driver have a few more. Can we have a mvebu refactoring patch bringing these things into a state container? Like a per-subdriver kmalloc:ed struct foo_state {}? Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
