From: Jassi Brar <jassisinghbrar@gmail.com> To: Sudeep Holla <sudeep.holla@arm.com> Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Liviu Dudau <Liviu.Dudau@arm.com>, Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>, "Jon Medhurst (Tixy)" <tixy@linaro.org>, Rob Herring <robh+dt@kernel.org>, Mark Rutland <Mark.Rutland@arm.com>, Devicetree List <devicetree@vger.kernel.org> Subject: Re: [PATCH 1/4] mailbox: add support for System Control and Power Interface(SCPI) protocol Date: Thu, 14 May 2015 12:32:55 +0530 [thread overview] Message-ID: <CABb+yY1083thUrtKfMKg1ZP2=d3Jr4hqRczqejpAkba+40UnkA@mail.gmail.com> (raw) In-Reply-To: <55538540.1010505@arm.com> On Wed, May 13, 2015 at 10:39 PM, Sudeep Holla <sudeep.holla@arm.com> wrote: > On 13/05/15 17:52, Jassi Brar wrote: >> >>> This patch adds support for System Control and Power Interface (SCPI) >>> Message Protocol used between the Application Cores(AP) and the System >>> Control Processor(SCP). The MHU peripheral provides a mechanism for >>> inter-processor communication between SCP's M3 processor and AP. >>> >>> SCP offers control and management of the core/cluster power states, >>> various power domain DVFS including the core/cluster, certain system >>> clocks configuration, thermal sensors and many others. >>> >>> This protocol driver provides interface for all the client drivers using >>> SCPI to make use of the features offered by the SCP. >>> >> Is the SCPI specification available somewhere to look into? >> > > Yes sorry posted the link separately(as reply to Tixy in the cover > letter) since it was not available when I posted the patches. > You can grab the protocol @[1] or [2] > Thanks for the link. I wish I had access to the spec earlier. >>> .../devicetree/bindings/mailbox/arm,scpi.txt | 121 ++++ >>> drivers/mailbox/Kconfig | 19 + >>> drivers/mailbox/Makefile | 2 + >>> drivers/mailbox/scpi_protocol.c | 694 >>> +++++++++++++++++++++ >>> >> Why in drivers/mailbox/ ? This is a 'consumer' driver and seems >> Juno(ARM) specific. >> > > Not just JUNO alone though it's first one to use, it will used in next > few platforms(foreseeable future) from ARM Ltd. > > I have put that in drivers/mailbox for 2 reasons: > 1. It's mailbox protocol :) > client/protocol drivers don't usually reside with controller drivers. drivers/firmware/ seems more appropriate. > 2. ARM64 doesn't have platform code like ARM32 and moreover it's > strictly not specific to JUNO or any single platform. It may > get reused on other platforms. > drivers/firmware/ should do too. BTW is scpi_protocol.c meant/tested to work over arm_mhu.c? The spec says so but I don't see how because you pass 'struct scpi_xfer*' as the message whereas arm_mhu.c expects u32*
WARNING: multiple messages have this Message-ID (diff)
From: Jassi Brar <jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> To: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> Cc: Linux Kernel Mailing List <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, Liviu Dudau <Liviu.Dudau-5wv7dgnIgG8@public.gmane.org>, Lorenzo Pieralisi <Lorenzo.Pieralisi-5wv7dgnIgG8@public.gmane.org>, "Jon Medhurst (Tixy)" <tixy-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Mark Rutland <Mark.Rutland-5wv7dgnIgG8@public.gmane.org>, Devicetree List <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> Subject: Re: [PATCH 1/4] mailbox: add support for System Control and Power Interface(SCPI) protocol Date: Thu, 14 May 2015 12:32:55 +0530 [thread overview] Message-ID: <CABb+yY1083thUrtKfMKg1ZP2=d3Jr4hqRczqejpAkba+40UnkA@mail.gmail.com> (raw) In-Reply-To: <55538540.1010505-5wv7dgnIgG8@public.gmane.org> On Wed, May 13, 2015 at 10:39 PM, Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> wrote: > On 13/05/15 17:52, Jassi Brar wrote: >> >>> This patch adds support for System Control and Power Interface (SCPI) >>> Message Protocol used between the Application Cores(AP) and the System >>> Control Processor(SCP). The MHU peripheral provides a mechanism for >>> inter-processor communication between SCP's M3 processor and AP. >>> >>> SCP offers control and management of the core/cluster power states, >>> various power domain DVFS including the core/cluster, certain system >>> clocks configuration, thermal sensors and many others. >>> >>> This protocol driver provides interface for all the client drivers using >>> SCPI to make use of the features offered by the SCP. >>> >> Is the SCPI specification available somewhere to look into? >> > > Yes sorry posted the link separately(as reply to Tixy in the cover > letter) since it was not available when I posted the patches. > You can grab the protocol @[1] or [2] > Thanks for the link. I wish I had access to the spec earlier. >>> .../devicetree/bindings/mailbox/arm,scpi.txt | 121 ++++ >>> drivers/mailbox/Kconfig | 19 + >>> drivers/mailbox/Makefile | 2 + >>> drivers/mailbox/scpi_protocol.c | 694 >>> +++++++++++++++++++++ >>> >> Why in drivers/mailbox/ ? This is a 'consumer' driver and seems >> Juno(ARM) specific. >> > > Not just JUNO alone though it's first one to use, it will used in next > few platforms(foreseeable future) from ARM Ltd. > > I have put that in drivers/mailbox for 2 reasons: > 1. It's mailbox protocol :) > client/protocol drivers don't usually reside with controller drivers. drivers/firmware/ seems more appropriate. > 2. ARM64 doesn't have platform code like ARM32 and moreover it's > strictly not specific to JUNO or any single platform. It may > get reused on other platforms. > drivers/firmware/ should do too. BTW is scpi_protocol.c meant/tested to work over arm_mhu.c? The spec says so but I don't see how because you pass 'struct scpi_xfer*' as the message whereas arm_mhu.c expects u32* -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-05-14 7:03 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-04-27 11:40 [PATCH 0/4] ARM64: add SCPI mailbox protocol, clock and CPUFreq support Sudeep Holla 2015-04-27 11:40 ` [PATCH 1/4] mailbox: add support for System Control and Power Interface(SCPI) protocol Sudeep Holla 2015-04-27 11:40 ` Sudeep Holla 2015-04-28 7:36 ` Paul Bolle 2015-04-28 8:41 ` Sudeep Holla 2015-04-28 13:54 ` Jon Medhurst (Tixy) 2015-04-29 10:53 ` Sudeep Holla 2015-04-29 11:43 ` Jon Medhurst (Tixy) 2015-04-29 12:25 ` Jon Medhurst (Tixy) 2015-04-29 13:08 ` Sudeep Holla 2015-04-30 8:49 ` Jon Medhurst (Tixy) 2015-04-29 13:01 ` Sudeep Holla 2015-05-13 16:52 ` Jassi Brar 2015-05-13 17:09 ` Sudeep Holla 2015-05-14 7:02 ` Jassi Brar [this message] 2015-05-14 7:02 ` Jassi Brar 2015-05-14 7:30 ` Jassi Brar 2015-05-14 8:25 ` Sudeep Holla 2015-05-14 8:25 ` Sudeep Holla 2015-04-27 11:40 ` [PATCH 2/4] clk: add support for clocks provided by SCP(System Control Processor) Sudeep Holla 2015-05-07 10:17 ` Lorenzo Pieralisi 2015-05-20 23:43 ` Stephen Boyd 2015-05-26 13:14 ` Sudeep Holla 2015-04-27 11:40 ` [PATCH 3/4] clk: scpi: add support for cpufreq virtual device Sudeep Holla 2015-05-20 23:45 ` Stephen Boyd 2015-05-26 13:25 ` Sudeep Holla 2015-04-27 11:40 ` [PATCH 4/4] cpufreq: arm_big_little: add SCPI interface driver Sudeep Holla 2015-04-29 5:44 ` Viresh Kumar 2015-04-29 9:39 ` Sudeep Holla 2015-05-01 13:19 ` Jon Medhurst (Tixy) 2015-05-01 13:32 ` Sudeep Holla 2015-05-01 14:12 ` Jon Medhurst (Tixy) 2015-05-01 14:15 ` Sudeep Holla 2015-05-01 17:10 ` Jon Medhurst (Tixy) 2015-05-01 17:14 ` Sudeep Holla 2015-04-27 18:11 ` [PATCH 0/4] ARM64: add SCPI mailbox protocol, clock and CPUFreq support Jon Medhurst (Tixy) 2015-04-28 8:47 ` Sudeep Holla 2015-04-28 14:21 ` Sudeep Holla
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='CABb+yY1083thUrtKfMKg1ZP2=d3Jr4hqRczqejpAkba+40UnkA@mail.gmail.com' \ --to=jassisinghbrar@gmail.com \ --cc=Liviu.Dudau@arm.com \ --cc=Lorenzo.Pieralisi@arm.com \ --cc=Mark.Rutland@arm.com \ --cc=devicetree@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=robh+dt@kernel.org \ --cc=sudeep.holla@arm.com \ --cc=tixy@linaro.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.