From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Date: Fri, 26 Sep 2014 17:52:29 +0000 Subject: Re: [PATCH v3 02/13] PM / Domains: Add DT bindings for power-on/off latencies Message-Id: List-Id: References: <1411662520-22795-1-git-send-email-geert+renesas@glider.be> <1411662520-22795-3-git-send-email-geert+renesas@glider.be> <20140926152835.GK10233@codeaurora.org> In-Reply-To: <20140926152835.GK10233@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org Hi Stephen, On Fri, Sep 26, 2014 at 5:28 PM, Stephen Boyd wrote: >> On 25 September 2014 18:28, Geert Uytterhoeven wrote: >> > +Optional properties: >> > + - power-on-latency: Power-on latency of the PM domain, in ns, >> > + - power-off-latency: Power-off latency of the PM domain, in ns. >> > + >> > Example: >> > >> > power: power-controller@12340000 { >> > compatible = "foo,power-controller"; >> > reg = <0x12340000 0x1000>; >> > #power-domain-cells = <1>; >> > + power-on-latency = <250000>; >> > + power-off-latency = <250000>; > > This assumes every power domain within a power controller has the > same latencies? That isn't always true. This should be an array > of values corresponding to the number of power domains if > #power-domain-cells is 1, or just one value if the cell count is > 0. Thanks, you're right. Will update. However, I'd still like to allow just one value in the #power-domain-cells = <1> case if the latencies are the same. Is that OK for you? Thanks again. 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 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755917AbaIZRwe (ORCPT ); Fri, 26 Sep 2014 13:52:34 -0400 Received: from mail-lb0-f174.google.com ([209.85.217.174]:51143 "EHLO mail-lb0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755543AbaIZRwc (ORCPT ); Fri, 26 Sep 2014 13:52:32 -0400 MIME-Version: 1.0 In-Reply-To: <20140926152835.GK10233@codeaurora.org> References: <1411662520-22795-1-git-send-email-geert+renesas@glider.be> <1411662520-22795-3-git-send-email-geert+renesas@glider.be> <20140926152835.GK10233@codeaurora.org> Date: Fri, 26 Sep 2014 19:52:29 +0200 X-Google-Sender-Auth: t7HMV5J8e0vqBKgR4fJOFhxFoD8 Message-ID: Subject: Re: [PATCH v3 02/13] PM / Domains: Add DT bindings for power-on/off latencies From: Geert Uytterhoeven To: Stephen Boyd Cc: Ulf Hansson , Geert Uytterhoeven , "Rafael J. Wysocki" , Simon Horman , Magnus Damm , Tomasz Figa , Philipp Zabel , Grygorii Strashko , Kevin Hilman , Linux-sh list , "linux-pm@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Stephen, On Fri, Sep 26, 2014 at 5:28 PM, Stephen Boyd wrote: >> On 25 September 2014 18:28, Geert Uytterhoeven wrote: >> > +Optional properties: >> > + - power-on-latency: Power-on latency of the PM domain, in ns, >> > + - power-off-latency: Power-off latency of the PM domain, in ns. >> > + >> > Example: >> > >> > power: power-controller@12340000 { >> > compatible = "foo,power-controller"; >> > reg = <0x12340000 0x1000>; >> > #power-domain-cells = <1>; >> > + power-on-latency = <250000>; >> > + power-off-latency = <250000>; > > This assumes every power domain within a power controller has the > same latencies? That isn't always true. This should be an array > of values corresponding to the number of power domains if > #power-domain-cells is 1, or just one value if the cell count is > 0. Thanks, you're right. Will update. However, I'd still like to allow just one value in the #power-domain-cells = <1> case if the latencies are the same. Is that OK for you? Thanks again. 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH v3 02/13] PM / Domains: Add DT bindings for power-on/off latencies Date: Fri, 26 Sep 2014 19:52:29 +0200 Message-ID: References: <1411662520-22795-1-git-send-email-geert+renesas@glider.be> <1411662520-22795-3-git-send-email-geert+renesas@glider.be> <20140926152835.GK10233@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140926152835.GK10233@codeaurora.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Stephen Boyd Cc: "devicetree@vger.kernel.org" , Ulf Hansson , Grygorii Strashko , Geert Uytterhoeven , Linux-sh list , "Rafael J. Wysocki" , "linux-pm@vger.kernel.org" , Magnus Damm , Tomasz Figa , "linux-kernel@vger.kernel.org" , Simon Horman , "linux-arm-kernel@lists.infradead.org" , Philipp Zabel , Kevin Hilman List-Id: devicetree@vger.kernel.org Hi Stephen, On Fri, Sep 26, 2014 at 5:28 PM, Stephen Boyd wrote: >> On 25 September 2014 18:28, Geert Uytterhoeven wrote: >> > +Optional properties: >> > + - power-on-latency: Power-on latency of the PM domain, in ns, >> > + - power-off-latency: Power-off latency of the PM domain, in ns. >> > + >> > Example: >> > >> > power: power-controller@12340000 { >> > compatible = "foo,power-controller"; >> > reg = <0x12340000 0x1000>; >> > #power-domain-cells = <1>; >> > + power-on-latency = <250000>; >> > + power-off-latency = <250000>; > > This assumes every power domain within a power controller has the > same latencies? That isn't always true. This should be an array > of values corresponding to the number of power domains if > #power-domain-cells is 1, or just one value if the cell count is > 0. Thanks, you're right. Will update. However, I'd still like to allow just one value in the #power-domain-cells = <1> case if the latencies are the same. Is that OK for you? Thanks again. 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: geert@linux-m68k.org (Geert Uytterhoeven) Date: Fri, 26 Sep 2014 19:52:29 +0200 Subject: [PATCH v3 02/13] PM / Domains: Add DT bindings for power-on/off latencies In-Reply-To: <20140926152835.GK10233@codeaurora.org> References: <1411662520-22795-1-git-send-email-geert+renesas@glider.be> <1411662520-22795-3-git-send-email-geert+renesas@glider.be> <20140926152835.GK10233@codeaurora.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Stephen, On Fri, Sep 26, 2014 at 5:28 PM, Stephen Boyd wrote: >> On 25 September 2014 18:28, Geert Uytterhoeven wrote: >> > +Optional properties: >> > + - power-on-latency: Power-on latency of the PM domain, in ns, >> > + - power-off-latency: Power-off latency of the PM domain, in ns. >> > + >> > Example: >> > >> > power: power-controller at 12340000 { >> > compatible = "foo,power-controller"; >> > reg = <0x12340000 0x1000>; >> > #power-domain-cells = <1>; >> > + power-on-latency = <250000>; >> > + power-off-latency = <250000>; > > This assumes every power domain within a power controller has the > same latencies? That isn't always true. This should be an array > of values corresponding to the number of power domains if > #power-domain-cells is 1, or just one value if the cell count is > 0. Thanks, you're right. Will update. However, I'd still like to allow just one value in the #power-domain-cells = <1> case if the latencies are the same. Is that OK for you? Thanks again. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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