Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Gustavo Sousa <gustavo.sousa@intel.com>
Cc: <intel-xe@lists.freedesktop.org>
Subject: Re: [PATCH] drm/xe: Fix sort order of .o lists in Makefile
Date: Wed, 15 Jan 2025 09:48:12 -0500	[thread overview]
Message-ID: <Z4fKrIvakPtqRDTG@intel.com> (raw)
In-Reply-To: <20250115140812.20799-1-gustavo.sousa@intel.com>

On Wed, Jan 15, 2025 at 11:08:04AM -0300, Gustavo Sousa wrote:
> The Makefile for xe asks us to keep the lists of object files sorted:
> 
>   # Please keep these build lists sorted!
> 
> Reshuffle the lists into the correct sort order. That was done by
> filtering each unsorted list through 'LC_ALL=C sort'.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> 
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
>  drivers/gpu/drm/xe/Makefile | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/Makefile b/drivers/gpu/drm/xe/Makefile
> index 5c97ad6ed738..34214400b8d1 100644
> --- a/drivers/gpu/drm/xe/Makefile
> +++ b/drivers/gpu/drm/xe/Makefile
> @@ -34,8 +34,8 @@ xe-y += xe_bb.o \
>  	xe_dma_buf.o \
>  	xe_drm_client.o \
>  	xe_exec.o \
> -	xe_execlist.o \
>  	xe_exec_queue.o \
> +	xe_execlist.o \
>  	xe_force_wake.o \
>  	xe_ggtt.o \
>  	xe_gpu_scheduler.o \
> @@ -66,11 +66,11 @@ xe-y += xe_bb.o \
>  	xe_guc_pc.o \
>  	xe_guc_submit.o \
>  	xe_heci_gsc.o \
> +	xe_huc.o \
>  	xe_hw_engine.o \
>  	xe_hw_engine_class_sysfs.o \
>  	xe_hw_engine_group.o \
>  	xe_hw_fence.o \
> -	xe_huc.o \
>  	xe_irq.o \
>  	xe_lrc.o \
>  	xe_migrate.o \
> @@ -90,8 +90,8 @@ xe-y += xe_bb.o \
>  	xe_range_fence.o \
>  	xe_reg_sr.o \
>  	xe_reg_whitelist.o \
> -	xe_rtp.o \
>  	xe_ring_ops.o \
> +	xe_rtp.o \
>  	xe_sa.o \
>  	xe_sched_job.o \
>  	xe_step.o \
> @@ -102,8 +102,8 @@ xe-y += xe_bb.o \
>  	xe_trace_bo.o \
>  	xe_trace_guc.o \
>  	xe_trace_lrc.o \
> -	xe_ttm_sys_mgr.o \
>  	xe_ttm_stolen_mgr.o \
> +	xe_ttm_sys_mgr.o \
>  	xe_ttm_vram_mgr.o \
>  	xe_tuning.o \
>  	xe_uc.o \
> @@ -112,8 +112,8 @@ xe-y += xe_bb.o \
>  	xe_vram.o \
>  	xe_vram_freq.o \
>  	xe_vsec.o \
> -	xe_wait_user_fence.o \
>  	xe_wa.o \
> +	xe_wait_user_fence.o \
>  	xe_wopcm.o
>  
>  xe-$(CONFIG_HMM_MIRROR) += xe_hmm.o
> @@ -221,6 +221,7 @@ xe-$(CONFIG_DRM_XE_DISPLAY) += \
>  	i915-display/intel_display_wa.o \
>  	i915-display/intel_dkl_phy.o \
>  	i915-display/intel_dmc.o \
> +	i915-display/intel_dmc_wl.o \
>  	i915-display/intel_dp.o \
>  	i915-display/intel_dp_aux.o \
>  	i915-display/intel_dp_aux_backlight.o \
> @@ -268,7 +269,6 @@ xe-$(CONFIG_DRM_XE_DISPLAY) += \
>  	i915-display/intel_vdsc.o \
>  	i915-display/intel_vga.o \
>  	i915-display/intel_vrr.o \
> -	i915-display/intel_dmc_wl.o \
>  	i915-display/intel_wm.o \
>  	i915-display/skl_scaler.o \
>  	i915-display/skl_universal_plane.o \
> -- 
> 2.48.0
> 

  parent reply	other threads:[~2025-01-15 14:48 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-15 14:08 [PATCH] drm/xe: Fix sort order of .o lists in Makefile Gustavo Sousa
2025-01-15 14:19 ` ✓ CI.Patch_applied: success for " Patchwork
2025-01-15 14:20 ` ✓ CI.checkpatch: " Patchwork
2025-01-15 14:21 ` ✓ CI.KUnit: " Patchwork
2025-01-15 14:37 ` [PATCH] " Lucas De Marchi
2025-01-15 14:39 ` ✓ CI.Build: success for " Patchwork
2025-01-15 14:41 ` ✓ CI.Hooks: " Patchwork
2025-01-15 14:43 ` ✓ CI.checksparse: " Patchwork
2025-01-15 14:48 ` Rodrigo Vivi [this message]
2025-01-15 15:09 ` ✗ Xe.CI.BAT: failure " Patchwork
2025-01-15 16:20   ` Gustavo Sousa
2025-01-16 10:48     ` Ravali, JupallyX
2025-01-30 17:43     ` Matt Roper
2025-02-03  4:49       ` Ravali, JupallyX
2025-01-15 20:44 ` ✗ Xe.CI.Full: " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Z4fKrIvakPtqRDTG@intel.com \
    --to=rodrigo.vivi@intel.com \
    --cc=gustavo.sousa@intel.com \
    --cc=intel-xe@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox