Re: [PATCH 3/4] drm/imx: add FB modifier support

2017-05-04 Thread Lucas Stach
Am Donnerstag, den 04.05.2017, 11:17 +0200 schrieb Daniel Vetter: > On Thu, May 4, 2017 at 11:12 AM, Lucas Stach wrote: > > Am Donnerstag, den 04.05.2017, 10:02 +0100 schrieb Daniel Stone: > >> On 4 May 2017 at 09:59, Lucas Stach wrote: > >> > Am Mittwoch, den 03.05.2017, 18:15 +0100 schrieb Dani

Re: [PATCH 3/4] drm/imx: add FB modifier support

2017-05-04 Thread Daniel Vetter
On Thu, May 4, 2017 at 11:12 AM, Lucas Stach wrote: > Am Donnerstag, den 04.05.2017, 10:02 +0100 schrieb Daniel Stone: >> On 4 May 2017 at 09:59, Lucas Stach wrote: >> > Am Mittwoch, den 03.05.2017, 18:15 +0100 schrieb Daniel Stone: >> >> What about planes which aren't present in this commit, but

Re: [PATCH 3/4] drm/imx: add FB modifier support

2017-05-04 Thread Lucas Stach
Am Donnerstag, den 04.05.2017, 10:02 +0100 schrieb Daniel Stone: > Hi, > > On 4 May 2017 at 09:59, Lucas Stach wrote: > > Am Mittwoch, den 03.05.2017, 18:15 +0100 schrieb Daniel Stone: > >> What about planes which aren't present in this commit, but are still > >> taking up a PRE unit? Will they h

Re: [PATCH 3/4] drm/imx: add FB modifier support

2017-05-04 Thread Daniel Stone
Hi, On 4 May 2017 at 09:59, Lucas Stach wrote: > Am Mittwoch, den 03.05.2017, 18:15 +0100 schrieb Daniel Stone: >> What about planes which aren't present in this commit, but are still >> taking up a PRE unit? Will they have their PRE stolen, or am I missing >> something? > > Yes, the plane->PRE a

Re: [PATCH 3/4] drm/imx: add FB modifier support

2017-05-04 Thread Lucas Stach
Am Mittwoch, den 03.05.2017, 18:15 +0100 schrieb Daniel Stone: > Hi Lucas, > > On 3 May 2017 at 17:28, Lucas Stach wrote: > > int available_pres = ipu_prg_max_active_channels(); > > int i; > > > > + /* > > +* We are going over the planes in 2 passes: first we assign

Re: [PATCH 3/4] drm/imx: add FB modifier support

2017-05-03 Thread Daniel Stone
Hi Lucas, On 3 May 2017 at 17:28, Lucas Stach wrote: > int available_pres = ipu_prg_max_active_channels(); > int i; > > + /* > +* We are going over the planes in 2 passes: first we assign PREs to > +* planes with a tiling modifier, which need the PREs to reso

[PATCH 3/4] drm/imx: add FB modifier support

2017-05-03 Thread Lucas Stach
This adds FB modifier support for the Vivante single buffer tiled formats, when the PRG/PRE engines are present. Signed-off-by: Lucas Stach --- drivers/gpu/drm/imx/imx-drm-core.c | 1 + drivers/gpu/drm/imx/ipuv3-plane.c | 55 ++ 2 files changed, 51 insertion