Le dim. 28 mars 2021 à 1:05, Laurent Pinchart 
[off-list ref] a écrit :

Hi Paul,

Thank you for the patch.

On Sat, Mar 27, 2021 at 11:57:41AM +0000, Paul Cercueil wrote:

quoted

 This performs the same operation as drmm_encoder_alloc(), but
 only allocates and returns a struct drm_encoder instance.

 v4: Rename macro drmm_plain_encoder_alloc() and move to
     <drm/drm_encoder.h>. Since it's not "simple" anymore it
     will now take funcs/name arguments as well.

 Signed-off-by: Paul Cercueil [off-list ref]

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Patchset applied to drm-misc-next.

Thanks!

-Paul

quoted

 ---
  include/drm/drm_encoder.h | 18 ++++++++++++++++++
  1 file changed, 18 insertions(+)

 diff --git a/include/drm/drm_encoder.h b/include/drm/drm_encoder.h
 index 5bf78b5bcb2b..6e91a0280f31 100644
 --- a/include/drm/drm_encoder.h
 +++ b/include/drm/drm_encoder.h
 @@ -224,6 +224,24 @@ void *__drmm_encoder_alloc(struct drm_device 
*dev,
  				      offsetof(type, member), funcs, \
  				      encoder_type, name, ##__VA_ARGS__))

 +/**
 + * drmm_plain_encoder_alloc - Allocate and initialize an encoder
 + * @dev: drm device
 + * @funcs: callbacks for this encoder (optional)
 + * @encoder_type: user visible type of the encoder
 + * @name: printf style format string for the encoder name, or NULL 
for default name
 + *
 + * This is a simplified version of drmm_encoder_alloc(), which 
only allocates
 + * and returns a struct drm_encoder instance, with no subclassing.
 + *
 + * Returns:
 + * Pointer to the new drm_encoder struct, or ERR_PTR on failure.
 + */
 +#define drmm_plain_encoder_alloc(dev, funcs, encoder_type, name, 
...) \
 +	((struct drm_encoder *) \
 +	 __drmm_encoder_alloc(dev, sizeof(struct drm_encoder), \
 +			      0, funcs, encoder_type, name, ##__VA_ARGS__))
 +
  /**
   * drm_encoder_index - find the index of a registered encoder
   * @encoder: encoder to find index for

--
Regards,

Laurent Pinchart