[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-22 Thread Michel Dänzer
On Die, 2011-03-22 at 08:43 -0500, Ilija Hadzic wrote: > > On Tue, 22 Mar 2011, Michel [ISO-8859-1] Dnzer wrote: > > >> If _DRM_VBLANK_HIGH_CRTC_MASK were included in _DRM_VBLANK_FLAGS_MASK > >> (or _DRM_VBLANK_TYPES_MASK, but that would make less sense), these > >> changes shouldn't be

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-22 Thread Michel Dänzer
On Die, 2011-03-22 at 06:16 -0500, Ilija Hadzic wrote: > Unless I oversaw something nothing was silently ignored. I believe I > responded to each of your comments (and comments by others), those I > agreed with I implemented, those I didn't agree with I didn't implement. I haven't seen any

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-22 Thread Michel Dänzer
On Fre, 2011-03-18 at 16:58 -0500, Ilija Hadzic wrote: > > This patch extends the interface to drm_wait_vblank ioctl so that crtc>1 > can be represented. It also adds a new capability to drm_getcap ioctl so > that the user space can check whether the new interface to drm_wait_vblank > is

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-22 Thread Ilija Hadzic
Sorry about overseeing additional comments. It definitely wasn't intentional. On Tue, 22 Mar 2011, Michel [ISO-8859-1] D?nzer wrote: >> >> If _DRM_VBLANK_HIGH_CRTC_MASK were included in _DRM_VBLANK_FLAGS_MASK >> (or _DRM_VBLANK_TYPES_MASK, but that would make less sense), these >> changes

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-22 Thread Ilija Hadzic
Unless I oversaw something nothing was silently ignored. I believe I responded to each of your comments (and comments by others), those I agreed with I implemented, those I didn't agree with I didn't implement. -- Ilija On Tue, 22 Mar 2011, Michel [ISO-8859-1] D?nzer wrote: > On Fre,

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-22 Thread Michel Dänzer
On Fre, 2011-03-18 at 16:58 -0500, Ilija Hadzic wrote: This patch extends the interface to drm_wait_vblank ioctl so that crtc1 can be represented. It also adds a new capability to drm_getcap ioctl so that the user space can check whether the new interface to drm_wait_vblank is supported

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-22 Thread Ilija Hadzic
Unless I oversaw something nothing was silently ignored. I believe I responded to each of your comments (and comments by others), those I agreed with I implemented, those I didn't agree with I didn't implement. -- Ilija On Tue, 22 Mar 2011, Michel [ISO-8859-1] D�nzer wrote: On Fre,

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-22 Thread Michel Dänzer
On Die, 2011-03-22 at 06:16 -0500, Ilija Hadzic wrote: Unless I oversaw something nothing was silently ignored. I believe I responded to each of your comments (and comments by others), those I agreed with I implemented, those I didn't agree with I didn't implement. I haven't seen any

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-22 Thread Ilija Hadzic
Sorry about overseeing additional comments. It definitely wasn't intentional. On Tue, 22 Mar 2011, Michel [ISO-8859-1] D�nzer wrote: If _DRM_VBLANK_HIGH_CRTC_MASK were included in _DRM_VBLANK_FLAGS_MASK (or _DRM_VBLANK_TYPES_MASK, but that would make less sense), these changes shouldn't be

OT: sending patches to the list (was: [PATCH] kernel/drm: vblank wait on crtc > 1)

2011-03-21 Thread Paul Menzel
Am Sonntag, den 20.03.2011, 18:47 -0500 schrieb Ilija Hadzic: > sorry about that, I use pine and thought that's as plain as it gets. I > guess next time I'll try just 'mail' from command line. Or `git send-email`?. Thanks, Paul ? manual: `git help send-email` -- next part

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-21 Thread Dave Airlie
On Sat, Mar 19, 2011 at 7:58 AM, Ilija Hadzic wrote: > > Hi Dave, > > Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the > capability to wait on vblank events for CRTCs that are greater than 1 and > thus cannot be represented with primary/secondary flags in the legacy >

OT: sending patches to the list (was: [PATCH] kernel/drm: vblank wait on crtc 1)

2011-03-21 Thread Paul Menzel
Am Sonntag, den 20.03.2011, 18:47 -0500 schrieb Ilija Hadzic: sorry about that, I use pine and thought that's as plain as it gets. I guess next time I'll try just 'mail' from command line. Or `git send-email`¹. Thanks, Paul ¹ manual: `git help send-email` signature.asc Description: This

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-20 Thread Ilija Hadzic
sorry about that, I use pine and thought that's as plain as it gets. I guess next time I'll try just 'mail' from command line. On Mon, 21 Mar 2011, Dave Airlie wrote: > On Sat, Mar 19, 2011 at 7:58 AM, Ilija Hadzic > wrote: >> >> Hi Dave, >> >> Below is a patch against drm-next branch of

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-20 Thread Ilija Hadzic
On Sat, 19 Mar 2011, Alex Deucher wrote: > On Fri, Mar 18, 2011 at 5:58 PM, Ilija Hadzic > wrote: >> >> Hi Dave, >> >> Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the >> capability to wait on vblank events for CRTCs that are greater than 1 and >> thus cannot be

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-20 Thread Ilija Hadzic
On Sat, 19 Mar 2011, Alex Deucher wrote: On Fri, Mar 18, 2011 at 5:58 PM, Ilija Hadzic ihad...@research.bell-labs.com wrote: Hi Dave, Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the capability to wait on vblank events for CRTCs that are greater than 1 and thus

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-20 Thread Ilija Hadzic
sorry about that, I use pine and thought that's as plain as it gets. I guess next time I'll try just 'mail' from command line. On Mon, 21 Mar 2011, Dave Airlie wrote: On Sat, Mar 19, 2011 at 7:58 AM, Ilija Hadzic ihad...@research.bell-labs.com wrote: Hi Dave, Below is a patch against

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-19 Thread Mario Kleiner
> > On Fri, 18 Mar 2011, Jesse Barnes wrote: >> >> I like the new param check, but I'd still prefer a new ioctl to >> abusing >> the old one. >> > > It's not "abusing" it but extending the interface in a > backwards-compatible manner. Introducing a new one would result in two > ioctls that

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-19 Thread Alex Deucher
On Fri, Mar 18, 2011 at 5:58 PM, Ilija Hadzic wrote: > > Hi Dave, > > Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the > capability to wait on vblank events for CRTCs that are greater than 1 and > thus cannot be represented with primary/secondary flags in the legacy >

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-19 Thread Alex Deucher
On Fri, Mar 18, 2011 at 5:58 PM, Ilija Hadzic wrote: > > Hi Dave, > > Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the > capability to wait on vblank events for CRTCs that are greater than 1 and > thus cannot be represented with primary/secondary flags in the legacy >

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-19 Thread Alex Deucher
On Sat, Mar 19, 2011 at 1:30 PM, Mario Kleiner wrote: >> >> On Fri, 18 Mar 2011, Jesse Barnes wrote: >>> >>> I like the new param check, but I'd still prefer a new ioctl to abusing >>> the old one. >>> >> >> It's not "abusing" it but extending the interface in a >> backwards-compatible manner.

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-19 Thread Mario Kleiner
On Fri, 18 Mar 2011, Jesse Barnes wrote: I like the new param check, but I'd still prefer a new ioctl to abusing the old one. It's not abusing it but extending the interface in a backwards-compatible manner. Introducing a new one would result in two ioctls that essentially do the same

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-19 Thread Alex Deucher
On Fri, Mar 18, 2011 at 5:58 PM, Ilija Hadzic ihad...@research.bell-labs.com wrote: Hi Dave, Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the capability to wait on vblank events for CRTCs that are greater than 1 and thus cannot be represented with

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-19 Thread Alex Deucher
On Fri, Mar 18, 2011 at 5:58 PM, Ilija Hadzic ihad...@research.bell-labs.com wrote: Hi Dave, Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the capability to wait on vblank events for CRTCs that are greater than 1 and thus cannot be represented with

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-18 Thread Ilija Hadzic
On Fri, 18 Mar 2011, Jesse Barnes wrote: > > I like the new param check, but I'd still prefer a new ioctl to abusing > the old one. > It's not "abusing" it but extending the interface in a backwards-compatible manner. Introducing a new one would result in two ioctls that essentially do the

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-18 Thread Ilija Hadzic
Hi Dave, Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the capability to wait on vblank events for CRTCs that are greater than 1 and thus cannot be represented with primary/secondary flags in the legacy interface. It was discussed on the dri-devel list in these two

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-18 Thread Jesse Barnes
On Fri, 18 Mar 2011 18:13:11 -0500 (CDT) Ilija Hadzic wrote: > > > On Fri, 18 Mar 2011, Jesse Barnes wrote: > > > > > I like the new param check, but I'd still prefer a new ioctl to abusing > > the old one. > > > > It's not "abusing" it but extending the interface in a >

[PATCH] kernel/drm: vblank wait on crtc > 1

2011-03-18 Thread Jesse Barnes
On Fri, 18 Mar 2011 16:58:04 -0500 (CDT) Ilija Hadzic wrote: > > Hi Dave, > > Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds > the capability to wait on vblank events for CRTCs that are greater than 1 > and thus cannot be represented with primary/secondary flags in

[PATCH] kernel/drm: vblank wait on crtc 1

2011-03-18 Thread Ilija Hadzic
Hi Dave, Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the capability to wait on vblank events for CRTCs that are greater than 1 and thus cannot be represented with primary/secondary flags in the legacy interface. It was discussed on the dri-devel list in these two

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-18 Thread Jesse Barnes
On Fri, 18 Mar 2011 16:58:04 -0500 (CDT) Ilija Hadzic ihad...@research.bell-labs.com wrote: Hi Dave, Below is a patch against drm-next branch of 2.6.38-rc8+ kernel that adds the capability to wait on vblank events for CRTCs that are greater than 1 and thus cannot be represented with

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-18 Thread Ilija Hadzic
On Fri, 18 Mar 2011, Jesse Barnes wrote: I like the new param check, but I'd still prefer a new ioctl to abusing the old one. It's not abusing it but extending the interface in a backwards-compatible manner. Introducing a new one would result in two ioctls that essentially do the same

Re: [PATCH] kernel/drm: vblank wait on crtc 1

2011-03-18 Thread Jesse Barnes
On Fri, 18 Mar 2011 18:13:11 -0500 (CDT) Ilija Hadzic ihad...@research.bell-labs.com wrote: On Fri, 18 Mar 2011, Jesse Barnes wrote: I like the new param check, but I'd still prefer a new ioctl to abusing the old one. It's not abusing it but extending the interface in a