[PATCH] drm: idiot-proof vblank

2014-08-06 Thread Daniel Vetter
On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: > After spending slightly more time than I'd care to admit debugging the > various and presumably spectacular way things fail when you pass too low > a value to drm_vblank_init() (thanks console-lock for not letting me see > the carnage!),

[PATCH] drm: idiot-proof vblank

2014-08-06 Thread Daniel Vetter
On Wed, Aug 06, 2014 at 03:06:40PM -0400, Rob Clark wrote: > On Wed, Aug 6, 2014 at 2:12 PM, Ville Syrj?l? > wrote: > > On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: > >> After spending slightly more time than I'd care to admit debugging the > >> various and presumably spectacular

[PATCH] drm: idiot-proof vblank

2014-08-06 Thread Ville Syrjälä
On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: > After spending slightly more time than I'd care to admit debugging the > various and presumably spectacular way things fail when you pass too low > a value to drm_vblank_init() (thanks console-lock for not letting me see > the carnage!),

[PATCH] drm: idiot-proof vblank

2014-08-06 Thread Rob Clark
On Wed, Aug 6, 2014 at 4:28 PM, Daniel Vetter wrote: > On Wed, Aug 06, 2014 at 03:06:40PM -0400, Rob Clark wrote: >> On Wed, Aug 6, 2014 at 2:12 PM, Ville Syrj?l? >> wrote: >> > On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: >> >> After spending slightly more time than I'd care to

[PATCH] drm: idiot-proof vblank

2014-08-06 Thread Rob Clark
On Wed, Aug 6, 2014 at 2:12 PM, Ville Syrj?l? wrote: > On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: >> After spending slightly more time than I'd care to admit debugging the >> various and presumably spectacular way things fail when you pass too low >> a value to drm_vblank_init()

[PATCH] drm: idiot-proof vblank

2014-08-06 Thread Rob Clark
After spending slightly more time than I'd care to admit debugging the various and presumably spectacular way things fail when you pass too low a value to drm_vblank_init() (thanks console-lock for not letting me see the carnage!), I decided it might be a good idea to add some sanity checking.