Messages in this thread | | | From | Linus Walleij <> | Date | Wed, 7 Aug 2019 15:11:29 +0200 | Subject | Re: [PATCH v7 01/20] pinctrl: tegra: Add suspend and resume support |
| |
On Wed, Aug 7, 2019 at 5:40 AM Sowjanya Komatineni <skomatineni@nvidia.com> wrote: > On 8/6/19 2:51 PM, Sowjanya Komatineni wrote: > > > > On 8/5/19 2:20 AM, Linus Walleij wrote: > >> On Wed, Jul 31, 2019 at 11:11 PM Sowjanya Komatineni > >> <skomatineni@nvidia.com> wrote: > >> > >>> This patch adds support for Tegra pinctrl driver suspend and resume. > >>> > >>> During suspend, context of all pinctrl registers are stored and > >>> on resume they are all restored to have all the pinmux and pad > >>> configuration for normal operation. > >>> > >>> Acked-by: Thierry Reding <treding@nvidia.com> > >>> Reviewed-by: Dmitry Osipenko <digetx@gmail.com> > >>> Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com> > >> Patch applied to the pinctrl tree. > >> > >> This patch seems finished. > >> > >> Also if the rest don't get merged for v5.4 then at least this is so > >> your patch stack gets more shallow. > >> > >> I hope it's fine to merge this separately, else tell me and I'll > >> pull it out. > >> > >> Yours, > >> Linus Walleij > > > > Yes, this patch can be merged separately. But, there's latest feedback > > from Dmitry to add barrier after writes to make sure pinmux register > > writes happen. > > > > So will update this patch to add barrier in v8. So, need to wait for v8. > > > > Thanks > > > > Sowjanya > > > I see it merged. So will exclude suspend/resume patch and will add patch > for necessary write barrier fix in v8 version.
Yeah just make an incremental patch, that's fine. If you want to overdo it you can add a Fixes: tag to the original patch, but I don't care much.
Yours, Linus Walleij
| |