From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH/RFC] PM / Domains: Minimize latencies by not delaying save/restore Date: Mon, 11 May 2015 10:00:09 -0700 Message-ID: <7hbnhrdp7q.fsf@deeprootsystems.com> References: <1431350673-871-1-git-send-email-ulf.hansson@linaro.org> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from mail-pd0-f169.google.com ([209.85.192.169]:36028 "EHLO mail-pd0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751980AbbEKRAN (ORCPT ); Mon, 11 May 2015 13:00:13 -0400 Received: by pdea3 with SMTP id a3so152425915pde.3 for ; Mon, 11 May 2015 10:00:13 -0700 (PDT) In-Reply-To: <1431350673-871-1-git-send-email-ulf.hansson@linaro.org> (Ulf Hansson's message of "Mon, 11 May 2015 15:24:33 +0200") Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Ulf Hansson Cc: "Rafael J. Wysocki" , linux-pm@vger.kernel.org, Len Brown , Pavel Machek , Geert Uytterhoeven , Lina Iyer , Dmitry Torokhov , Axel Haslam , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org Ulf Hansson writes: [...] > To address these two issues, let's change the policy in genpd. More > precisely, let's walk the hierarchy of the ->runtime_suspend() > callbacks immediately when genpd's ->runtime_suspend() callback at the > PM domain level is invoked. To help in understanding this change, you might also consider leading with something like this paragraph. IOW, start with an intro pargraph about *what* you are changing, and then go into the why. The current changlog goes into great detail about "why" before ever explaining the "what". Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@kernel.org (Kevin Hilman) Date: Mon, 11 May 2015 10:00:09 -0700 Subject: [PATCH/RFC] PM / Domains: Minimize latencies by not delaying save/restore In-Reply-To: <1431350673-871-1-git-send-email-ulf.hansson@linaro.org> (Ulf Hansson's message of "Mon, 11 May 2015 15:24:33 +0200") References: <1431350673-871-1-git-send-email-ulf.hansson@linaro.org> Message-ID: <7hbnhrdp7q.fsf@deeprootsystems.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Ulf Hansson writes: [...] > To address these two issues, let's change the policy in genpd. More > precisely, let's walk the hierarchy of the ->runtime_suspend() > callbacks immediately when genpd's ->runtime_suspend() callback at the > PM domain level is invoked. To help in understanding this change, you might also consider leading with something like this paragraph. IOW, start with an intro pargraph about *what* you are changing, and then go into the why. The current changlog goes into great detail about "why" before ever explaining the "what". Kevin