[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-07 Thread Daniel Vetter
Since I already applied Thierry's interface change I've added the unsigned here as a fixup and dropped patch 2 (since I have that already). Thanks, Daniel On Wed, Oct 07, 2015 at 02:54:24PM +0200, Vincent ABRIOU wrote: > Reviewed-by: Vincent Abriou > > On 10/02/2015 02:48 PM, Ville Syrjälä

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-07 Thread Vincent ABRIOU
Reviewed-by: Vincent Abriou On 10/02/2015 02:48 PM, Ville Syrjälä wrote: > On Fri, Oct 02, 2015 at 10:25:27AM +0200, Vincent ABRIOU wrote: >> >> >> On 09/30/2015 04:14 PM, Ville Syrjälä wrote: >>> On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: On Wed, Sep 30, 2015 at

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-02 Thread Ville Syrjälä
On Fri, Oct 02, 2015 at 10:25:27AM +0200, Vincent ABRIOU wrote: > > > On 09/30/2015 04:14 PM, Ville Syrjälä wrote: > > On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: > >> On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala at linux.intel.com > >> wrote: > >>> From: Ville

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-02 Thread Vincent ABRIOU
On 10/02/2015 02:48 PM, Ville Syrjälä wrote: > On Fri, Oct 02, 2015 at 10:25:27AM +0200, Vincent ABRIOU wrote: >> >> >> On 09/30/2015 04:14 PM, Ville Syrjälä wrote: >>> On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-02 Thread Vincent ABRIOU
On 09/30/2015 04:14 PM, Ville Syrjälä wrote: > On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: >> On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala at linux.intel.com >> wrote: >>> From: Ville Syrjälä >>> >>> drm_vblank_count() returns the software counter. We should

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-01 Thread Ville Syrjälä
On Thu, Oct 01, 2015 at 06:44:22PM +0300, Ville Syrjälä wrote: > On Thu, Oct 01, 2015 at 11:25:36AM -0400, Rob Clark wrote: > > On Wed, Sep 30, 2015 at 10:14 AM, Ville Syrjälä > > wrote: > > > On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: > > >> On Wed, Sep 30, 2015 at

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-01 Thread Ville Syrjälä
On Thu, Oct 01, 2015 at 11:25:36AM -0400, Rob Clark wrote: > On Wed, Sep 30, 2015 at 10:14 AM, Ville Syrjälä > wrote: > > On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: > >> On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala at linux.intel.com > >> wrote: > >> > From:

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-01 Thread Rob Clark
On Thu, Oct 1, 2015 at 11:47 AM, Ville Syrjälä wrote: > On Thu, Oct 01, 2015 at 06:44:22PM +0300, Ville Syrjälä wrote: >> On Thu, Oct 01, 2015 at 11:25:36AM -0400, Rob Clark wrote: >> > On Wed, Sep 30, 2015 at 10:14 AM, Ville Syrjälä >> > wrote: >> > > On Wed, Sep 30, 2015 at 04:08:02PM

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-01 Thread Rob Clark
On Wed, Sep 30, 2015 at 10:14 AM, Ville Syrjälä wrote: > On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: >> On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala at linux.intel.com >> wrote: >> > From: Ville Syrjälä >> > >> > drm_vblank_count() returns the software counter.

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-10-01 Thread Vincent ABRIOU
On 09/30/2015 04:14 PM, Ville Syrjälä wrote: > On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: >> On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala at linux.intel.com >> wrote: >>> From: Ville Syrjälä >>> >>> drm_vblank_count() returns the software counter. We should

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-09-30 Thread Ville Syrjälä
On Wed, Sep 30, 2015 at 04:08:02PM +0200, Daniel Vetter wrote: > On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala at linux.intel.com > wrote: > > From: Ville Syrjälä > > > > drm_vblank_count() returns the software counter. We should not pretend > > it's the hw counter since we use the

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-09-30 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä drm_vblank_count() returns the software counter. We should not pretend it's the hw counter since we use the hw counter to figuere out what the software counter value should be. So instead provide a new function drm_vblank_no_hw_counter() for

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-09-30 Thread Vincent ABRIOU
> -Original Message- > From: dri-devel [mailto:dri-devel-bounces at lists.freedesktop.org] On Behalf > Of Ville Syrjälä > Sent: mercredi 30 septembre 2015 16:15 > To: Daniel Vetter > Cc: Thierry Reding; dri-devel at lists.freedesktop.org > Subject: Re: [PATCH

[PATCH 1/2] drm: Stop using drm_vblank_count() as the hw frame counter

2015-09-30 Thread Daniel Vetter
On Wed, Sep 30, 2015 at 04:46:48PM +0300, ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > drm_vblank_count() returns the software counter. We should not pretend > it's the hw counter since we use the hw counter to figuere out what the > software counter value should be. So