drm: split build lists one per line and sort
authorJani Nikula <jani.nikula@intel.com>
Tue, 11 Oct 2022 14:08:30 +0000 (17:08 +0300)
committerJani Nikula <jani.nikula@intel.com>
Wed, 12 Oct 2022 14:00:05 +0000 (17:00 +0300)
While it takes more vertical space, sorted build lists with one object
per line are arguably easier to manage, especially when there are
conflicting changes.

Split anything with more than one object file.

v2: also split drm_cache.o and put it after drm_bridge.o (Andi)

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Acked-by: Maxime Ripard <maxime@cerno.tech>
Link: https://patchwork.freedesktop.org/patch/msgid/20221011140830.3257655-1-jani.nikula@intel.com
drivers/gpu/drm/Makefile
drivers/gpu/drm/display/Makefile

index 25d0ba31050938c0db4b76369ac894a59b40936c..6ad98d3ceff76d6fa259ed5fbe1bedc11cb22648 100644 (file)
@@ -3,32 +3,71 @@
 # Makefile for the drm device driver.  This driver provides support for the
 # Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher.
 
-drm-y       := drm_aperture.o drm_auth.o drm_cache.o \
-               drm_file.o drm_gem.o drm_ioctl.o \
-               drm_drv.o \
-               drm_sysfs.o drm_mm.o \
-               drm_crtc.o drm_fourcc.o drm_modes.o drm_edid.o drm_displayid.o \
-               drm_trace_points.o drm_prime.o \
-               drm_vma_manager.o \
-               drm_modeset_lock.o drm_atomic.o drm_bridge.o \
-               drm_framebuffer.o drm_connector.o drm_blend.o \
-               drm_encoder.o drm_mode_object.o drm_property.o \
-               drm_plane.o drm_color_mgmt.o drm_print.o \
-               drm_dumb_buffers.o drm_mode_config.o drm_vblank.o \
-               drm_syncobj.o drm_lease.o drm_writeback.o drm_client.o \
-               drm_client_modeset.o drm_atomic_uapi.o \
-               drm_managed.o drm_vblank_work.o
-drm-$(CONFIG_DRM_LEGACY) += drm_agpsupport.o drm_bufs.o drm_context.o drm_dma.o \
-                           drm_hashtab.o drm_irq.o drm_legacy_misc.o drm_lock.o \
-                           drm_memory.o drm_scatter.o drm_vm.o
+drm-y := \
+       drm_aperture.o \
+       drm_atomic.o \
+       drm_atomic_uapi.o \
+       drm_auth.o \
+       drm_blend.o \
+       drm_bridge.o \
+       drm_cache.o \
+       drm_client.o \
+       drm_client_modeset.o \
+       drm_color_mgmt.o \
+       drm_connector.o \
+       drm_crtc.o \
+       drm_displayid.o \
+       drm_drv.o \
+       drm_dumb_buffers.o \
+       drm_edid.o \
+       drm_encoder.o \
+       drm_file.o \
+       drm_fourcc.o \
+       drm_framebuffer.o \
+       drm_gem.o \
+       drm_ioctl.o \
+       drm_lease.o \
+       drm_managed.o \
+       drm_mm.o \
+       drm_mode_config.o \
+       drm_mode_object.o \
+       drm_modes.o \
+       drm_modeset_lock.o \
+       drm_plane.o \
+       drm_prime.o \
+       drm_print.o \
+       drm_property.o \
+       drm_syncobj.o \
+       drm_sysfs.o \
+       drm_trace_points.o \
+       drm_vblank.o \
+       drm_vblank_work.o \
+       drm_vma_manager.o \
+       drm_writeback.o
+drm-$(CONFIG_DRM_LEGACY) += \
+       drm_agpsupport.o \
+       drm_bufs.o \
+       drm_context.o \
+       drm_dma.o \
+       drm_hashtab.o \
+       drm_irq.o \
+       drm_legacy_misc.o \
+       drm_lock.o \
+       drm_memory.o \
+       drm_scatter.o \
+       drm_vm.o
 drm-$(CONFIG_DRM_LIB_RANDOM) += lib/drm_random.o
 drm-$(CONFIG_COMPAT) += drm_ioc32.o
 drm-$(CONFIG_DRM_PANEL) += drm_panel.o
 drm-$(CONFIG_OF) += drm_of.o
 drm-$(CONFIG_PCI) += drm_pci.o
-drm-$(CONFIG_DEBUG_FS) += drm_debugfs.o drm_debugfs_crc.o
+drm-$(CONFIG_DEBUG_FS) += \
+       drm_debugfs.o \
+       drm_debugfs_crc.o
 drm-$(CONFIG_DRM_LOAD_EDID_FIRMWARE) += drm_edid_load.o
-drm-$(CONFIG_DRM_PRIVACY_SCREEN) += drm_privacy_screen.o drm_privacy_screen_x86.o
+drm-$(CONFIG_DRM_PRIVACY_SCREEN) += \
+       drm_privacy_screen.o \
+       drm_privacy_screen_x86.o
 obj-$(CONFIG_DRM)      += drm.o
 
 obj-$(CONFIG_DRM_NOMODESET) += drm_nomodeset.o
@@ -57,16 +96,24 @@ obj-$(CONFIG_DRM_TTM_HELPER) += drm_ttm_helper.o
 # Modesetting helpers
 #
 
-drm_kms_helper-y := drm_bridge_connector.o drm_crtc_helper.o \
-               drm_encoder_slave.o drm_flip_work.o \
-               drm_probe_helper.o \
-               drm_plane_helper.o drm_atomic_helper.o \
-               drm_kms_helper_common.o \
-               drm_simple_kms_helper.o drm_modeset_helper.o \
-               drm_gem_atomic_helper.o \
-               drm_gem_framebuffer_helper.o \
-               drm_atomic_state_helper.o drm_damage_helper.o \
-               drm_format_helper.o drm_self_refresh_helper.o drm_rect.o
+drm_kms_helper-y := \
+       drm_atomic_helper.o \
+       drm_atomic_state_helper.o \
+       drm_bridge_connector.o \
+       drm_crtc_helper.o \
+       drm_damage_helper.o \
+       drm_encoder_slave.o \
+       drm_flip_work.o \
+       drm_format_helper.o \
+       drm_gem_atomic_helper.o \
+       drm_gem_framebuffer_helper.o \
+       drm_kms_helper_common.o \
+       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
 drm_kms_helper-$(CONFIG_DRM_FBDEV_EMULATION) += drm_fb_helper.o
 obj-$(CONFIG_DRM_KMS_HELPER) += drm_kms_helper.o
index 52cdda1180d923a6b0c837e7986964ae33ceed2e..17ac4a1006a8008794477d8a634d699f372415a0 100644 (file)
@@ -3,13 +3,15 @@
 obj-$(CONFIG_DRM_DP_AUX_BUS) += drm_dp_aux_bus.o
 
 drm_display_helper-y := drm_display_helper_mod.o
-drm_display_helper-$(CONFIG_DRM_DISPLAY_DP_HELPER) += drm_dp_dual_mode_helper.o \
-                                                     drm_dp_helper.o \
-                                                     drm_dp_mst_topology.o \
-                                                     drm_dsc_helper.o
+drm_display_helper-$(CONFIG_DRM_DISPLAY_DP_HELPER) += \
+       drm_dp_dual_mode_helper.o \
+       drm_dp_helper.o \
+       drm_dp_mst_topology.o \
+       drm_dsc_helper.o
 drm_display_helper-$(CONFIG_DRM_DISPLAY_HDCP_HELPER) += drm_hdcp_helper.o
-drm_display_helper-$(CONFIG_DRM_DISPLAY_HDMI_HELPER) += drm_hdmi_helper.o \
-                                                       drm_scdc_helper.o
+drm_display_helper-$(CONFIG_DRM_DISPLAY_HDMI_HELPER) += \
+       drm_hdmi_helper.o \
+       drm_scdc_helper.o
 drm_display_helper-$(CONFIG_DRM_DP_AUX_CHARDEV) += drm_dp_aux_dev.o
 drm_display_helper-$(CONFIG_DRM_DP_CEC) += drm_dp_cec.o