[PATCH 00/22] drm: Remove includes for drm_crtc_helper.h
Jani Nikula
jani.nikula at linux.intel.com
Mon Jan 16 08:19:56 PST 2023
On Mon, 16 Jan 2023, Thomas Zimmermann <tzimmermann at suse.de> wrote:
> A lot of source files include drm_crtc_helper.h for its contained
> include statements. This leads to excessive compile-time dependencies.
>
> Where possible, remove the include statements for drm_crtc_helper.h
> and include the required source files directly. Also remove the
> include statements from drm_crtc_helper.h itself, which doesn't need
> most of them.
>
> I built this patchset on x86-64, aarch64 and arm. Hopefully I found
> all include dependencies.
I think this is the right direction and I support this. Personally I
think it's enough to build test and fix any fallout afterwards.
To that end, I did build test this myself with my config, and it was
fine, though that probably doesn't add much coverage.
FWIW,
Acked-by: Jani Nikula <jani.nikula at intel.com>
--
Jani Nikula, Intel Open Source Graphics Center
More information about the Linux-rockchip
mailing list