From: Ilias Apalodimas <ilias.apalodimas@linaro.org>
To: Igor Opaniuk <igor.opaniuk@gmail.com>
Cc: AKASHI Takahiro <takahiro.akashi@linaro.org>,
Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>,
Bin Meng <bmeng@tinylab.org>,
Etienne Carriere <etienne.carriere@foss.st.com>,
Francis Laniel <francis.laniel@amarulasolutions.com>,
Heinrich Schuchardt <xypron.glpk@gmx.de>,
Jens Wiklander <jens.wiklander@linaro.org>,
Jorge Ramirez-Ortiz <jorge@foundries.io>,
Mattijs Korpershoek <mkorpershoek@baylibre.com>,
Miquel Raynal <miquel.raynal@bootlin.com>,
Patrice Chotard <patrice.chotard@foss.st.com>,
Roger Knecht <rknecht@pm.me>,
Sean Edmond <seanedmond@microsoft.com>,
Simon Glass <sjg@chromium.org>, Tom Rini <trini@konsulko.com>,
u-boot@lists.denx.de
Subject: Re: [PATCH v3 0/6] TEE: minor cleanup
Date: Thu, 4 Apr 2024 10:22:09 +0300 [thread overview]
Message-ID: <Zg5VIfnPElreq3ls@hera> (raw)
In-Reply-To: <CAC_iWj+Uq0NZhBKuAbbM5asvUuTBku0OZDSu+b68rtqNFqFCDA@mail.gmail.com>
Hi Igor,
I tried to apply this to master and send a PR, but unfortunately it needs
rebasing. Can you rebase it and resend it?
Thanks
/Ilias
On Wed, Mar 27, 2024 at 04:17:09PM +0200, Ilias Apalodimas wrote:
> Hi Igor,
>
> On Wed, 27 Mar 2024 at 14:13, Igor Opaniuk <igor.opaniuk@gmail.com> wrote:
> >
> > Ilias,
> >
> > On Mon, Mar 4, 2024 at 6:45 PM Igor Opaniuk <igor.opaniuk@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
> >> - Add calls for closing tee session after every read/write operation.
> >>
> >> CI build:
> >> [1] https://dev.azure.com/igoropaniuk/u-boot/_build/results?buildId=28&view=results
> >>
> >> Changes in v3:
> >> - Added calls for closing tee session after every read/write operation
> >>
> >> Changes in v2:
> >> - Fixed chimp_optee.c:37:9: error: implicit declaration of function 'memset'
> >> - Applied R-b and T-b tags
> >>
> >> Igor Opaniuk (6):
> >> tee: optee: fix description in Kconfig
> >> tee: sandbox: fix spelling errors
> >> cmd: optee_rpmb: close tee session
> >> cmd: optee_rpmb: build cmd for sandbox
> >> test: py: add optee_rpmb tests
> >> tee: remove common.h inclusion
> >>
> >> cmd/Kconfig | 4 +++-
> >> cmd/optee_rpmb.c | 23 +++++++++++++++++------
> >> 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 ++++++++++++++++++++
> >> 11 files changed, 49 insertions(+), 19 deletions(-)
> >> create mode 100644 test/py/tests/test_optee_rpmb.py
> >>
> >> --
> >> 2.34.1
> >>
> >
> > Are there currently any comments/objections that can prevent these cosmetic
> > changes from being applied? CI is happy now and all tests are passing successfully.
> >
> > If there are any - just let me know, thanks
>
> IIRC I did run this version on the CI and everything looked fine. I'll
> send a PR to Tom with it -- apologies for the delay
>
> Regards
> /Ilias
> >
> > Regards,
> > Igor
> >
> > --
> > Best regards - Atentamente - Meilleures salutations
> >
> > Igor Opaniuk
> >
> > mailto: igor.opaniuk@gmail.com
> > skype: igor.opanyuk
> > https://www.linkedin.com/in/iopaniuk
prev parent reply other threads:[~2024-04-04 7:22 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-04 17:44 [PATCH v3 0/6] TEE: minor cleanup Igor Opaniuk
2024-03-04 17:44 ` [PATCH v3 1/6] tee: optee: fix description in Kconfig Igor Opaniuk
2024-03-04 17:44 ` [PATCH v3 2/6] tee: sandbox: fix spelling errors Igor Opaniuk
2024-03-04 17:44 ` [PATCH v3 3/6] cmd: optee_rpmb: close tee session Igor Opaniuk
2024-03-04 17:44 ` [PATCH v3 4/6] cmd: optee_rpmb: build cmd for sandbox Igor Opaniuk
2024-03-04 17:44 ` [PATCH v3 5/6] test: py: add optee_rpmb tests Igor Opaniuk
2024-03-04 17:44 ` [PATCH v3 6/6] tee: remove common.h inclusion Igor Opaniuk
2024-03-08 13:27 ` Ilias Apalodimas
2024-03-08 18:08 ` Sam Protsenko
2024-03-27 12:13 ` [PATCH v3 0/6] TEE: minor cleanup Igor Opaniuk
2024-03-27 14:17 ` Ilias Apalodimas
2024-04-04 7:22 ` Ilias Apalodimas [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Zg5VIfnPElreq3ls@hera \
--to=ilias.apalodimas@linaro.org \
--cc=abdellatif.elkhlifi@arm.com \
--cc=bmeng@tinylab.org \
--cc=etienne.carriere@foss.st.com \
--cc=francis.laniel@amarulasolutions.com \
--cc=igor.opaniuk@gmail.com \
--cc=jens.wiklander@linaro.org \
--cc=jorge@foundries.io \
--cc=miquel.raynal@bootlin.com \
--cc=mkorpershoek@baylibre.com \
--cc=patrice.chotard@foss.st.com \
--cc=rknecht@pm.me \
--cc=seanedmond@microsoft.com \
--cc=sjg@chromium.org \
--cc=takahiro.akashi@linaro.org \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
--cc=xypron.glpk@gmx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.