On Mon, Aug 30, 2021 at 8:37 AM Zhiyong Tao <zhiyong.tao@xxxxxxxxxxxx> wrote: > > Setting this up will configure wake from suspend properly, > and wake only for the interrupts that are setup in wake_mask, > not all interrupts. This sounds like a fix. Please add a Fixes tag for this patch. This makes tracking things easier. So, Fixes: 6cf5e9ef362a ("pinctrl: add pinctrl driver on mt8195") Reviewed-by: Chen-Yu Tsai <wenst@xxxxxxxxxxxx> Also this patch is unrelated to the rest. You should send it separately. This would also help the maintainer merge it directly. > Signed-off-by: Zhiyong Tao <zhiyong.tao@xxxxxxxxxxxx> > --- > drivers/pinctrl/mediatek/pinctrl-mt8195.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/pinctrl/mediatek/pinctrl-mt8195.c b/drivers/pinctrl/mediatek/pinctrl-mt8195.c > index a7500e18bb1d..892e79703f98 100644 > --- a/drivers/pinctrl/mediatek/pinctrl-mt8195.c > +++ b/drivers/pinctrl/mediatek/pinctrl-mt8195.c > @@ -839,6 +839,7 @@ static struct platform_driver mt8195_pinctrl_driver = { > .driver = { > .name = "mt8195-pinctrl", > .of_match_table = mt8195_pinctrl_of_match, > + .pm = &mtk_paris_pinctrl_pm_ops, > }, > .probe = mt8195_pinctrl_probe, > }; > -- > 2.18.0 > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/linux-mediatek