Re: [PATCH] Documentation: Destage TEE subsystem documentation

[Vegard Nossum]

On 31/10/2023 11:07, Sumit Garg wrote:
> Add a separate documentation directory for TEE subsystem since it is a
> standalone subsystem which already offers devices consumed by multiple
> different subsystem drivers.
>
> Split overall TEE subsystem documentation modularly where the first
> module covers the overview of TEE subsystem itself along with generic
> features. Then the further modules are dedicated to different TEE
> implementations like:
> - OP-TEE
> - AMD-TEE
> - and so on for future TEE implementation support.
>
> Signed-off-by: Sumit Garg <sumit.garg@xxxxxxxxxx>
> ---
>   Documentation/staging/index.rst  |   1 -
>   Documentation/staging/tee.rst    | 364 -------------------------------
>   Documentation/subsystem-apis.rst |   1 +
>   Documentation/tee/amd-tee.rst    |  90 ++++++++
>   Documentation/tee/index.rst      |  19 ++
>   Documentation/tee/op-tee.rst     | 166 ++++++++++++++
>   Documentation/tee/tee.rst        | 122 +++++++++++
>   MAINTAINERS                      |   2 +-
>   8 files changed, 399 insertions(+), 366 deletions(-)
>   delete mode 100644 Documentation/staging/tee.rst
>   create mode 100644 Documentation/tee/amd-tee.rst
>   create mode 100644 Documentation/tee/index.rst
>   create mode 100644 Documentation/tee/op-tee.rst
>   create mode 100644 Documentation/tee/tee.rst

A comment from the peanut gallery:

The old tee.rst starts out describing the userspace API (include/uapi/,
/dev/tee, etc.). Maybe this part should go into
Documentation/userspace-api/.

The rest of the document describes the internal kernel API (how to write
a TEE driver). And maybe this part should go into Documentation/driver-api/.

At least I think that kind of split is what the documentation is going
for in the long term, so if you're moving these files around anyway it
might make sense to do it right away.


Vegard