Re: [PATCH 3/9] firmware: arm_scmi: add basic driver infrastructure for SCMI

From: Sudeep Holla
Date: Mon Jul 24 2017 - 13:33:20 EST




On 24/07/17 18:21, Jassi Brar wrote:
> On Mon, Jul 24, 2017 at 9:51 PM, Sudeep Holla <sudeep.holla@xxxxxxx> 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