[PATCH v2 2/2] drm: add overview diagram for drm stack

[Bagas Sanjaya]

On Fri, May 23, 2025 at 03:52:26PM -0400, Abdulrasaq Lawani wrote:
> diff --git a/Documentation/gpu/introduction.rst b/Documentation/gpu/introduction.rst
> index 91bb0efc96d69921a122c5265b1431fa18110a7a..2653048ad4bb0ecbc2b676295440442dddbabae6 100644
> --- a/Documentation/gpu/introduction.rst
> +++ b/Documentation/gpu/introduction.rst
> @@ -15,41 +15,43 @@ and the new vblank internals, in addition to all the regular features
>  found in current kernels.
>  
>  Overview of the Linux DRM Architecture
> ------------------------------------------------
> -+-----------------------------+
> -|     User-space Apps         |
> -| (Games, Browsers, ML, etc.) |
> -+-----------------------------+
> -              |
> -              v
> -+---------------------------------------+
> -|    Graphics APIs   |   Compute APIs   |
> -|  (OpenGL, Vulkan)  |  (OpenCL, CUDA)  |
> -+---------------------------------------+
> -          |                   |
> -          v                   v
> -+---------------------+  +-----------------------+
> -|  User-space Driver  |  |    Compute Runtime    |
> -|  (Mesa, AMD/NVIDIA) |  |  (OpenCL, CUDA, ROCm) |
> -+---------------------+  +-----------------------+
> -          |                   |
> -          +--------+----------+
> -                   |
> -                   v
> -        +-----------------------+
> -        |   libdrm (DRM API)    |
> -        +-----------------------+
> -                   |
> -                   v
> -+-------------------------------------------+
> -|     Kernel DRM/KMS Driver (i915, amdgpu,  |
> -|     nouveau, etc.)                        |
> -+-------------------------------------------+
> -        |                       |
> -        v                       v
> -+----------------+     +-------------------+
> -| GPU Display HW |     | GPU Compute Units |
> -+----------------+     +-------------------+
> +======================================

Why promoting the section heading?

> +::
> +
> +        +-----------------------------+
> +        |     User-space Apps         |
> +        | (Games, Browsers, ML, etc.) |
> +        +-----------------------------+
> +                      |
> +                      v
> +        +---------------------------------------+
> +        |    Graphics APIs   |   Compute APIs   |
> +        |  (OpenGL, Vulkan)  |  (OpenCL, CUDA)  |
> +        +---------------------------------------+
> +                |                   |
> +                v                   v
> +        +---------------------+  +-----------------------+
> +        |  User-space Driver  |  |    Compute Runtime    |
> +        |  (Mesa, AMD/NVIDIA) |  |  (OpenCL, CUDA, ROCm) |
> +        +---------------------+  +-----------------------+
> +                |                   |
> +                +--------+----------+
> +                         |
> +                         v
> +                +-----------------------+
> +                |   libdrm (DRM API)    |
> +                +-----------------------+
> +                          |
> +                          v
> +        +-------------------------------------------+
> +        |     Kernel DRM/KMS Driver (i915, amdgpu,  |
> +        |     nouveau, etc.)                        |
> +        +-------------------------------------------+
> +                |                       |
> +                v                       v
> +        +----------------+     +-------------------+
> +        | GPU Display HW |     | GPU Compute Units |
> +        +----------------+     +-------------------+
>  

This is not how reviews from v1 should be addressed. Instead of making
separate commit for review suggestions, you need to squash them into affected
commit(s) in v1. In this case, there should be only one resulting patch. Don't
forget to send it as v3.

Thanks.

-- 
An old man doll... just what I always wanted! - Clara
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20250526/02810211/attachment.sig>