[PATCH] drm: Link drm_rect.o into DRM core module
Ville Syrjälä
ville.syrjala at linux.intel.com
Fri Sep 20 10:12:11 UTC 2024
On Thu, Sep 19, 2024 at 08:53:43PM +0200, Thomas Zimmermann wrote:
> Several places in drm.ko use struct drm_rect and its helpers. This
> only works as the called interfaces are declared as static inline in
> the header file. Fix the issue by linking drm_rect.o into drm.ko.
Seems reasonable.
Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
PS when grepping for drm_rect I noticed drm_atomic_plane_set_property()
is using it as the format of the fb_damage_clips blob. That
can't be right as drm_rect is an internal thing and should
not be used in the uapi.
>
> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
> Suggested-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
> drivers/gpu/drm/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile
> index 784229d4504d..fc061fd424ed 100644
> --- a/drivers/gpu/drm/Makefile
> +++ b/drivers/gpu/drm/Makefile
> @@ -68,6 +68,7 @@ drm-y := \
> drm_prime.o \
> drm_print.o \
> drm_property.o \
> + drm_rect.o \
> drm_syncobj.o \
> drm_sysfs.o \
> drm_trace_points.o \
> @@ -140,7 +141,6 @@ drm_kms_helper-y := \
> drm_modeset_helper.o \
> drm_plane_helper.o \
> drm_probe_helper.o \
> - drm_rect.o \
> drm_self_refresh_helper.o \
> drm_simple_kms_helper.o
> drm_kms_helper-$(CONFIG_DRM_PANEL_BRIDGE) += bridge/panel.o
> --
> 2.46.0
--
Ville Syrjälä
Intel
More information about the dri-devel
mailing list