From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk0-f193.google.com ([209.85.220.193]:32904 "EHLO mail-qk0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751181AbdLNKwg (ORCPT ); Thu, 14 Dec 2017 05:52:36 -0500 MIME-Version: 1.0 In-Reply-To: References: <1510588003-16650-1-git-send-email-ulf.hansson@linaro.org> <1510588003-16650-3-git-send-email-ulf.hansson@linaro.org> <1926914.PsNMcYBlGW@aspire.rjw.lan> From: Geert Uytterhoeven Date: Thu, 14 Dec 2017 11:52:35 +0100 Message-ID: Subject: Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag To: Ulf Hansson Cc: "Rafael J. Wysocki" , Linux PM list , Kevin Hilman , Viresh Kumar , Geert Uytterhoeven , Simon Horman , Niklas Soderlund , Linux-Renesas Content-Type: text/plain; charset="UTF-8" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hi Ulf, On Mon, Dec 11, 2017 at 9:59 PM, Ulf Hansson wrote: > On 11 December 2017 at 11:48, Geert Uytterhoeven wrote: >> On Mon, Dec 11, 2017 at 11:24 AM, Ulf Hansson wrote: >>> On 10 December 2017 at 11:16, Geert Uytterhoeven wrote: >>>> To complicate matters, some drivers may be used on SoCs where the device >>>> needs to be kept running (clock and/or power domain), and on SoCs where the >>>> device is always running. This difference is typically handled by genpd, >>>> and the device driver may not even be aware. Of course the driver can just >>>> set IN_BAND_WAKEUP regardless, (else it has to check for the presence of >>>> clocks and/or power-domains properties itself, duplicating genpd >>>> core/driver code). >>>> >>>> So what about >>>> >>>> if (IN_BAND_WAKEUP || >>>> (GENPD_FLAG_ACTIVE_WAKEUP && !OUT_BAND_WAKEUP)) { >>> >>> We don't want to suspend the device in case of IN_BAND_WAKEUP, right!? >>> >>>> ... suspend device... >>>> } >> >> Oops, inverted logic. I should not write technical emails on Sunday morning. >> >> Yes, the device must be kept awake if either IN_BAND_WAKEUP is set, or >> if GENPD_FLAG_ACTIVE_WAKEUP is set but OUT_BAND_WAKEUP isn't. > > Putting together the pieces of information received here, you have > convinced me that we should stick to use the current > GENPD_FLAG_ACTIVE_WAKEUP for now, which allows genpds to opt-in for > start dealing with in-band-wakeups. Thank you! So I'll move forward with "[PATCH v2 0/3] PM / Domain: renesas: Fix active wakeup behavior" (https://www.spinics.net/lists/linux-renesas-soc/msg19941.html) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds