Re: [PATCH 10/16] drm/vblank: Remove DRM_VBLANKTIME_IN_VBLANK

From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Daniel Vetter <daniel.vetter@intel.com>,
	Intel Graphics Development <intel-gfx@lists.freedesktop.org>,
	DRI Development <dri-devel@lists.freedesktop.org>,
	Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
Subject: Re: [PATCH 10/16] drm/vblank: Remove DRM_VBLANKTIME_IN_VBLANK
Date: Wed, 22 Mar 2017 15:49:12 +0200	[thread overview]
Message-ID: <20170322134912.GF31595@intel.com> (raw)
In-Reply-To: <20170322083617.13361-11-daniel.vetter@ffwll.ch>

On Wed, Mar 22, 2017 at 09:36:11AM +0100, Daniel Vetter wrote:
> The core code doesn't care at all about this, it's entirely dead.
>
> Cc: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
>  drivers/gpu/drm/drm_irq.c | 8 --------
>  include/drm/drmP.h        | 1 -
>  2 files changed, 9 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c
> index 53a526c7b24d..4b0c7475ed13 100644
> --- a/drivers/gpu/drm/drm_irq.c
> +++ b/drivers/gpu/drm/drm_irq.c
> @@ -810,14 +810,6 @@ int drm_calc_vbltimestamp_from_scanoutpos(struct drm_device *dev,
>  	/* Return upper bound of timestamp precision error. */
>  	*max_error = duration_ns;
>  
> -	/* Check if in vblank area:
> -	 * vpos is >=0 in video scanout area, but negative
> -	 * within vblank area, counting down the number of lines until
> -	 * start of scanout.
> -	 */
> -	if (vbl_status & DRM_SCANOUTPOS_IN_VBLANK)
> -		ret |= DRM_VBLANKTIME_IN_VBLANK;
> -

Indeed nothing seems to look at it. And dropping it shouldn't change
anything as we still always set the other bit, so the 'ret > 0' check
in the caller should still work correctly.

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

>  	/* Convert scanout position into elapsed time at raw_time query
>  	 * since start of scanout at first display scanline. delta_ns
>  	 * can be negative if start of scanout hasn't happened yet.
> diff --git a/include/drm/drmP.h b/include/drm/drmP.h
> index 248d2408e56b..27ecfcdbf752 100644
> --- a/include/drm/drmP.h
> +++ b/include/drm/drmP.h
> @@ -323,7 +323,6 @@ struct pci_controller;
>  /* Flags and return codes for get_vblank_timestamp() driver function. */
>  #define DRM_CALLED_FROM_VBLIRQ 1
>  #define DRM_VBLANKTIME_SCANOUTPOS_METHOD (1 << 0)
> -#define DRM_VBLANKTIME_IN_VBLANK         (1 << 1)
>  
>  /* get_scanout_position() return flags */
>  #define DRM_SCANOUTPOS_VALID        (1 << 0)
> -- 
> 2.11.0
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx