[PATCH 01/21] drm/doc: Fix more kerneldoc/sphinx warnings
Sean Paul
seanpaul at google.com
Tue Aug 16 13:42:33 UTC 2016
On Aug 12, 2016 1:49 PM, "Daniel Vetter" <daniel.vetter at ffwll.ch> wrote:
>
> These are the leftovers I could only track down using keep_warnings =
> True. For some of them we might want to update our style guide on how
> to reference structures and constants, not sure ...
>
> Cc: Markus Heiser <markus.heiser at darmarit.de>
> Cc: Jonathan Corbet <corbet at lwn.net>
> Cc: linux-doc at vger.kernel.org
> Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
Reviewed-by: Sean Paul <seanpaul at chromium.org>
> ---
> drivers/gpu/drm/drm_crtc.c | 4 ++--
> drivers/gpu/drm/drm_fb_helper.c | 2 +-
> drivers/gpu/drm/drm_irq.c | 8 +++----
> drivers/gpu/drm/drm_simple_kms_helper.c | 2 +-
> drivers/gpu/drm/i915/i915_vgpu.c | 42
++++++++++++++++-----------------
> drivers/gpu/drm/i915/intel_audio.c | 6 ++---
> drivers/gpu/drm/i915/intel_guc_fwif.h | 5 ++--
> include/drm/drm_crtc.h | 8 +++----
> include/drm/drm_gem.h | 4 ++--
> 9 files changed, 41 insertions(+), 40 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> index c31298f0bb0e..4cb8f4b49d8b 100644
> --- a/drivers/gpu/drm/drm_crtc.c
> +++ b/drivers/gpu/drm/drm_crtc.c
> @@ -1272,7 +1272,7 @@ static unsigned int drm_num_planes(struct
drm_device *dev)
> * @plane: plane object to init
> * @possible_crtcs: bitmask of possible CRTCs
> * @funcs: callbacks for the new plane
> - * @formats: array of supported formats (%DRM_FORMAT_*)
> + * @formats: array of supported formats (DRM_FORMAT\_\*)
> * @format_count: number of elements in @formats
> * @type: type of plane (overlay, primary, cursor)
> * @name: printf style format string for the plane name, or NULL for
default name
> @@ -1387,7 +1387,7 @@ static void drm_plane_unregister_all(struct
drm_device *dev)
> * @plane: plane object to init
> * @possible_crtcs: bitmask of possible CRTCs
> * @funcs: callbacks for the new plane
> - * @formats: array of supported formats (%DRM_FORMAT_*)
> + * @formats: array of supported formats (DRM_FORMAT\_\*)
> * @format_count: number of elements in @formats
> * @is_primary: plane type (primary vs overlay)
> *
> diff --git a/drivers/gpu/drm/drm_fb_helper.c
b/drivers/gpu/drm/drm_fb_helper.c
> index aed79d31930c..c0d1066ea419 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -2193,7 +2193,7 @@ EXPORT_SYMBOL(drm_fb_helper_initial_config);
> * @fb_helper: the drm_fb_helper
> *
> * Scan the connectors attached to the fb_helper and try to put together
a
> - * setup after *notification of a change in output configuration.
> + * setup after notification of a change in output configuration.
> *
> * Called at runtime, takes the mode config locks to be able to
check/change the
> * modeset configuration. Must be run from process context (which
usually means
> diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c
> index 9bdce1cb6c5c..10611a936059 100644
> --- a/drivers/gpu/drm/drm_irq.c
> +++ b/drivers/gpu/drm/drm_irq.c
> @@ -713,10 +713,10 @@ EXPORT_SYMBOL(drm_calc_timestamping_constants);
> * Negative value on error, failure or if not supported in current
> * video mode:
> *
> - * -EINVAL - Invalid CRTC.
> - * -EAGAIN - Temporary unavailable, e.g., called before initial
modeset.
> - * -ENOTSUPP - Function not supported in current display mode.
> - * -EIO - Failed, e.g., due to failed scanout position query.
> + * -EINVAL Invalid CRTC.
> + * -EAGAIN Temporary unavailable, e.g., called before initial modeset.
> + * -ENOTSUPP Function not supported in current display mode.
> + * -EIO Failed, e.g., due to failed scanout position query.
> *
> * Returns or'ed positive status flags on success:
> *
> diff --git a/drivers/gpu/drm/drm_simple_kms_helper.c
b/drivers/gpu/drm/drm_simple_kms_helper.c
> index 0a02efe978ee..bada17166512 100644
> --- a/drivers/gpu/drm/drm_simple_kms_helper.c
> +++ b/drivers/gpu/drm/drm_simple_kms_helper.c
> @@ -137,7 +137,7 @@ static const struct drm_plane_funcs
drm_simple_kms_plane_funcs = {
> * @dev: DRM device
> * @pipe: simple display pipe object to initialize
> * @funcs: callbacks for the display pipe (optional)
> - * @formats: array of supported formats (%DRM_FORMAT_*)
> + * @formats: array of supported formats (DRM_FORMAT\_\*)
> * @format_count: number of elements in @formats
> * @connector: connector to attach and register
> *
> diff --git a/drivers/gpu/drm/i915/i915_vgpu.c
b/drivers/gpu/drm/i915/i915_vgpu.c
> index 142bac976919..ca2e91259948 100644
> --- a/drivers/gpu/drm/i915/i915_vgpu.c
> +++ b/drivers/gpu/drm/i915/i915_vgpu.c
> @@ -156,27 +156,27 @@ static int vgt_balloon_space(struct drm_mm *mm,
> * host point of view, the graphic address space is partitioned by
multiple
> * vGPUs in different VMs. ::
> *
> - * vGPU1 view Host view
> - * 0 ------> +-----------+ +-----------+
> - * ^ |###########| | vGPU3 |
> - * | |###########| +-----------+
> - * | |###########| | vGPU2 |
> - * | +-----------+ +-----------+
> - * mappable GM | available | ==> | vGPU1 |
> - * | +-----------+ +-----------+
> - * | |###########| | |
> - * v |###########| | Host |
> - * +=======+===========+ +===========+
> - * ^ |###########| | vGPU3 |
> - * | |###########| +-----------+
> - * | |###########| | vGPU2 |
> - * | +-----------+ +-----------+
> - * unmappable GM | available | ==> | vGPU1 |
> - * | +-----------+ +-----------+
> - * | |###########| | |
> - * | |###########| | Host |
> - * v |###########| | |
> - * total GM size ------> +-----------+ +-----------+
> + * vGPU1 view Host view
> + * 0 ------> +-----------+ +-----------+
> + * ^ |###########| | vGPU3 |
> + * | |###########| +-----------+
> + * | |###########| | vGPU2 |
> + * | +-----------+ +-----------+
> + * mappable GM | available | ==> | vGPU1 |
> + * | +-----------+ +-----------+
> + * | |###########| | |
> + * v |###########| | Host |
> + * +=======+===========+ +===========+
> + * ^ |###########| | vGPU3 |
> + * | |###########| +-----------+
> + * | |###########| | vGPU2 |
> + * | +-----------+ +-----------+
> + * unmappable GM | available | ==> | vGPU1 |
> + * | +-----------+ +-----------+
> + * | |###########| | |
> + * | |###########| | Host |
> + * v |###########| | |
> + * total GM size ------> +-----------+ +-----------+
> *
> * Returns:
> * zero on success, non-zero if configuration invalid or ballooning
failed
> diff --git a/drivers/gpu/drm/i915/intel_audio.c
b/drivers/gpu/drm/i915/intel_audio.c
> index d32f586f9c05..85389cdd0bec 100644
> --- a/drivers/gpu/drm/i915/intel_audio.c
> +++ b/drivers/gpu/drm/i915/intel_audio.c
> @@ -51,10 +51,10 @@
> * related registers. (The notable exception is the power management, not
> * covered here.)
> *
> - * The struct i915_audio_component is used to interact between the
graphics
> - * and audio drivers. The struct i915_audio_component_ops *ops in it is
> + * The struct &i915_audio_component is used to interact between the
graphics
> + * and audio drivers. The struct &i915_audio_component_ops @ops in it is
> * defined in graphics driver and called in audio driver. The
> - * struct i915_audio_component_audio_ops *audio_ops is called from i915
driver.
> + * struct &i915_audio_component_audio_ops @audio_ops is called from i915
driver.
> */
>
> static const struct {
> diff --git a/drivers/gpu/drm/i915/intel_guc_fwif.h
b/drivers/gpu/drm/i915/intel_guc_fwif.h
> index 944786d7075b..e40db2d2ae99 100644
> --- a/drivers/gpu/drm/i915/intel_guc_fwif.h
> +++ b/drivers/gpu/drm/i915/intel_guc_fwif.h
> @@ -155,6 +155,7 @@
> *
> * +-------------------------------+
> * | guc_css_header |
> + * | |
> * | contains major/minor version |
> * +-------------------------------+
> * | uCode |
> @@ -176,10 +177,10 @@
> *
> * 1. Header, uCode and RSA are must-have components.
> * 2. All firmware components, if they present, are in the sequence
illustrated
> - * in the layout table above.
> + * in the layout table above.
> * 3. Length info of each component can be found in header, in dwords.
> * 4. Modulus and exponent key are not required by driver. They may not
appear
> - * in fw. So driver will load a truncated firmware in this case.
> + * in fw. So driver will load a truncated firmware in this case.
> */
>
> struct guc_css_header {
> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> index b618b506b04d..194eebb2f9d7 100644
> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h
> @@ -1197,7 +1197,7 @@ struct drm_encoder_funcs {
> * @head: list management
> * @base: base KMS object
> * @name: human readable name, can be overwritten by the driver
> - * @encoder_type: one of the %DRM_MODE_ENCODER_<foo> types in drm_mode.h
> + * @encoder_type: one of the DRM_MODE_ENCODER_<foo> types in drm_mode.h
> * @possible_crtcs: bitmask of potential CRTC bindings
> * @possible_clones: bitmask of potential sibling encoders for cloning
> * @crtc: currently bound CRTC
> @@ -1250,7 +1250,7 @@ struct drm_encoder {
> * @head: list management
> * @base: base KMS object
> * @name: human readable name, can be overwritten by the driver
> - * @connector_type: one of the %DRM_MODE_CONNECTOR_<foo> types from
drm_mode.h
> + * @connector_type: one of the DRM_MODE_CONNECTOR_<foo> types from
drm_mode.h
> * @connector_type_id: index into connector type enum
> * @interlace_allowed: can this connector handle interlaced modes?
> * @doublescan_allowed: can this connector handle doublescan?
> @@ -1263,11 +1263,11 @@ struct drm_encoder {
> * @funcs: connector control functions
> * @edid_blob_ptr: DRM property containing EDID if present
> * @properties: property tracking for this connector
> - * @polled: a %DRM_CONNECTOR_POLL_<foo> value for core driven polling
> + * @polled: a DRM_CONNECTOR_POLL_<foo> value for core driven polling
> * @dpms: current dpms state
> * @helper_private: mid-layer private data
> * @cmdline_mode: mode line parsed from the kernel cmdline for this
connector
> - * @force: a %DRM_FORCE_<foo> state for forced mode sets
> + * @force: a DRM_FORCE_<foo> state for forced mode sets
> * @override_edid: has the EDID been overwritten through debugfs for
testing?
> * @encoder_ids: valid encoders for this connector
> * @encoder: encoder driving this connector, if any
> diff --git a/include/drm/drm_gem.h b/include/drm/drm_gem.h
> index fca1cd1b9c26..9f63736e6163 100644
> --- a/include/drm/drm_gem.h
> +++ b/include/drm/drm_gem.h
> @@ -210,8 +210,8 @@ drm_gem_object_reference(struct drm_gem_object *obj)
> * drm_gem_object_unreference_unlocked().
> *
> * Drivers should never call this directly in their code. Instead they
should
> - * wrap it up into a driver_gem_object_unreference(struct
driver_gem_object
> - * *obj) wrapper function, and use that. Shared code should never call
this, to
> + * wrap it up into a ``driver_gem_object_unreference(struct
driver_gem_object
> + * *obj)`` wrapper function, and use that. Shared code should never call
this, to
> * avoid breaking drivers by accident which still depend upon
dev->struct_mutex
> * locking.
> */
> --
> 2.8.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160816/a461c39f/attachment-0001.html>
More information about the dri-devel
mailing list