[PATCH v2 0/5] TEE: minor cleanup
Tom Rini
trini at konsulko.com
Mon Mar 4 17:08:17 CET 2024
On Mon, Mar 04, 2024 at 04:50:48PM +0100, Igor Opaniuk wrote:
> Hi Ilias,
>
> On Mon, Mar 4, 2024 at 12:16 PM Ilias Apalodimas <
> ilias.apalodimas at linaro.org> wrote:
>
> > Hi Igor,
> >
> > On Sun, 3 Mar 2024 at 00:01, Igor Opaniuk <igor.opaniuk at gmail.com> wrote:
> > >
> > >
> > > - Address some spelling errors and typos
> > > - Support CMD_OPTEE_RPMB for SANDBOX configurations and add python tests
> > > - Remove common.h inclusion for drivers/tee
> > >
> > > Changes in v2:
> > > - Fixed chimp_optee.c:37:9: error: implicit declaration of function
> > 'memset'
> > > - Applied R-b and T-b tags
> >
> > https://source.denx.de/u-boot/custodians/u-boot-tpm/-/pipelines/19808
> > this seems to have a few failures
> >
> > Cheers
> > /Ilias
> > >
> > > Igor Opaniuk (5):
> > > tee: optee: fix description in Kconfig
> > > tee: sandbox: fix spelling errors
> > > cmd: optee_rpmb: build cmd for sandbox
> > > test: py: add optee_rpmb tests
> > > tee: remove common.h inclusion
> > >
> > > cmd/Kconfig | 4 +++-
> > > drivers/tee/broadcom/chimp_optee.c | 3 ++-
> > > drivers/tee/optee/Kconfig | 2 +-
> > > drivers/tee/optee/core.c | 1 -
> > > drivers/tee/optee/i2c.c | 1 -
> > > drivers/tee/optee/rpmb.c | 1 -
> > > drivers/tee/optee/supplicant.c | 2 +-
> > > drivers/tee/sandbox.c | 10 +++++-----
> > > drivers/tee/tee-uclass.c | 1 -
> > > test/py/tests/test_optee_rpmb.py | 20 ++++++++++++++++++++
> > > 10 files changed, 32 insertions(+), 13 deletions(-)
> > > create mode 100644 test/py/tests/test_optee_rpmb.py
> > >
> > > --
> > > 2.34.1
> > >
> >
>
> It looks like it's a side effect, the test I added revealed a bug in
> "cmd/optee_rpmb.c" implementation, which I didn't touch (looks
> like it doesn't close the tee session automatically).
>
> I'll address it and add a fix to the patch series.
>
> Just a quick question, are Azure pipelines still used (so I can configure
> my own account and run all these before sending patches, as explained
> in [1])
>
> [1] https://docs.u-boot.org/en/latest/develop/ci_testing.html
Yes _but_ sadly sometimes the tests fail and need to be re-run due to
being (I believe) on the free class of machines and so they're a little
too slow and we race condition fail perhaps.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240304/3ea65ab6/attachment.sig>
More information about the U-Boot
mailing list