Re: [PATCHv8 0/3] optee: register drivers on optee bus

From: Jens Wiklander
Date: Thu Jun 18 2020 - 04:00:18 EST


On Wed, Jun 17, 2020 at 05:26:42PM +0300, Maxim Uvarov wrote:
> On Tue, 16 Jun 2020 at 11:29, Jens Wiklander <jens.wiklander@xxxxxxxxxx> wrote:
> >
> > Hi Maxim and Jarkko,
> >
> > On Mon, Jun 15, 2020 at 05:32:40PM +0300, Maxim Uvarov wrote:
> > > ping.
> > > Patchset was reviewed and all comments are codeverd. Optee-os patches
> > > were merged. These kernel patches look like they are hanging
> > > somewhere...
> >
> > I'm almost OK with this patchset, except that
> > Documentation/ABI/testing/sysfs-bus-optee-devices needs to be updated
> > for the new kernel version and TEE mailing list which we're changing right
> > now.
> >
>
> What is the new mailing list address? I did git pull the latest
> master and get_maintainer.pl still points to
> tee-dev@xxxxxxxxxxxxxxxxx

I'm sorry, I should have mentioned that. I just sent a pull request
to update it so it will have changed by the time these patches gets
merged. The new address is op-tee@xxxxxxxxxxxxxxxxxxxxxxxxx

Cheers,
Jens

>
> Maxim.
>
> > The last patch touches files I'm not maintainer of. That patch depends
> > on the previous patches so it makes sense to keep them together. If a
> > TPM device driver maintainer would ack that patch I can take it via my
> > tree. Or we can do it the other way around (with a v9 patchset),
> > whichever is preferred.
> >
> > Cheers,
> > Jens
> >
> > >
> > > Thanks,
> > > Maxim.
> > >
> > > On Thu, 4 Jun 2020 at 20:58, Maxim Uvarov <maxim.uvarov@xxxxxxxxxx> wrote:
> > > >
> > > > v8: - fix v7 check.
> > > > v7: - check return value of dev_set_name() (Jarkko Sakkinen)
> > > > v6: - description, comments, patches reorder and destroy workqueue (Sumit Garg)
> > > > v5: - removed pr_err and fix typos in description (Jarkko Sakkinen)
> > > > - added missed kfree in optee_open()
> > > > v4: - sysfs entry is optee-ta-uuid (Jerome Forissier, Sumit Garg)
> > > > - added Documentation/ABI/testing/sysfs-bus-optee-devices (Greg Kroah-Hartman)
> > > > v3: - support tee-suppicant restart (Jens Wiklander)
> > > > - description and comments (Jarkko Sakkinen)
> > > > - do not name optee drivers by index in sysfs (Sumit Garg)
> > > > v2: - write TEE with capital letters.
> > > > - declare __optee_enumerate_device() as static.
> > > >
> > > > Maxim Uvarov (3):
> > > > optee: use uuid for sysfs driver entry
> > > > optee: enable support for multi-stage bus enumeration
> > > > tpm_ftpm_tee: register driver on TEE bus
> > > >
> > > > .../ABI/testing/sysfs-bus-optee-devices | 8 +++
> > > > MAINTAINERS | 1 +
> > > > drivers/char/tpm/tpm_ftpm_tee.c | 70 ++++++++++++++++---
> > > > drivers/tee/optee/core.c | 27 ++++++-
> > > > drivers/tee/optee/device.c | 38 +++++-----
> > > > drivers/tee/optee/optee_private.h | 10 ++-
> > > > 6 files changed, 119 insertions(+), 35 deletions(-)
> > > > create mode 100644 Documentation/ABI/testing/sysfs-bus-optee-devices
> > > >
> > > > --
> > > > 2.17.1
> > > >