Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-21 Thread Daniel Vetter
On Sat, Aug 19, 2017 at 04:46:30PM +0200, Noralf Trønnes wrote: > > Den 16.08.2017 22.50, skrev Laurent Pinchart: > > Hi Noralf, > > > > One additional comment. > > > > On Wednesday 16 Aug 2017 23:37:54 Laurent Pinchart wrote: > > > On Sunday 13 Aug 2017 15:31:44 Noralf Trønnes wrote: > > > >

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-19 Thread Noralf Trønnes
Den 16.08.2017 22.50, skrev Laurent Pinchart: Hi Noralf, One additional comment. On Wednesday 16 Aug 2017 23:37:54 Laurent Pinchart wrote: On Sunday 13 Aug 2017 15:31:44 Noralf Trønnes wrote: This library provides helpers for drivers that don't subclass drm_framebuffer and are backed by

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Eric Anholt
Noralf Trønnes writes: > Den 16.08.2017 22.39, skrev Laurent Pinchart: >> Hi Noralf, >> >> On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: >>> Den 16.08.2017 19.24, skrev Eric Anholt: Noralf Trønnes writes: > This library provides helpers

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Laurent Pinchart
Hi Noralf, On Wednesday 16 Aug 2017 23:31:23 Noralf Trønnes wrote: > Den 16.08.2017 23.13, skrev Daniel Vetter: > > On Wed, Aug 16, 2017 at 11:11 PM, Laurent Pinchart wrote: > >> On Wednesday 16 Aug 2017 23:06:30 Daniel Vetter wrote: > >>> Perhaps pinged Laurent on irc since he's written/reviewed

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Laurent Pinchart
Hi Noralf, On Wednesday 16 Aug 2017 23:24:50 Noralf Trønnes wrote: > Den 16.08.2017 23.08, skrev Laurent Pinchart: > > On Wednesday 16 Aug 2017 23:03:36 Noralf Trønnes wrote: > >> Den 16.08.2017 22.39, skrev Laurent Pinchart: > >>> On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: >

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Noralf Trønnes
Den 16.08.2017 23.13, skrev Daniel Vetter: On Wed, Aug 16, 2017 at 11:11 PM, Laurent Pinchart wrote: On Wednesday 16 Aug 2017 23:06:30 Daniel Vetter wrote: Perhaps pinged Laurent on irc since he's written/reviewed a bunch of cma patches. Aside: Being on irc

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Noralf Trønnes
Den 16.08.2017 23.08, skrev Laurent Pinchart: Hi Noralf, On Wednesday 16 Aug 2017 23:03:36 Noralf Trønnes wrote: Den 16.08.2017 22.39, skrev Laurent Pinchart: On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: Den 16.08.2017 19.24, skrev Eric Anholt: Noralf Trønnes

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Daniel Vetter
On Wed, Aug 16, 2017 at 11:11 PM, Laurent Pinchart wrote: > On Wednesday 16 Aug 2017 23:06:30 Daniel Vetter wrote: >> Perhaps pinged Laurent on irc since he's written/reviewed a bunch of cma >> patches. Aside: Being on irc would be good, makes it quicker to >>

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Laurent Pinchart
Hi Daniel, On Wednesday 16 Aug 2017 23:06:30 Daniel Vetter wrote: > On Wed, Aug 16, 2017 at 11:03 PM, Noralf Trønnes wrote: > > Den 16.08.2017 22.39, skrev Laurent Pinchart: > >> On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: > >>> Den 16.08.2017 19.24, skrev Eric

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Laurent Pinchart
Hi Noralf, On Wednesday 16 Aug 2017 23:03:36 Noralf Trønnes wrote: > Den 16.08.2017 22.39, skrev Laurent Pinchart: > > On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: > >> Den 16.08.2017 19.24, skrev Eric Anholt: > >>> Noralf Trønnes writes: > This library

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Daniel Vetter
On Wed, Aug 16, 2017 at 11:03 PM, Noralf Trønnes wrote: > > Den 16.08.2017 22.39, skrev Laurent Pinchart: >> >> Hi Noralf, >> >> On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: >>> >>> Den 16.08.2017 19.24, skrev Eric Anholt: Noralf Trønnes

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Noralf Trønnes
Den 16.08.2017 22.39, skrev Laurent Pinchart: Hi Noralf, On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: Den 16.08.2017 19.24, skrev Eric Anholt: Noralf Trønnes writes: This library provides helpers for drivers that don't subclass drm_framebuffer and are backed

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Laurent Pinchart
Hi Noralf, One additional comment. On Wednesday 16 Aug 2017 23:37:54 Laurent Pinchart wrote: > On Sunday 13 Aug 2017 15:31:44 Noralf Trønnes wrote: > > This library provides helpers for drivers that don't subclass > > drm_framebuffer and are backed by drm_gem_object. The code is > > taken from

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Laurent Pinchart
Hi Noralf, On Wednesday 16 Aug 2017 21:52:02 Noralf Trønnes wrote: > Den 16.08.2017 19.24, skrev Eric Anholt: > > Noralf Trønnes writes: > >> This library provides helpers for drivers that don't subclass > >> drm_framebuffer and are backed by drm_gem_object. The code is > >>

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Laurent Pinchart
Ni Noralf, Thank you for the patch. On Sunday 13 Aug 2017 15:31:44 Noralf Trønnes wrote: > This library provides helpers for drivers that don't subclass > drm_framebuffer and are backed by drm_gem_object. The code is > taken from drm_fb_cma_helper. > > Signed-off-by: Noralf Trønnes

Re: [PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-16 Thread Eric Anholt
Noralf Trønnes writes: > This library provides helpers for drivers that don't subclass > drm_framebuffer and are backed by drm_gem_object. The code is > taken from drm_fb_cma_helper. > > Signed-off-by: Noralf Trønnes > Reviewed-by: Daniel Vetter

[PATCH v3 01/22] drm: Add GEM backed framebuffer library

2017-08-13 Thread Noralf Trønnes
This library provides helpers for drivers that don't subclass drm_framebuffer and are backed by drm_gem_object. The code is taken from drm_fb_cma_helper. Signed-off-by: Noralf Trønnes Reviewed-by: Daniel Vetter ---