Re: [PATCH v3 1/7] drm: add drmm_encoder_alloc()

2020-12-08 Thread Daniel Vetter
On Sat, Dec 05, 2020 at 08:57:38PM +0200, Laurent Pinchart wrote: > Hi Philipp, > > On Fri, Dec 04, 2020 at 11:12:20AM +0100, Philipp Zabel wrote: > > On Fri, 2020-12-04 at 11:17 +0200, Laurent Pinchart wrote: > > > On Fri, Sep 11, 2020 at 03:57:18PM +0200, Philipp Zabel wrote: > > > > Add an alte

Re: [PATCH v3 1/7] drm: add drmm_encoder_alloc()

2020-12-05 Thread Laurent Pinchart
Hi Philipp, On Fri, Dec 04, 2020 at 11:12:20AM +0100, Philipp Zabel wrote: > On Fri, 2020-12-04 at 11:17 +0200, Laurent Pinchart wrote: > > On Fri, Sep 11, 2020 at 03:57:18PM +0200, Philipp Zabel wrote: > > > Add an alternative to drm_encoder_init() that allocates and initializes > > > an encoder

Re: [PATCH v3 1/7] drm: add drmm_encoder_alloc()

2020-12-04 Thread Philipp Zabel
Hi Laurent, On Fri, 2020-12-04 at 11:17 +0200, Laurent Pinchart wrote: > Hi Philipp, > > Thank you for the patch. Thank you for the review. > On Fri, Sep 11, 2020 at 03:57:18PM +0200, Philipp Zabel wrote: > > Add an alternative to drm_encoder_init() that allocates and initializes > > an encoder

Re: [PATCH v3 1/7] drm: add drmm_encoder_alloc()

2020-12-04 Thread Laurent Pinchart
Hi Philipp, Thank you for the patch. On Fri, Sep 11, 2020 at 03:57:18PM +0200, Philipp Zabel wrote: > Add an alternative to drm_encoder_init() that allocates and initializes > an encoder and registers drm_encoder_cleanup() with > drmm_add_action_or_reset(). > > Signed-off-by: Philipp Zabel > --

[PATCH v3 1/7] drm: add drmm_encoder_alloc()

2020-09-11 Thread Philipp Zabel
Add an alternative to drm_encoder_init() that allocates and initializes an encoder and registers drm_encoder_cleanup() with drmm_add_action_or_reset(). Signed-off-by: Philipp Zabel --- Changes since v2: - call va_start() / va_end() unconditionally --- drivers/gpu/drm/drm_encoder.c | 101 +++