[PATCH 84/86] drm/drm_atomic_helper: Minimize include footprint

[Thomas Zimmermann]

Hi,

in the subject line, I'd use 'dependencies' instead of 'footprint'. Your 
choice.

Am 21.01.23 um 21:09 schrieb Sam Ravnborg via B4 Submission Endpoint:
> From: Sam Ravnborg <sam at ravnborg.org>
> 
> Reduce drm_atomic_helper.h to include or forward the minimal set of
> files/symbols - to minimize the impact of the files that includes this
> header file.
> 
> Signed-off-by: Sam Ravnborg <sam at ravnborg.org>
> ---
>   include/drm/drm_atomic_helper.h | 21 ++++++++++++++-------
>   1 file changed, 14 insertions(+), 7 deletions(-)
> 
> diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
> index 33f982cd1a27..c7f2bc46bd24 100644
> --- a/include/drm/drm_atomic_helper.h
> +++ b/include/drm/drm_atomic_helper.h
> @@ -28,11 +28,9 @@
>   #ifndef DRM_ATOMIC_HELPER_H_
>   #define DRM_ATOMIC_HELPER_H_
>   
> -#include <drm/drm_crtc.h>
> -#include <drm/drm_modeset_helper_vtables.h>
> -#include <drm/drm_modeset_helper.h>
> -#include <drm/drm_atomic_state_helper.h>
> -#include <drm/drm_util.h>
> +#include <linux/types.h>
> +
> +#include <drm/drm_plane.h>
>   
>   /*
>    * Drivers that don't allow primary plane scaling may pass this macro in place
> @@ -44,8 +42,17 @@
>   #define DRM_PLANE_NO_SCALING (1<<16)
>   
>   struct drm_atomic_state;
> -struct drm_private_obj;
> -struct drm_private_state;
> +struct drm_bridge;
> +struct drm_bridge_state;
> +struct drm_connector_state;
> +struct drm_crtc_state;
> +struct drm_device;
> +struct drm_encoder;
> +struct drm_framebuffer;
> +struct drm_mode_set;
> +struct drm_modeset_acquire_ctx;
> +struct drm_pending_vblank_event;
> +

No extra newline here.

>   
>   int drm_atomic_helper_check_modeset(struct drm_device *dev,
>   				struct drm_atomic_state *state);
> 

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20230123/f210d5a8/attachment.sig>