[PATCH 2/2] drm/atomic: Spell CRTC consistently

Thomas Zimmermann tzimmermann at suse.de
Thu Dec 5 17:03:55 UTC 2019


Both patches:

Reviewed-by: Thomas Zimmermann <tzimmermann at suse.de>

Am 05.12.19 um 17:49 schrieb Thierry Reding:
> From: Thierry Reding <treding at nvidia.com>
> 
> CRTC is an abbreviation and should be all caps in prose. Update all
> kerneldoc comments to use a consistent spelling.
> 
> Signed-off-by: Thierry Reding <treding at nvidia.com>
> ---
>  drivers/gpu/drm/drm_atomic.c        | 20 ++++-----
>  drivers/gpu/drm/drm_atomic_helper.c | 68 ++++++++++++++---------------
>  drivers/gpu/drm/drm_atomic_uapi.c   | 16 +++----
>  include/drm/drm_atomic.h            | 30 ++++++-------
>  include/drm/drm_atomic_helper.h     |  8 ++--
>  5 files changed, 71 insertions(+), 71 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
> index ab4508f25986..d33691512a8e 100644
> --- a/drivers/gpu/drm/drm_atomic.c
> +++ b/drivers/gpu/drm/drm_atomic.c
> @@ -251,7 +251,7 @@ EXPORT_SYMBOL(drm_atomic_state_clear);
>   * @ref: This atomic state to deallocate
>   *
>   * This frees all memory associated with an atomic state, including all the
> - * per-object state for planes, crtcs and connectors.
> + * per-object state for planes, CRTCs and connectors.
>   */
>  void __drm_atomic_state_free(struct kref *ref)
>  {
> @@ -272,12 +272,12 @@ void __drm_atomic_state_free(struct kref *ref)
>  EXPORT_SYMBOL(__drm_atomic_state_free);
>  
>  /**
> - * drm_atomic_get_crtc_state - get crtc state
> + * drm_atomic_get_crtc_state - get CRTC state
>   * @state: global atomic state object
> - * @crtc: crtc to get state object for
> + * @crtc: CRTC to get state object for
>   *
> - * This function returns the crtc state for the given crtc, allocating it if
> - * needed. It will also grab the relevant crtc lock to make sure that the state
> + * This function returns the CRTC state for the given CRTC, allocating it if
> + * needed. It will also grab the relevant CRTC lock to make sure that the state
>   * is consistent.
>   *
>   * Returns:
> @@ -1018,14 +1018,14 @@ static void drm_atomic_connector_print_state(struct drm_printer *p,
>  }
>  
>  /**
> - * drm_atomic_add_affected_connectors - add connectors for crtc
> + * drm_atomic_add_affected_connectors - add connectors for CRTC
>   * @state: atomic state
> - * @crtc: DRM crtc
> + * @crtc: DRM CRTC
>   *
>   * This function walks the current configuration and adds all connectors
>   * currently using @crtc to the atomic configuration @state. Note that this
>   * function must acquire the connection mutex. This can potentially cause
> - * unneeded seralization if the update is just for the planes on one crtc. Hence
> + * unneeded seralization if the update is just for the planes on one CRTC. Hence
>   * drivers and helpers should only call this when really needed (e.g. when a
>   * full modeset needs to happen due to some change).
>   *
> @@ -1078,9 +1078,9 @@ drm_atomic_add_affected_connectors(struct drm_atomic_state *state,
>  EXPORT_SYMBOL(drm_atomic_add_affected_connectors);
>  
>  /**
> - * drm_atomic_add_affected_planes - add planes for crtc
> + * drm_atomic_add_affected_planes - add planes for CRTC
>   * @state: atomic state
> - * @crtc: DRM crtc
> + * @crtc: DRM CRTC
>   *
>   * This function walks the current configuration and adds all planes
>   * currently used by @crtc to the atomic configuration @state. This is useful
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index 7bf245aa2d05..c00fcfc2406c 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -150,8 +150,8 @@ static int handle_conflicting_encoders(struct drm_atomic_state *state,
>  	 * is not set, an error is returned. Userspace can provide a solution
>  	 * through the atomic ioctl.
>  	 *
> -	 * If the flag is set conflicting connectors are removed from the crtc
> -	 * and the crtc is disabled if no encoder is left. This preserves
> +	 * If the flag is set conflicting connectors are removed from the CRTC
> +	 * and the CRTC is disabled if no encoder is left. This preserves
>  	 * compatibility with the legacy set_config behavior.
>  	 */
>  	drm_connector_list_iter_begin(state->dev, &conn_iter);
> @@ -556,27 +556,27 @@ mode_valid(struct drm_atomic_state *state)
>   * @state: the driver state object
>   *
>   * Check the state object to see if the requested state is physically possible.
> - * This does all the crtc and connector related computations for an atomic
> + * This does all the CRTC and connector related computations for an atomic
>   * update and adds any additional connectors needed for full modesets. It calls
>   * the various per-object callbacks in the follow order:
>   *
>   * 1. &drm_connector_helper_funcs.atomic_best_encoder for determining the new encoder.
>   * 2. &drm_connector_helper_funcs.atomic_check to validate the connector state.
>   * 3. If it's determined a modeset is needed then all connectors on the affected
> - *    crtc are added and &drm_connector_helper_funcs.atomic_check is run on them.
> + *    CRTC are added and &drm_connector_helper_funcs.atomic_check is run on them.
>   * 4. &drm_encoder_helper_funcs.mode_valid, &drm_bridge_funcs.mode_valid and
>   *    &drm_crtc_helper_funcs.mode_valid are called on the affected components.
>   * 5. &drm_bridge_funcs.mode_fixup is called on all encoder bridges.
>   * 6. &drm_encoder_helper_funcs.atomic_check is called to validate any encoder state.
> - *    This function is only called when the encoder will be part of a configured crtc,
> + *    This function is only called when the encoder will be part of a configured CRTC,
>   *    it must not be used for implementing connector property validation.
>   *    If this function is NULL, &drm_atomic_encoder_helper_funcs.mode_fixup is called
>   *    instead.
> - * 7. &drm_crtc_helper_funcs.mode_fixup is called last, to fix up the mode with crtc constraints.
> + * 7. &drm_crtc_helper_funcs.mode_fixup is called last, to fix up the mode with CRTC constraints.
>   *
>   * &drm_crtc_state.mode_changed is set when the input mode is changed.
>   * &drm_crtc_state.connectors_changed is set when a connector is added or
> - * removed from the crtc.  &drm_crtc_state.active_changed is set when
> + * removed from the CRTC.  &drm_crtc_state.active_changed is set when
>   * &drm_crtc_state.active changes, which is used for DPMS.
>   * See also: drm_atomic_crtc_needs_modeset()
>   *
> @@ -687,7 +687,7 @@ drm_atomic_helper_check_modeset(struct drm_device *dev,
>  
>  	/*
>  	 * After all the routing has been prepared we need to add in any
> -	 * connector which is itself unchanged, but whose crtc changes its
> +	 * connector which is itself unchanged, but whose CRTC changes its
>  	 * configuration. This must be done before calling mode_fixup in case a
>  	 * crtc only changed its mode but has the same set of connectors.
>  	 */
> @@ -736,13 +736,13 @@ EXPORT_SYMBOL(drm_atomic_helper_check_modeset);
>  /**
>   * drm_atomic_helper_check_plane_state() - Check plane state for validity
>   * @plane_state: plane state to check
> - * @crtc_state: crtc state to check
> + * @crtc_state: CRTC state to check
>   * @min_scale: minimum @src:@dest scaling factor in 16.16 fixed point
>   * @max_scale: maximum @src:@dest scaling factor in 16.16 fixed point
>   * @can_position: is it legal to position the plane such that it
> - *                doesn't cover the entire crtc?  This will generally
> + *                doesn't cover the entire CRTC?  This will generally
>   *                only be false for primary planes.
> - * @can_update_disabled: can the plane be updated while the crtc
> + * @can_update_disabled: can the plane be updated while the CRTC
>   *                       is disabled?
>   *
>   * Checks that a desired plane update is valid, and updates various
> @@ -839,7 +839,7 @@ EXPORT_SYMBOL(drm_atomic_helper_check_plane_state);
>   * &drm_crtc_helper_funcs.atomic_check and &drm_plane_helper_funcs.atomic_check
>   * hooks provided by the driver.
>   *
> - * It also sets &drm_crtc_state.planes_changed to indicate that a crtc has
> + * It also sets &drm_crtc_state.planes_changed to indicate that a CRTC has
>   * updated planes.
>   *
>   * RETURNS:
> @@ -903,7 +903,7 @@ EXPORT_SYMBOL(drm_atomic_helper_check_planes);
>   * @state: the driver state object
>   *
>   * Check the state object to see if the requested state is physically possible.
> - * Only crtcs and planes have check callbacks, so for any additional (global)
> + * Only CRTCs and planes have check callbacks, so for any additional (global)
>   * checking that a driver needs it can simply wrap that around this function.
>   * Drivers without such needs can directly use this as their
>   * &drm_mode_config_funcs.atomic_check callback.
> @@ -956,14 +956,14 @@ crtc_needs_disable(struct drm_crtc_state *old_state,
>  		   struct drm_crtc_state *new_state)
>  {
>  	/*
> -	 * No new_state means the crtc is off, so the only criteria is whether
> +	 * No new_state means the CRTC is off, so the only criteria is whether
>  	 * it's currently active or in self refresh mode.
>  	 */
>  	if (!new_state)
>  		return drm_atomic_crtc_effectively_active(old_state);
>  
>  	/*
> -	 * We need to run through the crtc_funcs->disable() function if the crtc
> +	 * We need to run through the crtc_funcs->disable() function if the CRTC
>  	 * is currently on, if it's transitioning to self refresh mode, or if
>  	 * it's in self refresh mode and needs to be fully disabled.
>  	 */
> @@ -1080,7 +1080,7 @@ disable_outputs(struct drm_device *dev, struct drm_atomic_state *old_state)
>   * @old_state: atomic state object with old state structures
>   *
>   * This function updates all the various legacy modeset state pointers in
> - * connectors, encoders and crtcs. It also updates the timestamping constants
> + * connectors, encoders and CRTCs. It also updates the timestamping constants
>   * used for precise vblank timestamps by calling
>   * drm_calc_timestamping_constants().
>   *
> @@ -1227,7 +1227,7 @@ crtc_set_mode(struct drm_device *dev, struct drm_atomic_state *old_state)
>   * This function shuts down all the outputs that need to be shut down and
>   * prepares them (if required) with the new mode.
>   *
> - * For compatibility with legacy crtc helpers this should be called before
> + * For compatibility with legacy CRTC helpers this should be called before
>   * drm_atomic_helper_commit_planes(), which is what the default commit function
>   * does. But drivers with different needs can group the modeset commits together
>   * and do the plane commits at the end. This is useful for drivers doing runtime
> @@ -1273,7 +1273,7 @@ static void drm_atomic_helper_commit_writebacks(struct drm_device *dev,
>   * This function enables all the outputs with the new configuration which had to
>   * be turned off for the update.
>   *
> - * For compatibility with legacy crtc helpers this should be called after
> + * For compatibility with legacy CRTC helpers this should be called after
>   * drm_atomic_helper_commit_planes(), which is what the default commit function
>   * does. But drivers with different needs can group the modeset commits together
>   * and do the plane commits at the end. This is useful for drivers doing runtime
> @@ -1403,12 +1403,12 @@ int drm_atomic_helper_wait_for_fences(struct drm_device *dev,
>  EXPORT_SYMBOL(drm_atomic_helper_wait_for_fences);
>  
>  /**
> - * drm_atomic_helper_wait_for_vblanks - wait for vblank on crtcs
> + * drm_atomic_helper_wait_for_vblanks - wait for vblank on CRTCs
>   * @dev: DRM device
>   * @old_state: atomic state object with old state structures
>   *
>   * Helper to, after atomic commit, wait for vblanks on all affected
> - * crtcs (ie. before cleaning up old framebuffers using
> + * CRTCs (ie. before cleaning up old framebuffers using
>   * drm_atomic_helper_cleanup_planes()). It will only wait on CRTCs where the
>   * framebuffers have actually changed to optimize for the legacy cursor and
>   * plane update use-case.
> @@ -1597,7 +1597,7 @@ static void commit_tail(struct drm_atomic_state *old_state)
>  
>  	/*
>  	 * We cannot safely access new_crtc_state after
> -	 * drm_atomic_helper_commit_hw_done() so figure out which crtc's have
> +	 * drm_atomic_helper_commit_hw_done() so figure out which CRTCs have
>  	 * self-refresh active beforehand:
>  	 */
>  	for_each_new_crtc_in_state(old_state, crtc, new_crtc_state, i)
> @@ -2197,7 +2197,7 @@ EXPORT_SYMBOL(drm_atomic_helper_wait_for_dependencies);
>   * drm_atomic_helper_fake_vblank - fake VBLANK events if needed
>   * @old_state: atomic state object with old state structures
>   *
> - * This function walks all CRTCs and fake VBLANK events on those with
> + * This function walks all CRTCs and fakes VBLANK events on those with

This fix could also go into patch 1.

>   * &drm_crtc_state.no_vblank set to true and &drm_crtc_state.event != NULL.
>   * The primary use of this function is writeback connectors working in oneshot
>   * mode and faking VBLANK events. In this case they only fake the VBLANK event
> @@ -2393,7 +2393,7 @@ static bool plane_crtc_active(const struct drm_plane_state *state)
>   * @flags: flags for committing plane state
>   *
>   * This function commits the new plane state using the plane and atomic helper
> - * functions for planes and crtcs. It assumes that the atomic state has already
> + * functions for planes and CRTCs. It assumes that the atomic state has already
>   * been pushed into the relevant object state pointers, since this step can no
>   * longer fail.
>   *
> @@ -2514,15 +2514,15 @@ void drm_atomic_helper_commit_planes(struct drm_device *dev,
>  EXPORT_SYMBOL(drm_atomic_helper_commit_planes);
>  
>  /**
> - * drm_atomic_helper_commit_planes_on_crtc - commit plane state for a crtc
> - * @old_crtc_state: atomic state object with the old crtc state
> + * drm_atomic_helper_commit_planes_on_crtc - commit plane state for a CRTC
> + * @old_crtc_state: atomic state object with the old CRTC state
>   *
>   * This function commits the new plane state using the plane and atomic helper
> - * functions for planes on the specific crtc. It assumes that the atomic state
> + * functions for planes on the specific CRTC. It assumes that the atomic state
>   * has already been pushed into the relevant object state pointers, since this
>   * step can no longer fail.
>   *
> - * This function is useful when plane updates should be done crtc-by-crtc
> + * This function is useful when plane updates should be done CRTC-by-CRTC
>   * instead of one global step like drm_atomic_helper_commit_planes() does.
>   *
>   * This function can only be savely used when planes are not allowed to move
> @@ -2812,10 +2812,10 @@ EXPORT_SYMBOL(drm_atomic_helper_swap_state);
>   * @plane: plane object to update
>   * @crtc: owning CRTC of owning plane
>   * @fb: framebuffer to flip onto plane
> - * @crtc_x: x offset of primary plane on crtc
> - * @crtc_y: y offset of primary plane on crtc
> - * @crtc_w: width of primary plane rectangle on crtc
> - * @crtc_h: height of primary plane rectangle on crtc
> + * @crtc_x: x offset of primary plane on @crtc
> + * @crtc_y: y offset of primary plane on @crtc
> + * @crtc_w: width of primary plane rectangle on @crtc
> + * @crtc_h: height of primary plane rectangle on @crtc
>   * @src_x: x offset of @fb for panning
>   * @src_y: y offset of @fb for panning
>   * @src_w: width of source rectangle in @fb
> @@ -2921,7 +2921,7 @@ EXPORT_SYMBOL(drm_atomic_helper_disable_plane);
>   * @set: mode set configuration
>   * @ctx: lock acquisition context
>   *
> - * Provides a default crtc set_config handler using the atomic driver interface.
> + * Provides a default CRTC set_config handler using the atomic driver interface.
>   *
>   * NOTE: For backwards compatibility with old userspace this automatically
>   * resets the "link-status" property to GOOD, to force any link
> @@ -3334,7 +3334,7 @@ static int page_flip_common(struct drm_atomic_state *state,
>  
>  /**
>   * drm_atomic_helper_page_flip - execute a legacy page flip
> - * @crtc: DRM crtc
> + * @crtc: DRM CRTC
>   * @fb: DRM framebuffer
>   * @event: optional DRM event to signal upon completion
>   * @flags: flip flags for non-vblank sync'ed updates
> @@ -3378,7 +3378,7 @@ EXPORT_SYMBOL(drm_atomic_helper_page_flip);
>  
>  /**
>   * drm_atomic_helper_page_flip_target - do page flip on target vblank period.
> - * @crtc: DRM crtc
> + * @crtc: DRM CRTC
>   * @fb: DRM framebuffer
>   * @event: optional DRM event to signal upon completion
>   * @flags: flip flags for non-vblank sync'ed updates
> diff --git a/drivers/gpu/drm/drm_atomic_uapi.c b/drivers/gpu/drm/drm_atomic_uapi.c
> index 0d466d3b0809..a1e5e262bae2 100644
> --- a/drivers/gpu/drm/drm_atomic_uapi.c
> +++ b/drivers/gpu/drm/drm_atomic_uapi.c
> @@ -160,12 +160,12 @@ int drm_atomic_set_mode_prop_for_crtc(struct drm_crtc_state *state,
>  EXPORT_SYMBOL(drm_atomic_set_mode_prop_for_crtc);
>  
>  /**
> - * drm_atomic_set_crtc_for_plane - set crtc for plane
> + * drm_atomic_set_crtc_for_plane - set CRTC for plane
>   * @plane_state: the plane whose incoming state to update
> - * @crtc: crtc to use for the plane
> + * @crtc: CRTC to use for the plane
>   *
> - * Changing the assigned crtc for a plane requires us to grab the lock and state
> - * for the new crtc, as needed. This function takes care of all these details
> + * Changing the assigned CRTC for a plane requires us to grab the lock and state
> + * for the new CRTC, as needed. This function takes care of all these details
>   * besides updating the pointer in the state object itself.
>   *
>   * Returns:
> @@ -279,12 +279,12 @@ drm_atomic_set_fence_for_plane(struct drm_plane_state *plane_state,
>  EXPORT_SYMBOL(drm_atomic_set_fence_for_plane);
>  
>  /**
> - * drm_atomic_set_crtc_for_connector - set crtc for connector
> + * drm_atomic_set_crtc_for_connector - set CRTC for connector
>   * @conn_state: atomic state object for the connector
> - * @crtc: crtc to use for the connector
> + * @crtc: CRTC to use for the connector
>   *
> - * Changing the assigned crtc for a connector requires us to grab the lock and
> - * state for the new crtc, as needed. This function takes care of all these
> + * Changing the assigned CRTC for a connector requires us to grab the lock and
> + * state for the new CRTC, as needed. This function takes care of all these
>   * details besides updating the pointer in the state object itself.
>   *
>   * Returns:
> diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
> index 1c10e0d667e6..e43f9e0a9964 100644
> --- a/include/drm/drm_atomic.h
> +++ b/include/drm/drm_atomic.h
> @@ -363,7 +363,7 @@ struct drm_atomic_state {
>  	 * When a connector or plane is not bound to any CRTC, it's still important
>  	 * to preserve linearity to prevent the atomic states from being freed to early.
>  	 *
> -	 * This commit (if set) is not bound to any crtc, but will be completed when
> +	 * This commit (if set) is not bound to any CRTC, but will be completed when
>  	 * drm_atomic_helper_commit_hw_done() is called.
>  	 */
>  	struct drm_crtc_commit *fake_commit;
> @@ -476,12 +476,12 @@ drm_atomic_get_new_connector_for_encoder(struct drm_atomic_state *state,
>  					 struct drm_encoder *encoder);
>  
>  /**
> - * drm_atomic_get_existing_crtc_state - get crtc state, if it exists
> + * drm_atomic_get_existing_crtc_state - get CRTC state, if it exists
>   * @state: global atomic state object
> - * @crtc: crtc to grab
> + * @crtc: CRTC to grab
>   *
> - * This function returns the crtc state for the given crtc, or NULL
> - * if the crtc is not part of the global atomic state.
> + * This function returns the CRTC state for the given CRTC, or NULL
> + * if the CRTC is not part of the global atomic state.
>   *
>   * This function is deprecated, @drm_atomic_get_old_crtc_state or
>   * @drm_atomic_get_new_crtc_state should be used instead.
> @@ -494,12 +494,12 @@ drm_atomic_get_existing_crtc_state(struct drm_atomic_state *state,
>  }
>  
>  /**
> - * drm_atomic_get_old_crtc_state - get old crtc state, if it exists
> + * drm_atomic_get_old_crtc_state - get old CRTC state, if it exists
>   * @state: global atomic state object
> - * @crtc: crtc to grab
> + * @crtc: CRTC to grab
>   *
> - * This function returns the old crtc state for the given crtc, or
> - * NULL if the crtc is not part of the global atomic state.
> + * This function returns the old CRTC state for the given CRTC, or
> + * NULL if the CRTC is not part of the global atomic state.
>   */
>  static inline struct drm_crtc_state *
>  drm_atomic_get_old_crtc_state(struct drm_atomic_state *state,
> @@ -508,12 +508,12 @@ drm_atomic_get_old_crtc_state(struct drm_atomic_state *state,
>  	return state->crtcs[drm_crtc_index(crtc)].old_state;
>  }
>  /**
> - * drm_atomic_get_new_crtc_state - get new crtc state, if it exists
> + * drm_atomic_get_new_crtc_state - get new CRTC state, if it exists
>   * @state: global atomic state object
> - * @crtc: crtc to grab
> + * @crtc: CRTC to grab
>   *
> - * This function returns the new crtc state for the given crtc, or
> - * NULL if the crtc is not part of the global atomic state.
> + * This function returns the new CRTC state for the given CRTC, or
> + * NULL if the CRTC is not part of the global atomic state.
>   */
>  static inline struct drm_crtc_state *
>  drm_atomic_get_new_crtc_state(struct drm_atomic_state *state,
> @@ -967,11 +967,11 @@ drm_atomic_crtc_needs_modeset(const struct drm_crtc_state *state)
>  }
>  
>  /**
> - * drm_atomic_crtc_effectively_active - compute whether crtc is actually active
> + * drm_atomic_crtc_effectively_active - compute whether CRTC is actually active
>   * @state: &drm_crtc_state for the CRTC
>   *
>   * When in self refresh mode, the crtc_state->active value will be false, since
> - * the crtc is off. However in some cases we're interested in whether the crtc
> + * the CRTC is off. However in some cases we're interested in whether the CRTC
>   * is active, or effectively active (ie: it's connected to an active display).
>   * In these cases, use this function instead of just checking active.
>   */
> diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
> index bf4e07141d81..9db3cac48f4f 100644
> --- a/include/drm/drm_atomic_helper.h
> +++ b/include/drm/drm_atomic_helper.h
> @@ -152,7 +152,7 @@ int drm_atomic_helper_legacy_gamma_set(struct drm_crtc *crtc,
>  /**
>   * drm_atomic_crtc_for_each_plane - iterate over planes currently attached to CRTC
>   * @plane: the loop cursor
> - * @crtc:  the crtc whose planes are iterated
> + * @crtc:  the CRTC whose planes are iterated
>   *
>   * This iterates over the current state, useful (for example) when applying
>   * atomic state after it has been checked and swapped.  To iterate over the
> @@ -166,7 +166,7 @@ int drm_atomic_helper_legacy_gamma_set(struct drm_crtc *crtc,
>  /**
>   * drm_crtc_atomic_state_for_each_plane - iterate over attached planes in new state
>   * @plane: the loop cursor
> - * @crtc_state: the incoming crtc-state
> + * @crtc_state: the incoming CRTC state
>   *
>   * Similar to drm_crtc_for_each_plane(), but iterates the planes that will be
>   * attached if the specified state is applied.  Useful during for example
> @@ -180,7 +180,7 @@ int drm_atomic_helper_legacy_gamma_set(struct drm_crtc *crtc,
>   * drm_crtc_atomic_state_for_each_plane_state - iterate over attached planes in new state
>   * @plane: the loop cursor
>   * @plane_state: loop cursor for the plane's state, must be const
> - * @crtc_state: the incoming crtc-state
> + * @crtc_state: the incoming CRTC state
>   *
>   * Similar to drm_crtc_for_each_plane(), but iterates the planes that will be
>   * attached if the specified state is applied.  Useful during for example
> @@ -189,7 +189,7 @@ int drm_atomic_helper_legacy_gamma_set(struct drm_crtc *crtc,
>   *
>   * Compared to just drm_atomic_crtc_state_for_each_plane() this also fills in a
>   * const plane_state. This is useful when a driver just wants to peek at other
> - * active planes on this crtc, but does not need to change it.
> + * active planes on this CRTC, but does not need to change it.
>   */
>  #define drm_atomic_crtc_state_for_each_plane_state(plane, plane_state, crtc_state) \
>  	drm_for_each_plane_mask(plane, (crtc_state)->state->dev, (crtc_state)->plane_mask) \
> 

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20191205/336c033c/attachment.sig>


More information about the dri-devel mailing list