On 1/29/24 23:49, Andi Shyti wrote: > Hi Linus, > > On Sat, Jan 27, 2024 at 11:31:11PM +0100, Linus Walleij wrote: >> On Fri, Jan 26, 2024 at 3:37 PM Thomas Richard >> <thomas.richard@xxxxxxxxxxx> wrote: >> >>> The goal is to extend the active period of pinctrl. >>> Some devices may need active pinctrl after suspend() and/or before >>> resume(). >>> So move suspend()/resume() to suspend_noirq()/resume_noirq() in order to >>> have active pinctrl until suspend_noirq() (included), and from >>> resume_noirq() (included). >>> >>> The deprecated API has been removed to use the new one (dev_pm_ops struct). >>> >>> Signed-off-by: Thomas Richard <thomas.richard@xxxxxxxxxxx> >> >> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> >> >> Do you want to merge this as a series or is this something I >> should just apply? > > there is still a comment from me pending. Hi Andi, Based on your comment, for the next iteration, I will move the cleanup in a dedicated patch. @Linus, you can apply pinctrl patches once everything is ok for you. Regards, -- Thomas Richard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
