Skip to content

Commit

Permalink
drm/xe: Sort includes
Browse files Browse the repository at this point in the history
Sort includes and split them in blocks:

1) .h corresponding to the .c. Example: xe_bb.c should have a "#include
   "xe_bb.h" first.
2) #include <linux/...>
3) #include <drm/...>
4) local includes
5) i915 includes

This is accomplished by running
`clang-format --style=file -i --sort-includes drivers/gpu/drm/xe/*.[ch]`
and ignoring all the changes after the includes. There are also some
manual tweaks to split the blocks.

v2: Also sort includes in headers

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
  • Loading branch information
Lucas De Marchi authored and Rodrigo Vivi committed Dec 19, 2023
1 parent ba11f1b commit ea9f879
Show file tree
Hide file tree
Showing 66 changed files with 172 additions and 125 deletions.
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_bb.c
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,11 @@
*/

#include "xe_bb.h"
#include "xe_sa.h"

#include "xe_device.h"
#include "xe_engine_types.h"
#include "xe_hw_fence.h"
#include "xe_sa.h"
#include "xe_sched_job.h"
#include "xe_vm_types.h"

Expand Down
1 change: 0 additions & 1 deletion drivers/gpu/drm/xe/xe_bo.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
* Copyright © 2021 Intel Corporation
*/


#include "xe_bo.h"

#include <linux/dma-buf.h>
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_bo_evict.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_bo.h"
#include "xe_bo_evict.h"

#include "xe_bo.h"
#include "xe_device.h"
#include "xe_ggtt.h"
#include "xe_gt.h"
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_debugfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,14 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_debugfs.h"

#include <linux/string_helpers.h>

#include <drm/drm_debugfs.h>

#include "xe_bo.h"
#include "xe_device.h"
#include "xe_debugfs.h"
#include "xe_gt_debugfs.h"
#include "xe_step.h"

Expand Down
8 changes: 4 additions & 4 deletions drivers/gpu/drm/xe/xe_device.c
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,12 @@

#include "xe_device.h"

#include <drm/drm_gem_ttm_helper.h>
#include <drm/drm_aperture.h>
#include <drm/drm_atomic_helper.h>
#include <drm/drm_gem_ttm_helper.h>
#include <drm/drm_ioctl.h>
#include <drm/xe_drm.h>
#include <drm/drm_managed.h>
#include <drm/drm_atomic_helper.h>
#include <drm/xe_drm.h>

#include "xe_bo.h"
#include "xe_debugfs.h"
Expand All @@ -20,8 +20,8 @@
#include "xe_exec.h"
#include "xe_gt.h"
#include "xe_irq.h"
#include "xe_module.h"
#include "xe_mmio.h"
#include "xe_module.h"
#include "xe_pcode.h"
#include "xe_pm.h"
#include "xe_query.h"
Expand Down
2 changes: 1 addition & 1 deletion drivers/gpu/drm/xe/xe_device.h
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ struct xe_file;
#include <drm/drm_util.h>

#include "xe_device_types.h"
#include "xe_macros.h"
#include "xe_force_wake.h"
#include "xe_macros.h"

#include "gt/intel_gpu_commands.h"

Expand Down
9 changes: 4 additions & 5 deletions drivers/gpu/drm/xe/xe_dma_buf.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,20 +3,19 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_dma_buf.h"

#include <kunit/test.h>
#include <linux/dma-buf.h>
#include <linux/pci-p2pdma.h>

#include <drm/drm_device.h>
#include <drm/drm_prime.h>

#include <drm/ttm/ttm_tt.h>

#include <kunit/test.h>
#include <linux/pci-p2pdma.h>

#include "tests/xe_test.h"
#include "xe_bo.h"
#include "xe_device.h"
#include "xe_dma_buf.h"
#include "xe_ttm_vram_mgr.h"
#include "xe_vm.h"

Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_engine.c
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,11 @@

#include "xe_engine.h"

#include <linux/nospec.h>

#include <drm/drm_device.h>
#include <drm/drm_file.h>
#include <drm/xe_drm.h>
#include <linux/nospec.h>

#include "xe_device.h"
#include "xe_gt.h"
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_exec.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,15 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_exec.h"

#include <drm/drm_device.h>
#include <drm/drm_file.h>
#include <drm/xe_drm.h>

#include "xe_bo.h"
#include "xe_device.h"
#include "xe_engine.h"
#include "xe_exec.h"
#include "xe_macros.h"
#include "xe_sched_job.h"
#include "xe_sync.h"
Expand Down
10 changes: 5 additions & 5 deletions drivers/gpu/drm/xe/xe_execlist.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,27 +3,27 @@
* Copyright © 2021 Intel Corporation
*/

#include <drm/drm_managed.h>

#include "xe_execlist.h"

#include <drm/drm_managed.h>

#include "xe_bo.h"
#include "xe_device.h"
#include "xe_engine.h"
#include "xe_hw_fence.h"
#include "xe_gt.h"
#include "xe_hw_fence.h"
#include "xe_lrc.h"
#include "xe_macros.h"
#include "xe_mmio.h"
#include "xe_mocs.h"
#include "xe_ring_ops_types.h"
#include "xe_sched_job.h"

#include "i915_reg.h"
#include "gt/intel_engine_regs.h"
#include "gt/intel_gpu_commands.h"
#include "gt/intel_gt_regs.h"
#include "gt/intel_lrc_reg.h"
#include "gt/intel_engine_regs.h"
#include "i915_reg.h"

#define XE_EXECLIST_HANG_LIMIT 1

Expand Down
5 changes: 3 additions & 2 deletions drivers/gpu/drm/xe/xe_force_wake.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,14 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_force_wake.h"

#include <drm/drm_util.h>

#include "xe_force_wake.h"
#include "xe_gt.h"
#include "xe_mmio.h"
#include "gt/intel_gt_regs.h"

#include "gt/intel_gt_regs.h"

#define XE_FORCE_WAKE_ACK_TIMEOUT_MS 50

Expand Down
6 changes: 3 additions & 3 deletions drivers/gpu/drm/xe/xe_ggtt.c
Original file line number Diff line number Diff line change
Expand Up @@ -6,20 +6,20 @@
#include "xe_ggtt.h"

#include <linux/sizes.h>
#include <drm/i915_drm.h>

#include <drm/drm_managed.h>
#include <drm/i915_drm.h>

#include "xe_device.h"
#include "xe_bo.h"
#include "xe_device.h"
#include "xe_gt.h"
#include "xe_gt_tlb_invalidation.h"
#include "xe_map.h"
#include "xe_mmio.h"
#include "xe_wopcm.h"

#include "i915_reg.h"
#include "gt/intel_gt_regs.h"
#include "i915_reg.h"

/* FIXME: Common file, preferably auto-gen */
#define MTL_GGTT_PTE_PAT0 BIT_ULL(52)
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_gt.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_gt.h"

#include <linux/minmax.h>

#include <drm/drm_managed.h>
Expand All @@ -14,7 +16,6 @@
#include "xe_execlist.h"
#include "xe_force_wake.h"
#include "xe_ggtt.h"
#include "xe_gt.h"
#include "xe_gt_clock.h"
#include "xe_gt_mcr.h"
#include "xe_gt_pagefault.h"
Expand Down
7 changes: 4 additions & 3 deletions drivers/gpu/drm/xe/xe_gt_clock.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,16 @@
* Copyright © 2022 Intel Corporation
*/

#include "i915_reg.h"
#include "gt/intel_gt_regs.h"
#include "xe_gt_clock.h"

#include "xe_device.h"
#include "xe_gt.h"
#include "xe_gt_clock.h"
#include "xe_macros.h"
#include "xe_mmio.h"

#include "gt/intel_gt_regs.h"
#include "i915_reg.h"

static u32 read_reference_ts_freq(struct xe_gt *gt)
{
u32 ts_override = xe_mmio_read32(gt, GEN9_TIMESTAMP_OVERRIDE.reg);
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_gt_debugfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,15 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_gt_debugfs.h"

#include <drm/drm_debugfs.h>
#include <drm/drm_managed.h>

#include "xe_device.h"
#include "xe_force_wake.h"
#include "xe_ggtt.h"
#include "xe_gt.h"
#include "xe_gt_debugfs.h"
#include "xe_gt_mcr.h"
#include "xe_gt_topology.h"
#include "xe_hw_engine.h"
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_gt_mcr.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_gt.h"
#include "xe_gt_mcr.h"

#include "xe_gt.h"
#include "xe_gt_topology.h"
#include "xe_gt_types.h"
#include "xe_mmio.h"
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_gt_pagefault.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,15 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_gt_pagefault.h"

#include <linux/circ_buf.h>

#include <drm/drm_managed.h>
#include <drm/ttm/ttm_execbuf_util.h>

#include "xe_bo.h"
#include "xe_gt.h"
#include "xe_gt_pagefault.h"
#include "xe_gt_tlb_invalidation.h"
#include "xe_guc.h"
#include "xe_guc_ct.h"
Expand Down
5 changes: 4 additions & 1 deletion drivers/gpu/drm/xe/xe_gt_sysfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,14 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_gt_sysfs.h"

#include <linux/kobject.h>
#include <linux/sysfs.h>

#include <drm/drm_managed.h>

#include "xe_gt.h"
#include "xe_gt_sysfs.h"

static void xe_gt_sysfs_kobj_release(struct kobject *kobj)
{
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_gt_tlb_invalidation.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@
* Copyright © 2023 Intel Corporation
*/

#include "xe_gt.h"
#include "xe_gt_tlb_invalidation.h"

#include "xe_gt.h"
#include "xe_guc.h"
#include "xe_guc_ct.h"
#include "xe_trace.h"
Expand Down
3 changes: 2 additions & 1 deletion drivers/gpu/drm/xe/xe_gt_topology.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,11 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_gt_topology.h"

#include <linux/bitmap.h>

#include "xe_gt.h"
#include "xe_gt_topology.h"
#include "xe_mmio.h"

#define XE_MAX_DSS_FUSE_BITS (32 * XE_MAX_DSS_FUSE_REGS)
Expand Down
14 changes: 8 additions & 6 deletions drivers/gpu/drm/xe/xe_guc.c
Original file line number Diff line number Diff line change
Expand Up @@ -3,24 +3,26 @@
* Copyright © 2022 Intel Corporation
*/

#include "xe_guc.h"

#include "xe_bo.h"
#include "xe_device.h"
#include "xe_guc.h"
#include "xe_force_wake.h"
#include "xe_gt.h"
#include "xe_guc_ads.h"
#include "xe_guc_ct.h"
#include "xe_guc_hwconfig.h"
#include "xe_guc_log.h"
#include "xe_guc_reg.h"
#include "xe_guc_pc.h"
#include "xe_guc_reg.h"
#include "xe_guc_submit.h"
#include "xe_gt.h"
#include "xe_mmio.h"
#include "xe_platform_types.h"
#include "xe_uc_fw.h"
#include "xe_wopcm.h"
#include "xe_mmio.h"
#include "xe_force_wake.h"
#include "i915_reg_defs.h"

#include "gt/intel_gt_regs.h"
#include "i915_reg_defs.h"

/* TODO: move to common file */
#define GUC_PVC_MOCS_INDEX_MASK REG_GENMASK(25, 24)
Expand Down
2 changes: 1 addition & 1 deletion drivers/gpu/drm/xe/xe_guc.h
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@
#ifndef _XE_GUC_H_
#define _XE_GUC_H_

#include "xe_hw_engine_types.h"
#include "xe_guc_types.h"
#include "xe_hw_engine_types.h"
#include "xe_macros.h"

struct drm_printer;
Expand Down
Loading

0 comments on commit ea9f879

Please sign in to comment.