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

From: Jassi Brar
Date: Mon Jul 24 2017 - 11:42:01 EST


On Mon, Jul 24, 2017 at 3:20 PM, Sudeep Holla <sudeep.holla@xxxxxxx> wrote:
>
>
> On 08/07/17 06:32, Jassi Brar wrote:
>> Hi Roy, Matt, Nishant, Harb Abdulhamid, Loc,
>>
>> I have a gut feeling you guys were part of the SCMI spec committee. If
>> so, could you please chime in?
>>
>
> I take complete silence as no objection.
>
I take it as no self respecting developer/architect would stand by
your FUBAR implementation of SCMI.

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);

As the maintainer I have tried to explain it to you in a dozen ways.
If you still can't see how it's broken, you can't be helped.

Checking out until someone else has any point.
NAK.