[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-15 Thread Mario Kleiner
On 02/11/2013 10:13 AM, Thierry Reding wrote: > On Tue, Jan 22, 2013 at 06:37:39PM +0100, Mario Kleiner wrote: >> On 14.01.13 17:05, Thierry Reding wrote: >>> Implement support for the VBLANK IOCTL. Note that Tegra is somewhat >>> special in this case because it doesn't use the generic IRQ support

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-15 Thread Mario Kleiner
On 02/11/2013 10:13 AM, Thierry Reding wrote: On Tue, Jan 22, 2013 at 06:37:39PM +0100, Mario Kleiner wrote: On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra is somewhat special in this case because it doesn't use the generic IRQ support provided

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-11 Thread Thierry Reding
On Tue, Jan 22, 2013 at 06:37:39PM +0100, Mario Kleiner wrote: > On 14.01.13 17:05, Thierry Reding wrote: > >Implement support for the VBLANK IOCTL. Note that Tegra is somewhat > >special in this case because it doesn't use the generic IRQ support > >provided by the DRM core (DRIVER_HAVE_IRQ) but r

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-11 Thread Thierry Reding
On Wed, Jan 23, 2013 at 10:02:20AM +0200, Terje Bergstr?m wrote: > On 22.01.2013 21:59, Mario Kleiner wrote: > > The current swap scheduling is based on having an accurate software > > vblank counter. Atm. that counter is maintained in software, incremented > > during vblank irq. At irq off -> on

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-11 Thread Terje Bergström
On 11.02.2013 01:08, Thierry Reding wrote: > Are the syncpoints incremented even if the VBLANK interrupts are turned > off? If that's the case they could indeed be used as a hardware counter > rather than the fake approach used right now. > > Maybe we should leave the code as it is right now and f

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-11 Thread Terje Bergström
On 11.02.2013 01:08, Thierry Reding wrote: > Are the syncpoints incremented even if the VBLANK interrupts are turned > off? If that's the case they could indeed be used as a hardware counter > rather than the fake approach used right now. > > Maybe we should leave the code as it is right now and f

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-11 Thread Thierry Reding
On Tue, Jan 22, 2013 at 06:37:39PM +0100, Mario Kleiner wrote: > On 14.01.13 17:05, Thierry Reding wrote: > >Implement support for the VBLANK IOCTL. Note that Tegra is somewhat > >special in this case because it doesn't use the generic IRQ support > >provided by the DRM core (DRIVER_HAVE_IRQ) but r

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-02-11 Thread Thierry Reding
On Wed, Jan 23, 2013 at 10:02:20AM +0200, Terje Bergström wrote: > On 22.01.2013 21:59, Mario Kleiner wrote: > > The current swap scheduling is based on having an accurate software > > vblank counter. Atm. that counter is maintained in software, incremented > > during vblank irq. At irq off -> on

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-23 Thread Terje Bergström
On 22.01.2013 21:59, Mario Kleiner wrote: > The current swap scheduling is based on having an accurate software > vblank counter. Atm. that counter is maintained in software, incremented > during vblank irq. At irq off -> on transition we need a hw counter to > reinitialize. And there is a timeo

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Terje Bergström
On 22.01.2013 21:59, Mario Kleiner wrote: > The current swap scheduling is based on having an accurate software > vblank counter. Atm. that counter is maintained in software, incremented > during vblank irq. At irq off -> on transition we need a hw counter to > reinitialize. And there is a timeo

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 22.01.13 19:49, Jon Mayo wrote: On Tue, Jan 22, 2013 at 10:39 AM, Lucas Stach wrote: Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra is somewhat special in this case because i

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Jon Mayo
On Tue, Jan 22, 2013 at 11:20 AM, Mario Kleiner wrote: > On 22.01.13 19:39, Lucas Stach wrote: >> >> Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: >>> >>> On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra is somewhat s

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Jon Mayo
On Tue, Jan 22, 2013 at 10:39 AM, Lucas Stach wrote: > Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: >> On 14.01.13 17:05, Thierry Reding wrote: >> > Implement support for the VBLANK IOCTL. Note that Tegra is somewhat >> > special in this case because it doesn't use the generic I

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra is somewhat special in this case because it doesn't use the generic IRQ support provided by the DRM core (DRIVER_HAVE_IRQ) but rather registers one interrupt handler for each display controller. Whil

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 22.01.13 20:27, Jon Mayo wrote: > On Tue, Jan 22, 2013 at 11:20 AM, Mario Kleiner > wrote: >> On 22.01.13 19:39, Lucas Stach wrote: >>> >>> Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: On 14.01.13 17:05, Thierry Reding wrote: > > Implement support for the VB

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 22.01.13 19:49, Jon Mayo wrote: > On Tue, Jan 22, 2013 at 10:39 AM, Lucas Stach wrote: >> Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: >>> On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra is somewhat special in this

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 22.01.13 19:39, Lucas Stach wrote: > Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: >> On 14.01.13 17:05, Thierry Reding wrote: >>> Implement support for the VBLANK IOCTL. Note that Tegra is somewhat >>> special in this case because it doesn't use the generic IRQ support >>> pro

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Lucas Stach
Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: > On 14.01.13 17:05, Thierry Reding wrote: > > Implement support for the VBLANK IOCTL. Note that Tegra is somewhat > > special in this case because it doesn't use the generic IRQ support > > provided by the DRM core (DRIVER_HAVE_IRQ) b

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 14.01.13 17:05, Thierry Reding wrote: > Implement support for the VBLANK IOCTL. Note that Tegra is somewhat > special in this case because it doesn't use the generic IRQ support > provided by the DRM core (DRIVER_HAVE_IRQ) but rather registers one > interrupt handler for each display controller.

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 22.01.13 20:27, Jon Mayo wrote: On Tue, Jan 22, 2013 at 11:20 AM, Mario Kleiner wrote: On 22.01.13 19:39, Lucas Stach wrote: Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Jon Mayo
On Tue, Jan 22, 2013 at 11:20 AM, Mario Kleiner wrote: > On 22.01.13 19:39, Lucas Stach wrote: >> >> Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: >>> >>> On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra is somewhat s

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Mario Kleiner
On 22.01.13 19:39, Lucas Stach wrote: Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: On 14.01.13 17:05, Thierry Reding wrote: Implement support for the VBLANK IOCTL. Note that Tegra is somewhat special in this case because it doesn't use the generic IRQ support provided by the

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Jon Mayo
On Tue, Jan 22, 2013 at 10:39 AM, Lucas Stach wrote: > Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: >> On 14.01.13 17:05, Thierry Reding wrote: >> > Implement support for the VBLANK IOCTL. Note that Tegra is somewhat >> > special in this case because it doesn't use the generic I

Re: [PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-22 Thread Lucas Stach
Am Dienstag, den 22.01.2013, 18:37 +0100 schrieb Mario Kleiner: > On 14.01.13 17:05, Thierry Reding wrote: > > Implement support for the VBLANK IOCTL. Note that Tegra is somewhat > > special in this case because it doesn't use the generic IRQ support > > provided by the DRM core (DRIVER_HAVE_IRQ) b

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-14 Thread Thierry Reding
Implement support for the VBLANK IOCTL. Note that Tegra is somewhat special in this case because it doesn't use the generic IRQ support provided by the DRM core (DRIVER_HAVE_IRQ) but rather registers one interrupt handler for each display controller. While at it, clean up the way that interrupts a

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-14 Thread Thierry Reding
Implement support for the VBLANK IOCTL. Note that Tegra is somewhat special in this case because it doesn't use the generic IRQ support provided by the DRM core (DRIVER_HAVE_IRQ) but rather registers one interrupt handler for each display controller. While at it, clean up the way that interrupts a