From: Sudeep Holla <sudeep.holla@arm.com>
To: Jassi Brar <jassisinghbrar@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
Rob Herring <robh@kernel.org>, Nishanth Menon <nm@ti.com>,
"arnd@arndb.de" <arnd@arndb.de>,
Harb Abdulhamid <harba@codeaurora.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Roy Franz <roy.franz@cavium.com>,
Jassi Brar <jaswinder.singh@linaro.org>,
DTML <devicetree@vger.kernel.org>, Loc Ho <lho@apm.com>,
Matt Sealey <neko@bakuhatsu.net>,
Sudeep Holla <sudeep.holla@arm.com>,
Alexey Klimov <alexey.klimov@arm.com>,
ALKML <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 3/9] firmware: arm_scmi: add basic driver infrastructure for SCMI
Date: Mon, 24 Jul 2017 18:30:41 +0100 [thread overview]
Message-ID: <39eeddc1-99fb-092a-a833-cf4024695d87@arm.com> (raw)
In-Reply-To: <CABb+yY0tX3kgAPTYOSqVMXwVPXFBYGwdwo5gy+TgdtRpZkwx6Q@mail.gmail.com>
On 24/07/17 18:21, Jassi Brar wrote:
> On Mon, Jul 24, 2017 at 9:51 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>> On 24/07/17 16:41, Jassi Brar wrote:
>
>>> SCMI calls
>>> mbox_send_message(struct mbox_chan *chan, struct scmi_xfer *xfer);
>>>
>>> whereas the API expects
>>> mbox_send_message(struct mbox_chan *chan, struct controller_specific *xfer);
>>>
>>
>> That's not hard to change ...
>
> Cool, please change it and lets move on.
>
But tell me why first ? You said it needs to work with any controller.
And now you say I need to change it to some controller_specific ptr,
won't that make it controller specific protocol ?
You responded on something you wanted ignoring all the other parts,
still waiting for those response.
1. Tell me how my changes to MHU and SCMI are related ?
2. How my MHU changes and PCC are different ? Why do you think
this SCMI is FUBAR design which the clients using PCC are not ?
--
Regards,
Sudeep
next prev parent reply other threads:[~2017-07-24 17:30 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-26 15:55 [PATCH 0/9] firmware: ARM System Control and Management Interface(SCMI) support Sudeep Holla
2017-06-26 15:55 ` [PATCH 1/9] Documentation: dt-bindings: add support for mailbox client shared memory Sudeep Holla
[not found] ` <1498492513-29771-2-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2017-06-28 22:56 ` Rob Herring
2017-06-30 10:24 ` Sudeep Holla
2017-06-26 15:55 ` [PATCH 4/9] firmware: arm_scmi: add common infrastructure and support for base protocol Sudeep Holla
[not found] ` <1498492513-29771-1-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2017-06-26 15:55 ` [PATCH 2/9] Documentation: add DT binding for ARM System Control and Management Interface(SCMI) protocol Sudeep Holla
[not found] ` <1498492513-29771-3-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2017-06-28 23:04 ` Rob Herring
2017-06-30 10:31 ` Sudeep Holla
2017-06-26 15:55 ` [PATCH 3/9] firmware: arm_scmi: add basic driver infrastructure for SCMI Sudeep Holla
[not found] ` <1499446341-5956-1-git-send-email-jaswinder.singh@linaro.org>
[not found] ` <1499446341-5956-1-git-send-email-jaswinder.singh-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-07-07 17:39 ` Sudeep Holla
2017-07-08 5:32 ` Jassi Brar
[not found] ` <CABb+yY3Xo-meg5bTZ5Cj1DAvf_0yOw0QB5Kq6U7VG+L_L+Mo3w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-07-24 9:50 ` Sudeep Holla
[not found] ` <f165d171-a55b-662f-888c-3be2c3e8f11a-5wv7dgnIgG8@public.gmane.org>
2017-07-24 15:41 ` Jassi Brar
[not found] ` <CABb+yY3NNUE7YXAhBLpMzVZkB7Kgx6iZk8a--crHa9Wpw6Ls4Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-07-24 16:21 ` Sudeep Holla
2017-07-24 17:21 ` Jassi Brar
2017-07-24 17:30 ` Sudeep Holla [this message]
2017-06-26 15:55 ` [PATCH 5/9] firmware: arm_scmi: add initial support for performance protocol Sudeep Holla
2017-06-26 15:55 ` [PATCH 6/9] firmware: arm_scmi: add initial support for clock protocol Sudeep Holla
2017-06-26 15:55 ` [PATCH 7/9] firmware: arm_scmi: add initial support for power protocol Sudeep Holla
2017-06-26 15:55 ` [PATCH 8/9] firmware: arm_scmi: add initial support for sensor protocol Sudeep Holla
2017-06-26 15:55 ` [PATCH 9/9] firmware: arm_scmi: probe and initialise all the supported protocols 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=39eeddc1-99fb-092a-a833-cf4024695d87@arm.com \
--to=sudeep.holla@arm.com \
--cc=alexey.klimov@arm.com \
--cc=arnd@arndb.de \
--cc=devicetree@vger.kernel.org \
--cc=harba@codeaurora.org \
--cc=jassisinghbrar@gmail.com \
--cc=jaswinder.singh@linaro.org \
--cc=lho@apm.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=neko@bakuhatsu.net \
--cc=nm@ti.com \
--cc=robh@kernel.org \
--cc=roy.franz@cavium.com \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).