[PATCH 4/5] drm/print: Include drm_device.h
Jani Nikula
jani.nikula at intel.com
Thu Jan 23 15:14:31 UTC 2025
On Thu, 23 Jan 2025, Jani Nikula <jani.nikula at intel.com> wrote:
> From: Gustavo Sousa <gustavo.sousa at intel.com>
>
> The header drm_print.h uses members of struct drm_device pointers, as
> such, it should include drm_device.h to let the compiler know the full
> type definition.
>
> Without such include, users of drm_print.h that don't explicitly need
> drm_device.h would bump into build errors and be forced to include the
> latter.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa at intel.com>
> Reviewed-by: Jani Nikula <jani.nikula at intel.com>
This posting should have had
Signed-off-by: Jani Nikula <jani.nikula at intel.com>
but obviously not needed if the original [1] is merged instead.
[1] https://lore.kernel.org/r/20250121210935.84357-1-gustavo.sousa@intel.com
>
> ---
>
> Including here as a dependency. May be merged independently.
> ---
> include/drm/drm_print.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
> index f77fe1531cf8..9732f514566d 100644
> --- a/include/drm/drm_print.h
> +++ b/include/drm/drm_print.h
> @@ -32,6 +32,7 @@
> #include <linux/dynamic_debug.h>
>
> #include <drm/drm.h>
> +#include <drm/drm_device.h>
>
> struct debugfs_regset32;
> struct drm_device;
--
Jani Nikula, Intel
More information about the dri-devel
mailing list