docs: Update readme - Githubissues

ARMmbed / mbed-os-tf-m-regression-tests

An Mbed OS application that runs the TF-M regression tests

2 stars 12 forks source link

docs: Update readme #63

Closed jainvikas8 closed 3 years ago

jainvikas8 commented 3 years ago

The following has been updated:

TF-M v1.1 support with Mbed OS
Help option for each script
Vagrant environment is only for development and build as the targets can't be accessed by USB
Running the Mbed OS application section
Update instructions for executing all tests suites

jainvikas8 commented 3 years ago

Forced pushed to address review comments from @evedon

jainvikas8 commented 3 years ago

Forced pushed to address further comments.

jainvikas8 commented 3 years ago

PSA example

This is a much improved README. Thanks.

I am still finding the section "Building TF-M" not clear though. Why do we need to build TF-M? It is written "We are building for the ARM Musca B1 (ARM_MUSCA_B1) in our example below": is it the PSA example (not in this repo) or a test suite?

The "Building TF-M" section doesn't build any test suite, but the files and binaries generated are checked in to Mbed OS repository at the time of release (say TF-M v1.2) which further supports the building of https://github.com/ARMmbed/mbed-os-example-psa without the user requiring to go through the complex process.

evedon commented 3 years ago

The "Building TF-M" section doesn't build any test suite, but the files and binaries generated are checked in to Mbed OS repository at the time of release (say TF-M v1.2) which further supports the building of https://github.com/ARMmbed/mbed-os-example-psa without the user requiring to go through the complex process.

Can you please add this explanation to your note in the "Building TF-M" section?

jainvikas8 commented 3 years ago

Forced pushed to address review comments.