[PATCH 2/6] arm_ffa: introduce armffa command

Simon Glass sjg at google.com
Fri Jul 28 03:52:00 CEST 2023


Hi Abdellatif,

On Thu, 27 Jul 2023 at 03:00, Abdellatif El Khlifi
<abdellatif.elkhlifi at arm.com> wrote:
>
> Hi Heinrich,
>
> On Thu, Jul 27, 2023 at 07:51:42AM +0200, Heinrich Schuchardt wrote:
> > On 3/29/22 17:16, abdellatif.elkhlifi at arm.com wrote:
> > > From: Abdellatif El Khlifi <abdellatif.elkhlifi at arm.com>
> > >
> > > Provide armffa command showcasing the use of the FF-A driver
> > >
> > > The armffa command allows to query secure partitions data from
> > > the secure world and exchanging messages with the partitions.
> > >
> > > Signed-off-by: Abdellatif El Khlifi <abdellatif.elkhlifi at arm.com>
> > > Cc: Tom Rini <trini at konsulko.com>
> > > ---
> > >   MAINTAINERS             |   1 +
> > >   cmd/Kconfig             |  10 ++
> > >   cmd/Makefile            |   2 +
> > >   cmd/armffa.c            | 266 ++++++++++++++++++++++++++++++++++++++++
> > >   drivers/arm-ffa/Kconfig |   1 +
> >
> > We want to have all commands to be documented in /doc/usage/cmd/.
> >
> > Could you, please, provide the missing /doc/usage/cmd/armffa.rst file.
> >
>
> It's here [1].

I'd suggest mentioning that in the commit message for this patch, or
even putting the testin the same patch, along with the test for the
command. I find it hard to keep track of this too.

>
> [1]: https://lore.kernel.org/all/20230726094503.100497-6-abdellatif.elkhlifi@arm.com/#Z31doc:usage:cmd:armffa.rst
>
> Cheers,
> Abdellatif

Regards,
Simon


More information about the U-Boot mailing list