linux-kernel - Re: [PATCH V2] mfd: sprd: Add wakeup capability for PMIC irq

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20200605060904.GB3714@dell>
Date:   Fri, 5 Jun 2020 07:09:04 +0100
From:   Lee Jones <lee.jones@...aro.org>
To:     Chunyan Zhang <zhang.lyra@...il.com>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Orson Zhai <orsonzhai@...il.com>,
        Baolin Wang <baolin.wang7@...il.com>,
        Chunyan Zhang <chunyan.zhang@...soc.com>
Subject: Re: [PATCH V2] mfd: sprd: Add wakeup capability for PMIC irq

On Fri, 05 Jun 2020, Chunyan Zhang wrote:

> On Thu, 4 Jun 2020 at 19:16, Lee Jones <lee.jones@...aro.org> wrote:
> >
> > On Thu, 04 Jun 2020, Chunyan Zhang wrote:
> >
> > > Hi Lee,
> > >
> > > On Wed, 27 May 2020 at 14:21, Chunyan Zhang <zhang.lyra@...il.com> wrote:
> > > >
> > > > From: Baolin Wang <baolin.wang7@...il.com>
> > > >
> > > > When changing to use suspend-to-idle to save power, the PMIC irq can not
> > > > wakeup the system due to lack of wakeup capability, which will cause
> > > > the sub-irqs (such as power key) of the PMIC can not wake up the system.
> > > > Thus we can add the wakeup capability for PMIC irq to solve this issue,
> > > > as well as removing the IRQF_NO_SUSPEND flag to allow PMIC irq to be
> > > > a wakeup source.
> > > >
> > > > Reported-by: Chunyan Zhang <zhang.lyra@...il.com>
> > > > Signed-off-by: Baolin Wang <baolin.wang7@...il.com>
> > > > Tested-by: Chunyan Zhang <chunyan.zhang@...soc.com>
> > > > ---
> > > > Changes from v1:
> > > > * addressed comments from Lee;
> > > > * added tested-by from Chunyan.
> > > > (This patch is rebased on branch for-mfd-next)
> > >
> > > Could you please pick up this patch if there's no more comments :)
> >
> > Please don't send contentless nags.  Your patch is in my queue.
> >
> > The merge-window is currently open, which is a) a busy time for
> 
> I understand that very much, and also appreciate all maintainer's
> works for the great opensource project.
> And just because of that I know you're busy during merge-window, I was
> worried that this patch was left out, since I didn't see it at
> mfd-next.

It was left out, as it was sent during -rc7.  Very few patches sent to
late in the release cycle are applied during this time, since it
leaves too little time to soak in -next.

The patch should make it into the next kernel release (v5.9) without
issue.

> I apologize if the last email made you unhappay. I will not send
> reminder email to you anymore.

If more than 2 weeks have passed and you genuinely believe your patch
has been forgotten.  The correct means to solve the issue is to submit
a [RESEND].

> > Maintainers and b) a time where some of us take a little breather
> > before the next release.  Please wait until -rc1 has been released
> > before sending out your [RESENDS].

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog