2022-07-09 10:09:47

by Mauro Carvalho Chehab

Subject: [PATCH v3 09/21] drm/scheduler: fix a kernel-doc warning

The dev field was not documented:

include/drm/gpu_scheduler.h:463: warning: Function parameter or member 'dev' not described in 'drm_gpu_scheduler'

Document it.

Reviewed-by: Alex Deucher <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
---

To avoid mailbombing on a large number of people, only mailing lists were C/C on the cover.
See [PATCH v3 00/21] at: https://lore.kernel.org/all/[email protected]/

include/drm/gpu_scheduler.h | 1 +
1 file changed, 1 insertion(+)

diff --git a/include/drm/gpu_scheduler.h b/include/drm/gpu_scheduler.h
index addb135eeea6..c7c487e0c40e 100644
--- a/include/drm/gpu_scheduler.h
+++ b/include/drm/gpu_scheduler.h
@@ -435,6 +435,7 @@ struct drm_sched_backend_ops {
* @_score: score used when the driver doesn't provide one
* @ready: marks if the underlying HW is ready to work
* @free_guilty: A hit to time out handler to free the guilty job.
+ * @dev: pointer to struct device.
*
* One scheduler is implemented for each hardware ring.
*/
--
2.36.1

2022-08-11 17:17:42

by Alex Deucher

Subject: Re: [PATCH v3 09/21] drm/scheduler: fix a kernel-doc warning

On Sat, Jul 9, 2022 at 6:08 AM Mauro Carvalho Chehab <[email protected]> wrote:
>
> The dev field was not documented:
>
> include/drm/gpu_scheduler.h:463: warning: Function parameter or member 'dev' not described in 'drm_gpu_scheduler'
>
> Document it.
>
> Reviewed-by: Alex Deucher <[email protected]>
> Signed-off-by: Mauro Carvalho Chehab <[email protected]>

Patches 9, 10 are:
Reviewed-by: Alex Deucher <[email protected]>

Do you have commit rights to drm-misc or do you need me to push them?

Thanks,

Alex


> ---
>
> To avoid mailbombing on a large number of people, only mailing lists were C/C on the cover.
> See [PATCH v3 00/21] at: https://lore.kernel.org/all/[email protected]/
>
> include/drm/gpu_scheduler.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/drm/gpu_scheduler.h b/include/drm/gpu_scheduler.h
> index addb135eeea6..c7c487e0c40e 100644
> --- a/include/drm/gpu_scheduler.h
> +++ b/include/drm/gpu_scheduler.h
> @@ -435,6 +435,7 @@ struct drm_sched_backend_ops {
> * @_score: score used when the driver doesn't provide one
> * @ready: marks if the underlying HW is ready to work
> * @free_guilty: A hit to time out handler to free the guilty job.
> + * @dev: pointer to struct device.
> *
> * One scheduler is implemented for each hardware ring.
> */
> --
> 2.36.1
>