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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox