DemesneGH commented on code in PR #263:
URL: 
https://github.com/apache/teaclave-trustzone-sdk/pull/263#discussion_r2622070129


##########
docs/emulate-and-dev-in-docker-std.md:
##########
@@ -19,9 +29,16 @@ The **dev-env with std support** enables **developing TA 
using Rust std** by pro
 - **Rust standard library tailored for OP-TEE** - Build TAs using collections, 
networking, serialization capabilities
 - **Mixed development support** - Combine different host and TA architectures, 
including switching between no-std/std in the same project
 
-## 1. Setting Up the Dev-Env with Std Support
+## Workflow: Developing std-enabled TAs with cargo-optee

Review Comment:
   Recommend to reorganize:
   ```
   ## Setting Up the Dev-Env with Std Support
   ...
   ## Build STD TA
   ### 1. cargo-optee ...
   ### 2. make ...
   ```
   
   This doc just provides the command to build STD TAs, for the description of 
cargo-optee, we can add a link to the cargo-optee README. For the no-std TAs, 
we can provides the building command in the other emulate-... doc.
   
   Please also reorganize the another emulate-... doc, the goal is to make each 
doc clear and avoid duplicated contents.
   
   
   



##########
README.md:
##########
@@ -35,6 +35,8 @@ The Docker image automates the entire setup process for 
TrustZone emulation
 in QEMU, enabling you to focus on writing and testing your applications 
 efficiently, without the hassle of manual configuration.
 
+**UPDATES** 
我们更新了一个更加方便的工具,更好的封装让开发者更加便捷[cargo-optee](./cargo-optee/README.md), 
后续我们可能会慢慢放弃makefile结合cargo的方式,使用更加rust的方式开发. 
为了平稳过度,我们仍然保留的使用Makefile进行构建的相关工具组件。

Review Comment:
   Could you provide an English version to keep consistency in this doc? Thanks



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to