drm/imagination: Numerous documentation fixes.

Some reported by Stephen Rothwell. The rest were found by running the
kernel-doc build script.
Some indentation fixes.

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202311241526.Y2WZeUau-lkp@intel.com/
Signed-off-by: Donald Robson <donald.robson@imgtec.com>
Signed-off-by: Maxime Ripard <mripard@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20231128173507.95119-1-donald.robson@imgtec.com
This commit is contained in:
Donald Robson 2023-11-28 17:35:07 +00:00 committed by Maxime Ripard
parent 325b71e820
commit 3cc808e323
No known key found for this signature in database
GPG Key ID: E3EF0D6F671851C5
12 changed files with 38 additions and 30 deletions

View File

@ -3,7 +3,7 @@ drm/imagination PowerVR Graphics Driver
======================================= =======================================
.. kernel-doc:: drivers/gpu/drm/imagination/pvr_drv.c .. kernel-doc:: drivers/gpu/drm/imagination/pvr_drv.c
:doc: PowerVR Graphics Driver :doc: PowerVR (Series 6 and later) and IMG Graphics Driver
Contents Contents
======== ========

View File

@ -45,9 +45,6 @@ DEV_QUERY
drm_pvr_heap drm_pvr_heap
drm_pvr_dev_query_heap_info drm_pvr_dev_query_heap_info
.. kernel-doc:: include/uapi/drm/pvr_drm.h
:doc: Flags for DRM_PVR_DEV_QUERY_HEAP_INFO_GET.
.. kernel-doc:: include/uapi/drm/pvr_drm.h .. kernel-doc:: include/uapi/drm/pvr_drm.h
:identifiers: drm_pvr_static_data_area_usage :identifiers: drm_pvr_static_data_area_usage
drm_pvr_static_data_area drm_pvr_static_data_area
@ -121,7 +118,7 @@ CREATE_FREE_LIST and DESTROY_FREE_LIST
:identifiers: drm_pvr_ioctl_destroy_free_list_args :identifiers: drm_pvr_ioctl_destroy_free_list_args
CREATE_HWRT_DATASET and DESTROY_HWRT_DATASET CREATE_HWRT_DATASET and DESTROY_HWRT_DATASET
-------------------------------------- --------------------------------------------
.. kernel-doc:: include/uapi/drm/pvr_drm.h .. kernel-doc:: include/uapi/drm/pvr_drm.h
:doc: PowerVR IOCTL CREATE_HWRT_DATASET and DESTROY_HWRT_DATASET interfaces :doc: PowerVR IOCTL CREATE_HWRT_DATASET and DESTROY_HWRT_DATASET interfaces

View File

@ -86,6 +86,7 @@ pvr_cccb_get_size_of_cmd_with_hdr(u32 cmd_size)
/** /**
* pvr_cccb_cmdseq_can_fit() - Check if a command sequence can fit in the CCCB. * pvr_cccb_cmdseq_can_fit() - Check if a command sequence can fit in the CCCB.
* @pvr_cccb: Target Client CCB.
* @size: Command sequence size. * @size: Command sequence size.
* *
* Returns: * Returns:

View File

@ -203,17 +203,29 @@ struct pvr_device {
struct mutex lock; struct mutex lock;
} queues; } queues;
/**
* @watchdog: Watchdog for communications with firmware.
*/
struct { struct {
/** @work: Work item for watchdog callback. */ /** @work: Work item for watchdog callback. */
struct delayed_work work; struct delayed_work work;
/** @old_kccb_cmds_executed: KCCB command execution count at last watchdog poll. */ /**
* @old_kccb_cmds_executed: KCCB command execution count at last
* watchdog poll.
*/
u32 old_kccb_cmds_executed; u32 old_kccb_cmds_executed;
/** @kccb_stall_count: Number of watchdog polls KCCB has been stalled for. */ /**
* @kccb_stall_count: Number of watchdog polls KCCB has been
* stalled for.
*/
u32 kccb_stall_count; u32 kccb_stall_count;
} watchdog; } watchdog;
/**
* @kccb: Circular buffer for communications with firmware.
*/
struct { struct {
/** @ccb: Kernel CCB. */ /** @ccb: Kernel CCB. */
struct pvr_ccb ccb; struct pvr_ccb ccb;
@ -225,8 +237,8 @@ struct pvr_device {
struct pvr_fw_object *rtn_obj; struct pvr_fw_object *rtn_obj;
/** /**
* @rtn: Pointer to CPU mapping of KCCB return slots. Must be accessed by * @rtn: Pointer to CPU mapping of KCCB return slots. Must be
* READ_ONCE()/WRITE_ONCE(). * accessed by READ_ONCE()/WRITE_ONCE().
*/ */
u32 *rtn; u32 *rtn;
@ -293,14 +305,13 @@ struct pvr_file {
/** /**
* @pvr_dev: A reference to the powervr-specific wrapper for the * @pvr_dev: A reference to the powervr-specific wrapper for the
* associated device. Saves on repeated calls to * associated device. Saves on repeated calls to to_pvr_device().
* to_pvr_device().
*/ */
struct pvr_device *pvr_dev; struct pvr_device *pvr_dev;
/** /**
* @ctx_handles: Array of contexts belonging to this file. Array members * @ctx_handles: Array of contexts belonging to this file. Array members
* are of type "struct pvr_context *". * are of type "struct pvr_context *".
* *
* This array is used to allocate handles returned to userspace. * This array is used to allocate handles returned to userspace.
*/ */

View File

@ -481,7 +481,8 @@ pvr_fw_object_unmap_and_destroy(struct pvr_fw_object *fw_obj)
} }
/** /**
* pvr_fw_get_dma_addr() - Get DMA address for given offset in firmware object * pvr_fw_object_get_dma_addr() - Get DMA address for given offset in firmware
* object.
* @fw_obj: Pointer to object to lookup address in. * @fw_obj: Pointer to object to lookup address in.
* @offset: Offset within object to lookup address at. * @offset: Offset within object to lookup address at.
* @dma_addr_out: Pointer to location to store DMA address. * @dma_addr_out: Pointer to location to store DMA address.

View File

@ -122,13 +122,13 @@ struct pvr_fw_layout_entry {
* struct pvr_fw_device_info_header - Device information header. * struct pvr_fw_device_info_header - Device information header.
*/ */
struct pvr_fw_device_info_header { struct pvr_fw_device_info_header {
/* BRN Mask size (in u64s). */ /** @brn_mask_size: BRN mask size (in u64s). */
u64 brn_mask_size; u64 brn_mask_size;
/* ERN Mask size (in u64s). */ /** @ern_mask_size: ERN mask size (in u64s). */
u64 ern_mask_size; u64 ern_mask_size;
/* Feature Mask size (in u64s). */ /** @feature_mask_size: Feature mask size (in u64s). */
u64 feature_mask_size; u64 feature_mask_size;
/* Feature Parameter size (in u64s). */ /** @feature_param_size: Feature parameter size (in u64s). */
u64 feature_param_size; u64 feature_param_size;
}; };

View File

@ -64,6 +64,7 @@ struct pvr_hwrt_dataset {
/** @common_fw_obj: FW object representing common FW-side structure. */ /** @common_fw_obj: FW object representing common FW-side structure. */
struct pvr_fw_object *common_fw_obj; struct pvr_fw_object *common_fw_obj;
/** @common: Common HWRT data. */
struct rogue_fwif_hwrtdata_common common; struct rogue_fwif_hwrtdata_common common;
/** @data: HWRT data structures belonging to this set. */ /** @data: HWRT data structures belonging to this set. */

View File

@ -378,7 +378,7 @@ prepare_job_syncs(struct pvr_file *pvr_file,
/** /**
* prepare_job_syncs_for_each() - Prepare all sync objects for an array of jobs. * prepare_job_syncs_for_each() - Prepare all sync objects for an array of jobs.
* @file: PowerVR file. * @pvr_file: PowerVR file.
* @job_data: Array of precreated jobs and their sync_ops. * @job_data: Array of precreated jobs and their sync_ops.
* @job_count: Number of jobs. * @job_count: Number of jobs.
* @signal_array: xarray to receive signal sync objects. * @signal_array: xarray to receive signal sync objects.
@ -696,8 +696,6 @@ pvr_jobs_link_geom_frag(struct pvr_job_data *job_data, u32 *job_count)
* @pvr_dev: Target PowerVR device. * @pvr_dev: Target PowerVR device.
* @pvr_file: Pointer to PowerVR file structure. * @pvr_file: Pointer to PowerVR file structure.
* @args: Ioctl args. * @args: Ioctl args.
* @job_count: Number of jobs in @jobs_args. On error this will be updated
* with the index into @jobs_args where the error occurred.
* *
* This initial implementation is entirely synchronous; on return the GPU will * This initial implementation is entirely synchronous; on return the GPU will
* be idle. This will not be the case for future implementations. * be idle. This will not be the case for future implementations.

View File

@ -335,8 +335,9 @@ pvr_mmu_backing_page_fini(struct pvr_mmu_backing_page *page)
/** /**
* pvr_mmu_backing_page_sync() - Flush a MMU backing page from the CPU to the * pvr_mmu_backing_page_sync() - Flush a MMU backing page from the CPU to the
* device. * device.
* @page: Target backing page. * @page: Target backing page.
* @flags: MMU flush flags. Must be one of %PVR_MMU_SYNC_LEVEL_*_FLAGS.
* *
* .. caution:: * .. caution::
* *

View File

@ -50,7 +50,7 @@ struct pvr_queue_cccb_fence_ctx {
*/ */
struct pvr_job *job; struct pvr_job *job;
/** @lock: Lock protecting access to the job object. */ /** @job_lock: Lock protecting access to the job object. */
struct mutex job_lock; struct mutex job_lock;
}; };
@ -114,7 +114,7 @@ struct pvr_queue {
} timeline_ufo; } timeline_ufo;
/** /**
* last_queued_job_scheduled_fence: The scheduled fence of the last * @last_queued_job_scheduled_fence: The scheduled fence of the last
* job queued to this queue. * job queued to this queue.
* *
* We use it to insert frag -> geom dependencies when issuing combined * We use it to insert frag -> geom dependencies when issuing combined

View File

@ -42,7 +42,7 @@ struct pvr_vm_context {
/** @mmu_ctx: The context for binding to physical memory. */ /** @mmu_ctx: The context for binding to physical memory. */
struct pvr_mmu_context *mmu_ctx; struct pvr_mmu_context *mmu_ctx;
/** @gpuva_mgr: GPUVA manager object associated with this context. */ /** @gpuvm_mgr: GPUVM object associated with this context. */
struct drm_gpuvm gpuvm_mgr; struct drm_gpuvm gpuvm_mgr;
/** @lock: Global lock on this VM. */ /** @lock: Global lock on this VM. */

View File

@ -741,20 +741,18 @@ enum drm_pvr_ctx_priority {
*/ */
enum drm_pvr_ctx_type { enum drm_pvr_ctx_type {
/** /**
* @DRM_PVR_CTX_TYPE_RENDER: Render context. Use &struct * @DRM_PVR_CTX_TYPE_RENDER: Render context.
* drm_pvr_ioctl_create_render_context_args for context creation arguments.
*/ */
DRM_PVR_CTX_TYPE_RENDER = 0, DRM_PVR_CTX_TYPE_RENDER = 0,
/** /**
* @DRM_PVR_CTX_TYPE_COMPUTE: Compute context. Use &struct * @DRM_PVR_CTX_TYPE_COMPUTE: Compute context.
* drm_pvr_ioctl_create_compute_context_args for context creation arguments.
*/ */
DRM_PVR_CTX_TYPE_COMPUTE, DRM_PVR_CTX_TYPE_COMPUTE,
/** /**
* @DRM_PVR_CTX_TYPE_TRANSFER_FRAG: Transfer context for fragment data masters. Use * @DRM_PVR_CTX_TYPE_TRANSFER_FRAG: Transfer context for fragment data
* &struct drm_pvr_ioctl_create_transfer_context_args for context creation arguments. * master.
*/ */
DRM_PVR_CTX_TYPE_TRANSFER_FRAG, DRM_PVR_CTX_TYPE_TRANSFER_FRAG,
}; };