From: Sudeep Holla <sudeep.holla@arm.com> To: Jassi Brar <jassisinghbrar@gmail.com> Cc: Sudeep Holla <sudeep.holla@arm.com>, 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 09:25:24 +0100 [thread overview] Message-ID: <55545BF4.9090809@arm.com> (raw) In-Reply-To: <CABb+yY0J1j=wMaKj+1nohU7qJpVS83BD_AX3Mf56T5+6V6+NOA@mail.gmail.com> On 14/05/15 08:30, Jassi Brar wrote: > On Thu, May 14, 2015 at 12:32 PM, Jassi Brar <jassisinghbrar@gmail.com> wrote: >> >> 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* >> Yes it's tested using arm_mhu.c, and I have even sent updates to the binding that's incomplete as of now and *must* be pulled into v4.1. Please make sure it gets in. Otherwise clocks are optional but the driver fails to probe without that. I was initially wondering why the MHU probe is not called. scpi_xfer has the slot as first element which will have the right doorbell bit(in this case slot#0) set always. > It seems your remote doesn't interpret the value in STAT register... > so it just works. Not exactly. If you look at Figure 2-1 in the spec, it shows how STAT (along with SET/CLEAR) is used to identify the protocol. The remote can implement multiple protocol. E.g. SCPI(on Slot#0 - main topic of this series), ACPI(PCC/CPPC say on Slot#1), ABC(on Slot#X)...etc. > However the SCPI spec recommends seeing STAT register as '31 slots' > ... maybe we should try to support that. > Correct but only Slot#0 is used for SCPI. Regards, Sudeep
WARNING: multiple messages have this Message-ID (diff)
From: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> To: Jassi Brar <jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org>, 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 09:25:24 +0100 [thread overview] Message-ID: <55545BF4.9090809@arm.com> (raw) In-Reply-To: <CABb+yY0J1j=wMaKj+1nohU7qJpVS83BD_AX3Mf56T5+6V6+NOA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> On 14/05/15 08:30, Jassi Brar wrote: > On Thu, May 14, 2015 at 12:32 PM, Jassi Brar <jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: >> >> 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* >> Yes it's tested using arm_mhu.c, and I have even sent updates to the binding that's incomplete as of now and *must* be pulled into v4.1. Please make sure it gets in. Otherwise clocks are optional but the driver fails to probe without that. I was initially wondering why the MHU probe is not called. scpi_xfer has the slot as first element which will have the right doorbell bit(in this case slot#0) set always. > It seems your remote doesn't interpret the value in STAT register... > so it just works. Not exactly. If you look at Figure 2-1 in the spec, it shows how STAT (along with SET/CLEAR) is used to identify the protocol. The remote can implement multiple protocol. E.g. SCPI(on Slot#0 - main topic of this series), ACPI(PCC/CPPC say on Slot#1), ABC(on Slot#X)...etc. > However the SCPI spec recommends seeing STAT register as '31 slots' > ... maybe we should try to support that. > Correct but only Slot#0 is used for SCPI. Regards, Sudeep -- 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 8:25 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 2015-05-14 7:02 ` Jassi Brar 2015-05-14 7:30 ` Jassi Brar 2015-05-14 8:25 ` Sudeep Holla [this message] 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=55545BF4.9090809@arm.com \ --to=sudeep.holla@arm.com \ --cc=Liviu.Dudau@arm.com \ --cc=Lorenzo.Pieralisi@arm.com \ --cc=Mark.Rutland@arm.com \ --cc=devicetree@vger.kernel.org \ --cc=jassisinghbrar@gmail.com \ --cc=linux-kernel@vger.kernel.org \ --cc=robh+dt@kernel.org \ --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.