Thomas Zimmermann <tzimmerm...@suse.de> writes:

> Merge the connector functions of ofdrm and simpledrm. Replace the
> code in each driver with the shared helpers. Set up callbacks with
> initializer macros.
>
> No effective code changes. The sysfb connector only returns the
> preconfigured display mode.
>
> Signed-off-by: Thomas Zimmermann <tzimmerm...@suse.de>
> ---

...

> +#define DRM_SYSFB_CONNECTOR_FUNCS \
> +     .reset = drm_atomic_helper_connector_reset, \
> +     .fill_modes = drm_helper_probe_single_connector_modes, \
> +     .atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state, \
> +     .atomic_destroy_state = drm_atomic_helper_connector_destroy_state
> +
>  /*

...

>  static const struct drm_connector_funcs ofdrm_connector_funcs = {
> -     .reset = drm_atomic_helper_connector_reset,
> -     .fill_modes = drm_helper_probe_single_connector_modes,
> +     DRM_SYSFB_CONNECTOR_FUNCS,
>       .destroy = drm_connector_cleanup,

Why not include the .destroy callback in DRM_SYSFB_CONNECTOR_FUNCS ?

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

Reply via email to