Re: [PATCH v4 11/11] doc: board: ti: am62lx_evm: add initial docs for the AM62L-EVM

Hi Bryan,

On 2026-08-04T21:55:02, Bryan Brattlof <[email protected]> wrote:
> doc: board: ti: am62lx_evm: add initial docs for the AM62L-EVM
>
> Add the initial documentation on the bootflow, build, and deubgging
> procedures along with the relevant links to more information about the
> AM62Lx processor and the reference board.
>
> Tested-by: Anshul Dalal <[email protected]>
> Reviewed-by: Anshul Dalal <[email protected]>
> Signed-off-by: Bryan Brattlof <[email protected]>
>
> doc/board/ti/am62lx_evm.rst                     | 222 ++++++++++
>  doc/board/ti/img/am62lx_boot_diagram_phase1.svg | 450 ++++++++++++++++++++
>  doc/board/ti/img/am62lx_boot_diagram_phase2.svg | 525 ++++++++++++++++++++++++
>  doc/board/ti/img/am62lx_tiboot3.bin.svg         | 170 ++++++++
>  doc/board/ti/img/am62lx_tispl.bin.svg           | 222 ++++++++++
>  doc/board/ti/img/am62lx_u-boot.img.svg          | 143 +++++++
>  doc/board/ti/k3.rst                             |   1 +
>  7 files changed, 1733 insertions(+)

> diff --git a/doc/board/ti/am62lx_evm.rst b/doc/board/ti/am62lx_evm.rst
> @@ -0,0 +1,222 @@
> +The AM62L is a general purpose processor, however some of the
> +applications well suited for it include: Human Machine Interfaces (HMI),
> +Medical patient monitoring , Building automation, Smart secure gateways,

Stray space before the comma after 'monitoring'.

> diff --git a/doc/board/ti/am62lx_evm.rst b/doc/board/ti/am62lx_evm.rst
> @@ -0,0 +1,222 @@
> +However because the lack of any MCU cores the boot ROM is located on one
> +of the A53 cores which is responsible for loading TI's root-of-trust
> +firmware (TIFS) along with a BL1 image generated from the
> +ARM-Trusted-Firmware repository. The BL1 image is responsible for
> +initializing the debug console, the DDR controller, and speeding up the
> +A53 to it's maximum allowable.

'However because the lack of' should be 'However, because of the lack
of'. And 'it's' should be 'its' - plus 'maximum allowable' is missing
a noun (frequency?).

> diff --git a/doc/board/ti/am62lx_evm.rst b/doc/board/ti/am62lx_evm.rst
> @@ -0,0 +1,222 @@
> +For the second boot phase, with DDR active ROM can now load the larger

Please add a comma after 'active' - as written it reads as 'DDR active ROM'.

Regards,
Simon