[Intel-xe] [PATCH] fixup! drm/xe: Make DRM_XE_DEVICE_QUERY_ENGINES future proof

Francois Dugast francois.dugast at intel.com
Tue Dec 5 19:42:11 UTC 2023


On Tue, Dec 05, 2023 at 09:43:49AM -0800, Lucas De Marchi wrote:
> Fix kerneldoc build, as being warned by CI with the Hooks execution.
> 
> 	$ find drivers/gpu/drm/xe/ -name '*.[ch]' -not -path 'drivers/gpu/drm/xe/display/*' | \
> 		xargs ./scripts/kernel-doc -Werror -none include/uapi/drm/xe_drm.h
> 	include/uapi/drm/xe_drm.h:152: warning: Function parameter or member 'engine_class' not described in 'drm_xe_engine_class_instance'
> 	include/uapi/drm/xe_drm.h:152: warning: Function parameter or member 'engine_instance' not described in 'drm_xe_engine_class_instance'
> 	include/uapi/drm/xe_drm.h:152: warning: Function parameter or member 'gt_id' not described in 'drm_xe_engine_class_instance'
> 	3 warnings as Errors
> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>

Thanks. FYI kerneldoc series is in preparation to fix a few more things.

Reviewed-by: Francois Dugast <francois.dugast at intel.com>

> ---
>  include/uapi/drm/xe_drm.h | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
> index eb03a49c17a1..0895e4d2a981 100644
> --- a/include/uapi/drm/xe_drm.h
> +++ b/include/uapi/drm/xe_drm.h
> @@ -129,7 +129,6 @@ struct xe_user_extension {
>   * It is returned as part of the @drm_xe_engine, but it also is used as
>   * the input of engine selection for both @drm_xe_exec_queue_create and
>   * @drm_xe_query_engine_cycles
> - *
>   */
>  struct drm_xe_engine_class_instance {
>  #define DRM_XE_ENGINE_CLASS_RENDER		0
> @@ -143,9 +142,11 @@ struct drm_xe_engine_class_instance {
>  	 */
>  #define DRM_XE_ENGINE_CLASS_VM_BIND_ASYNC	5
>  #define DRM_XE_ENGINE_CLASS_VM_BIND_SYNC	6
> +	/** @engine_class: engine class id */
>  	__u16 engine_class;
> -
> +	/** @engine_instance: engine instance id */
>  	__u16 engine_instance;
> +	/** @gt_id: Unique ID of this GT within the PCI Device */
>  	__u16 gt_id;
>  	/** @pad: MBZ */
>  	__u16 pad;
> -- 
> 2.40.1
> 


More information about the Intel-xe mailing list