Re: [PATCH] drm/dsi: Make set_tear_scanline command consistent

[Daniel Vetter <daniel@ffwll.ch>]

On Mon, Jun 13, 2016 at 03:33:27PM +0200, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
> 
> Use a consistent name for the function that implements set_tear_scanline
> and reword and reformat the kerneldoc slightly.
> 
> Signed-off-by: Thierry Reding <treding@nvidia.com>

Applied to drm-misc.
-Daniel

> ---
>  drivers/gpu/drm/drm_mipi_dsi.c | 16 ++++++++--------
>  include/drm/drm_mipi_dsi.h     |  2 +-
>  2 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_mipi_dsi.c b/drivers/gpu/drm/drm_mipi_dsi.c
> index 7938ce7ebed8..5d70b1bec038 100644
> --- a/drivers/gpu/drm/drm_mipi_dsi.c
> +++ b/drivers/gpu/drm/drm_mipi_dsi.c
> @@ -983,17 +983,17 @@ int mipi_dsi_dcs_set_tear_on(struct mipi_dsi_device *dsi,
>  EXPORT_SYMBOL(mipi_dsi_dcs_set_tear_on);
>  
>  /**
> - * mipi_dsi_set_tear_scanline() - turn on the display module's Tearing Effect
> - * output signal on the TE signal line when display module reaches line N
> - * defined by STS[n:0].
> + * mipi_dsi_dcs_set_tear_scanline() - set the scanline to use as trigger for
> + *    the Tearing Effect output signal of the display module
>   * @dsi: DSI peripheral device
> - * @param: STS[10:0]
> + * @scanline: scanline to use as trigger
> + *
>   * Return: 0 on success or a negative error code on failure
>   */
> -int mipi_dsi_set_tear_scanline(struct mipi_dsi_device *dsi, u16 param)
> +int mipi_dsi_dcs_set_tear_scanline(struct mipi_dsi_device *dsi, u16 scanline)
>  {
> -	u8 payload[3] = { MIPI_DCS_SET_TEAR_SCANLINE, param >> 8,
> -			  param & 0xff };
> +	u8 payload[3] = { MIPI_DCS_SET_TEAR_SCANLINE, scanline >> 8,
> +			  scanline & 0xff };
>  	ssize_t err;
>  
>  	err = mipi_dsi_generic_write(dsi, payload, sizeof(payload));
> @@ -1002,7 +1002,7 @@ int mipi_dsi_set_tear_scanline(struct mipi_dsi_device *dsi, u16 param)
>  
>  	return 0;
>  }
> -EXPORT_SYMBOL(mipi_dsi_set_tear_scanline);
> +EXPORT_SYMBOL(mipi_dsi_dcs_set_tear_scanline);
>  
>  /**
>   * mipi_dsi_dcs_set_pixel_format() - sets the pixel format for the RGB image
> diff --git a/include/drm/drm_mipi_dsi.h b/include/drm/drm_mipi_dsi.h
> index ec552854a8f8..6ccf144e300b 100644
> --- a/include/drm/drm_mipi_dsi.h
> +++ b/include/drm/drm_mipi_dsi.h
> @@ -263,7 +263,7 @@ int mipi_dsi_dcs_set_column_address(struct mipi_dsi_device *dsi, u16 start,
>  				    u16 end);
>  int mipi_dsi_dcs_set_page_address(struct mipi_dsi_device *dsi, u16 start,
>  				  u16 end);
> -int mipi_dsi_set_tear_scanline(struct mipi_dsi_device *dsi, u16 param);
> +int mipi_dsi_dcs_set_tear_scanline(struct mipi_dsi_device *dsi, u16 scanline);
>  int mipi_dsi_dcs_set_tear_off(struct mipi_dsi_device *dsi);
>  int mipi_dsi_dcs_set_tear_on(struct mipi_dsi_device *dsi,
>  			     enum mipi_dsi_dcs_tear_mode mode);
> -- 
> 2.8.3
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel