[PATCH] accel: Link to compute accelerator subsystem intro

Bagas Sanjaya posted 1 patch 3 years, 1 month ago
include/drm/drm_file.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
[PATCH] accel: Link to compute accelerator subsystem intro
Posted by Bagas Sanjaya 3 years, 1 month ago
Commit 2c204f3d53218d ("accel: add dedicated minor for accelerator
devices") adds link to accelerator nodes section of DRM internals doc
(Documentation/gpu/drm-internals.rst), but the target doesn't exist.
Instead, there is only an introduction doc for computer accelerator
subsytem.

Link to that doc until there is documentation of accelerator internals.

Fixes: 2c204f3d53218d ("accel: add dedicated minor for accelerator devices")
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---

 This comes from discussion on Dylan Le's attempted fix, where Jeffrey
 Hugo suspected that the proper target should be accelerator subsystem
 doc [2].

 [1]: https://lore.kernel.org/linux-doc/7799513f0fb5aee27b64752c037980471be669c5.1677120686.git.self@dylanle.dev/
 [2]: https://lore.kernel.org/linux-doc/CAOCk7NrAKZ2G-VbQW3MJUZmkKkO1Ccrs2wo12kg6QHysH-d=Yg@mail.gmail.com/

 include/drm/drm_file.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/include/drm/drm_file.h b/include/drm/drm_file.h
index 0d1f853092ab8a..ecffe24e2b1b0a 100644
--- a/include/drm/drm_file.h
+++ b/include/drm/drm_file.h
@@ -408,7 +408,8 @@ static inline bool drm_is_render_client(const struct drm_file *file_priv)
  * Returns true if this is an open file of the compute acceleration node, i.e.
  * &drm_file.minor of @file_priv is a accel minor.
  *
- * See also the :ref:`section on accel nodes <drm_accel_node>`.
+ * See also :doc:`Introduction to compute accelerators subsystem
+ * </accel/introduction>`.
  */
 static inline bool drm_is_accel_client(const struct drm_file *file_priv)
 {

base-commit: 5c8cf1664f288098a971a1d1e65716a2b6a279e1
-- 
An old man doll... just what I always wanted! - Clara
Re: [PATCH] accel: Link to compute accelerator subsystem intro
Posted by Jeffrey Hugo 3 years, 1 month ago
On 3/6/2023 9:35 PM, Bagas Sanjaya wrote:
> Commit 2c204f3d53218d ("accel: add dedicated minor for accelerator
> devices") adds link to accelerator nodes section of DRM internals doc
> (Documentation/gpu/drm-internals.rst), but the target doesn't exist.
> Instead, there is only an introduction doc for computer accelerator
> subsytem.
> 
> Link to that doc until there is documentation of accelerator internals.
> 
> Fixes: 2c204f3d53218d ("accel: add dedicated minor for accelerator devices")
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>

Reviewed-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
Re: [PATCH] accel: Link to compute accelerator subsystem intro
Posted by Oded Gabbay 3 years ago
On Tue, Mar 7, 2023 at 6:04 PM Jeffrey Hugo <quic_jhugo@quicinc.com> wrote:
>
> On 3/6/2023 9:35 PM, Bagas Sanjaya wrote:
> > Commit 2c204f3d53218d ("accel: add dedicated minor for accelerator
> > devices") adds link to accelerator nodes section of DRM internals doc
> > (Documentation/gpu/drm-internals.rst), but the target doesn't exist.
> > Instead, there is only an introduction doc for computer accelerator
> > subsytem.
> >
> > Link to that doc until there is documentation of accelerator internals.
> >
> > Fixes: 2c204f3d53218d ("accel: add dedicated minor for accelerator devices")
> > Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
>
> Reviewed-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
Thanks.
Will upstream through my tree (habanalabs-next)
Oded