Re: [PATCH v8 00/17] Introduce a unified API for SCMI Server testing

From: Sudeep Holla
Date: Fri Jan 20 2023 - 06:38:10 EST


On Fri, Jan 20, 2023 at 09:56:32AM +0000, Cristian Marussi wrote:
> On Fri, Jan 20, 2023 at 09:11:24AM +0100, Vincent Guittot wrote:
>
> Hi Vincent,
>
> thanks for trying this out.
>
> > I haven't seen a description of the <M>. I figured out that this is
> > the protocol id to which the channel was associated in DT while
> > testing it but it could be good to describe this somewhere.
>
> Ah, damn yes, I had made a note to myself to add an explicit description
> of how the channel number IDs are chosen, then I forgot :P
>
> I'll add a follow-on patch once is queued.
>

Please post it ASAP so that I can include it before I tag, just trying
to avoid missing it now and sending it as fix later :).

> > Apart from this minor thing, I have tested it with an scmi server
> > embedded in OPTEE and run the scmi compliance tests. Everything works
> > fine using the default mode or using one specific channel.
> >
> > FWIW
> > Tested-by: Vincent Guittot <vincent.guittot@xxxxxxxxxx>

Thanks Vincent and Florian. I just pulled and merged this last night, will
update to include your Tested-by tags.

--
Regards,
Sudeep