From: Lina Iyer <lina.iyer@linaro.org> To: Kevin Hilman <khilman@kernel.org> Cc: Kumar Gala <galak@codeaurora.org>, sboyd@codeaurora.org, daniel.lezcano@linaro.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, msivasub@codeaurora.org, lorenzo.pieralisi@arm.com, linux-pm@vger.kernel.org Subject: Re: [PATCH v6 1/5] qcom: spm: Add Subsystem Power Manager driver Date: Fri, 26 Sep 2014 13:12:53 -0600 [thread overview] Message-ID: <20140926191253.GH390@ilina-mac.local> (raw) In-Reply-To: <7hwq8q2og3.fsf@deeprootsystems.com> On Fri, Sep 26 2014 at 13:04 -0600, Kevin Hilman wrote: >Lina Iyer <lina.iyer@linaro.org> writes: > >> On Wed, Sep 24 2014 at 12:07 -0600, Kumar Gala wrote: >>> >>>On Sep 23, 2014, at 6:51 PM, Lina Iyer <lina.iyer@linaro.org> wrote: >>> >>>> Based on work by many authors, available at codeaurora.org >>>> >>>> SPM is a hardware block that controls the peripheral logic surrounding >>>> the application cores (cpu/l$). When the core executes WFI instruction, >>>> the SPM takes over the putting the core in low power state as >>>> configured. The wake up for the SPM is an interrupt at the GIC, which >>>> then completes the rest of low power mode sequence and brings the core >>>> out of low power mode. >>>> >>>> The SPM has a set of control registers that configure the SPMs >>>> individually based on the type of the core and the runtime conditions. >>>> SPM is a finite state machine block to which a sequence is provided and >>>> it interprets the bytes and executes them in sequence. Each low power >>>> mode that the core can enter into is provided to the SPM as a sequence. >>>> >>>> Configure the SPM to set the core (cpu or L2) into its low power mode, >>>> the index of the first command in the sequence is set in the SPM_CTL >>>> register. When the core executes ARM wfi instruction, it triggers the >>>> SPM state machine to start executing from that index. The SPM state >>>> machine waits until the interrupt occurs and starts executing the rest >>>> of the sequence until it hits the end of the sequence. The end of the >>>> sequence jumps the core out of its low power mode. >>>> >>>> Signed-off-by: Lina Iyer <lina.iyer@linaro.org> >>>> [lina: simplify the driver for initial submission, clean up and update >>>> commit text] >>>> --- >>>> Documentation/devicetree/bindings/arm/msm/spm.txt | 43 +++ >>>> drivers/soc/qcom/Kconfig | 8 + >>>> drivers/soc/qcom/Makefile | 1 + >>>> drivers/soc/qcom/spm.c | 388 ++++++++++++++++++++++ >>>> include/soc/qcom/spm.h | 38 +++ >>>> 5 files changed, 478 insertions(+) >>>> create mode 100644 Documentation/devicetree/bindings/arm/msm/spm.txt >>>> create mode 100644 drivers/soc/qcom/spm.c >>>> create mode 100644 include/soc/qcom/spm.h >>> >>>General comment, lets use qcom instead of msm for various things. >>> >>>[snip] >>> >> OK, Done. I renamed all msm_ functions to qcom_ functions as well. >> > >Does that apply to the other parts of this series too? like the msm-pm >and cpuidle layers? > Yes, there are no more msm's anymore :) >Kevin > >
WARNING: multiple messages have this Message-ID (diff)
From: lina.iyer@linaro.org (Lina Iyer) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v6 1/5] qcom: spm: Add Subsystem Power Manager driver Date: Fri, 26 Sep 2014 13:12:53 -0600 [thread overview] Message-ID: <20140926191253.GH390@ilina-mac.local> (raw) In-Reply-To: <7hwq8q2og3.fsf@deeprootsystems.com> On Fri, Sep 26 2014 at 13:04 -0600, Kevin Hilman wrote: >Lina Iyer <lina.iyer@linaro.org> writes: > >> On Wed, Sep 24 2014 at 12:07 -0600, Kumar Gala wrote: >>> >>>On Sep 23, 2014, at 6:51 PM, Lina Iyer <lina.iyer@linaro.org> wrote: >>> >>>> Based on work by many authors, available at codeaurora.org >>>> >>>> SPM is a hardware block that controls the peripheral logic surrounding >>>> the application cores (cpu/l$). When the core executes WFI instruction, >>>> the SPM takes over the putting the core in low power state as >>>> configured. The wake up for the SPM is an interrupt at the GIC, which >>>> then completes the rest of low power mode sequence and brings the core >>>> out of low power mode. >>>> >>>> The SPM has a set of control registers that configure the SPMs >>>> individually based on the type of the core and the runtime conditions. >>>> SPM is a finite state machine block to which a sequence is provided and >>>> it interprets the bytes and executes them in sequence. Each low power >>>> mode that the core can enter into is provided to the SPM as a sequence. >>>> >>>> Configure the SPM to set the core (cpu or L2) into its low power mode, >>>> the index of the first command in the sequence is set in the SPM_CTL >>>> register. When the core executes ARM wfi instruction, it triggers the >>>> SPM state machine to start executing from that index. The SPM state >>>> machine waits until the interrupt occurs and starts executing the rest >>>> of the sequence until it hits the end of the sequence. The end of the >>>> sequence jumps the core out of its low power mode. >>>> >>>> Signed-off-by: Lina Iyer <lina.iyer@linaro.org> >>>> [lina: simplify the driver for initial submission, clean up and update >>>> commit text] >>>> --- >>>> Documentation/devicetree/bindings/arm/msm/spm.txt | 43 +++ >>>> drivers/soc/qcom/Kconfig | 8 + >>>> drivers/soc/qcom/Makefile | 1 + >>>> drivers/soc/qcom/spm.c | 388 ++++++++++++++++++++++ >>>> include/soc/qcom/spm.h | 38 +++ >>>> 5 files changed, 478 insertions(+) >>>> create mode 100644 Documentation/devicetree/bindings/arm/msm/spm.txt >>>> create mode 100644 drivers/soc/qcom/spm.c >>>> create mode 100644 include/soc/qcom/spm.h >>> >>>General comment, lets use qcom instead of msm for various things. >>> >>>[snip] >>> >> OK, Done. I renamed all msm_ functions to qcom_ functions as well. >> > >Does that apply to the other parts of this series too? like the msm-pm >and cpuidle layers? > Yes, there are no more msm's anymore :) >Kevin > >
next prev parent reply other threads:[~2014-09-26 19:12 UTC|newest] Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-09-23 23:51 [PATCH v6 0/5] QCOM 8074 cpuidle driver Lina Iyer 2014-09-23 23:51 ` Lina Iyer 2014-09-23 23:51 ` [PATCH v6 1/5] qcom: spm: Add Subsystem Power Manager driver Lina Iyer 2014-09-23 23:51 ` Lina Iyer 2014-09-24 1:58 ` Lina Iyer 2014-09-24 1:58 ` Lina Iyer 2014-09-24 16:33 ` Kumar Gala 2014-09-24 16:33 ` Kumar Gala 2014-09-24 17:21 ` Lina Iyer 2014-09-24 17:21 ` Lina Iyer 2014-09-24 17:53 ` Kumar Gala 2014-09-24 17:53 ` Kumar Gala 2014-09-24 19:29 ` Lina Iyer 2014-09-24 19:29 ` Lina Iyer 2014-09-26 14:45 ` Lina Iyer 2014-09-26 14:45 ` Lina Iyer 2014-09-26 14:53 ` Kumar Gala 2014-09-26 14:53 ` Kumar Gala 2014-09-26 15:07 ` Lina Iyer 2014-09-26 15:07 ` Lina Iyer 2014-09-24 17:43 ` Josh Cartwright 2014-09-24 17:43 ` Josh Cartwright 2014-09-24 19:01 ` Lina Iyer 2014-09-24 19:01 ` Lina Iyer 2014-09-24 18:07 ` Kumar Gala 2014-09-24 18:07 ` Kumar Gala 2014-09-24 18:47 ` Lina Iyer 2014-09-24 18:47 ` Lina Iyer 2014-09-26 19:04 ` Kevin Hilman 2014-09-26 19:04 ` Kevin Hilman 2014-09-26 19:12 ` Lina Iyer [this message] 2014-09-26 19:12 ` Lina Iyer 2014-09-26 19:30 ` Kevin Hilman 2014-09-26 19:30 ` Kevin Hilman 2014-09-26 16:59 ` Kevin Hilman 2014-09-26 16:59 ` Kevin Hilman 2014-09-26 17:19 ` Lina Iyer 2014-09-26 17:19 ` Lina Iyer 2014-09-23 23:51 ` [PATCH v6 2/5] arm: dts: qcom: Add SPM device bindings for 8974 Lina Iyer 2014-09-23 23:51 ` Lina Iyer 2014-09-24 6:18 ` Pramod Gurav 2014-09-24 6:18 ` Pramod Gurav 2014-09-24 13:49 ` Lina Iyer 2014-09-24 13:49 ` Lina Iyer 2014-09-24 14:03 ` Kumar Gala 2014-09-24 14:03 ` Kumar Gala 2014-09-24 14:13 ` Lina Iyer 2014-09-24 14:13 ` Lina Iyer 2014-09-24 17:21 ` Stephen Boyd 2014-09-24 17:21 ` Stephen Boyd 2014-09-24 17:23 ` Lina Iyer 2014-09-24 17:23 ` Lina Iyer 2014-09-24 21:46 ` Stephen Boyd 2014-09-24 21:46 ` Stephen Boyd 2014-09-23 23:51 ` [PATCH v6 3/5] qcom: msm-pm: Add cpu low power mode functions Lina Iyer 2014-09-23 23:51 ` Lina Iyer 2014-09-23 23:51 ` [PATCH v6 4/5] qcom: cpuidle: Add cpuidle driver for QCOM cpus Lina Iyer 2014-09-23 23:51 ` Lina Iyer 2014-09-23 23:51 ` [PATCH v6 5/5] arm: dts: qcom: Add idle states device nodes for 8974 Lina Iyer 2014-09-23 23:51 ` Lina Iyer
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20140926191253.GH390@ilina-mac.local \ --to=lina.iyer@linaro.org \ --cc=daniel.lezcano@linaro.org \ --cc=galak@codeaurora.org \ --cc=khilman@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=lorenzo.pieralisi@arm.com \ --cc=msivasub@codeaurora.org \ --cc=sboyd@codeaurora.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.