On Tue, 08 Dec 2015, ville.syrjala@xxxxxxxxxxxxxxx wrote:
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> Done with coccinelle for the most part. However, it thinks '...' is
> part of the semantic patch, so I put an 'int DOTDOTDOT' placeholder
> in its place and got rid of it with sed afterwards.
>
> I didn't convert drm_crtc_init() since passing the varargs through
> would mean either cpp macros or va_list, and I figured we don't
> care about these legacy functions enough to warrant the extra pain.
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> index 9f6e234e7029..468a14f266a7 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> @@ -344,7 +344,7 @@ int atmel_hlcdc_crtc_create(struct drm_device *dev)
> ret = drm_crtc_init_with_planes(dev, &crtc->base,
> &planes->primary->base,
> planes->cursor ? &planes->cursor->base : NULL,
> - &atmel_hlcdc_crtc_funcs);
> + &atmel_hlcdc_crtc_funcs, NULL);
> if (ret < 0)
> goto fail;
>
> diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> index 99e7efebedda..62306d336d0d 100644
> --- a/drivers/gpu/drm/drm_crtc.c
> +++ b/drivers/gpu/drm/drm_crtc.c
> @@ -657,6 +657,7 @@ DEFINE_WW_CLASS(crtc_ww_class);
> * @primary: Primary plane for CRTC
> * @cursor: Cursor plane for CRTC
> * @funcs: callbacks for the new CRTC
> + * @name: printf style format string for the CRTC name
Maybe add ", or NULL"? (Does kernel-doc support documenting varargs
somehow?)
Please add the appropriate __printf(x, y) annotation for the compiler
above the declaration in drm_crtc.h.
The same for patches 1-3 in the series.
BR,
Jani.
> *
> * Inits a new object created as base part of a driver crtc object.
> *
> @@ -666,7 +667,8 @@ DEFINE_WW_CLASS(crtc_ww_class);
> int drm_crtc_init_with_planes(struct drm_device *dev, struct drm_crtc *crtc,
> struct drm_plane *primary,
> struct drm_plane *cursor,
> - const struct drm_crtc_funcs *funcs)
> + const struct drm_crtc_funcs *funcs,
> + const char *name, ...)
> {
> struct drm_mode_config *config = &dev->mode_config;
> int ret;
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
