Re: [PATCH v2 5/7] docs: nova-core: Document devinit process

[Bagas Sanjaya <bagasdotme@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1037 bytes --]

On Sat, May 03, 2025 at 12:07:57AM -0400, Joel Fernandes wrote:
> +.. SPDX-License-Identifier: GPL-2.0
> +==================================
> +Device Initialization (devinit)
> +==================================

Separate SPDX line from title heading.

> +These low-level GPU firmware components are typically:
> +1. Located in the VBIOS ROM in the same ROM partition (see vbios.rst and fwsec.rst).
> +2. Executed in sequence on different microcontrollers:
> +   - The devinit engine typically but not necessarily runs on the PMU.
> +   - On an Ampere GPU, the FWSEC typically runs on the GSP (GPU System Processor) in
> +     heavy-secure mode.

Please separate numbered list from preceding sentence, and the bullet sublist
from parent numbered list by a line.

> +Runtime Considerations
> +---------------------
> <snipped>...
> +Security and Access Control
> +--------------------------

Match section underline length with the text.

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]