Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Lucas De Marchi <lucas.demarchi@intel.com>
Cc: intel-xe@lists.freedesktop.org,
	Maarten Lankhorst <maarten.lankhorst@intel.com>,
	Matthew Auld <matthew.auld@intel.com>
Subject: Re: [Intel-xe] [PATCH v2 08/11] drm/xe: Remove dependency on i915_reg.h
Date: Fri, 17 Feb 2023 15:27:12 -0500	[thread overview]
Message-ID: <Y+/jICXsAkeW0Onw@intel.com> (raw)
In-Reply-To: <20230217005226.106499-9-lucas.demarchi@intel.com>

On Thu, Feb 16, 2023 at 04:52:23PM -0800, Lucas De Marchi wrote:
> Copy the macros used by xe in i915_reg.h to regs/xe_regs.h. A minimal
> cleanup is done while copying so they adhere minimally to the coding
> style.  Further reordering and cleaning is left for later.
> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
>  drivers/gpu/drm/xe/regs/xe_regs.h      | 111 +++++++++++++++++++++++++
>  drivers/gpu/drm/xe/xe_execlist.c       |   3 +-
>  drivers/gpu/drm/xe/xe_ggtt.c           |   3 +-
>  drivers/gpu/drm/xe/xe_gt_clock.c       |   3 +-
>  drivers/gpu/drm/xe/xe_guc_pc.c         |   3 +-
>  drivers/gpu/drm/xe/xe_hw_engine.c      |   3 +-
>  drivers/gpu/drm/xe/xe_irq.c            |   3 +-
>  drivers/gpu/drm/xe/xe_lrc.c            |   3 +-
>  drivers/gpu/drm/xe/xe_mmio.c           |   3 +-
>  drivers/gpu/drm/xe/xe_pci.c            |   3 +-
>  drivers/gpu/drm/xe/xe_ring_ops.c       |   3 +-
>  drivers/gpu/drm/xe/xe_ttm_stolen_mgr.c |   3 +-
>  drivers/gpu/drm/xe/xe_wa.c             |   3 +-
>  13 files changed, 123 insertions(+), 24 deletions(-)
>  create mode 100644 drivers/gpu/drm/xe/regs/xe_regs.h
> 
> diff --git a/drivers/gpu/drm/xe/regs/xe_regs.h b/drivers/gpu/drm/xe/regs/xe_regs.h
> new file mode 100644
> index 000000000000..53f1ed54fb1c
> --- /dev/null
> +++ b/drivers/gpu/drm/xe/regs/xe_regs.h
> @@ -0,0 +1,111 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * Copyright © 2023 Intel Corporation
> + */
> +#ifndef _XE_REGS_H_
> +#define _XE_REGS_H_
> +
> +#include "i915_reg_defs.h"
> +
> +#define GU_CNTL					_MMIO(0x101010)
> +#define   LMEM_INIT				REG_BIT(7)
> +
> +#define RENDER_RING_BASE			0x02000
> +#define BSD_RING_BASE				0x04000
> +#define GEN6_BSD_RING_BASE			0x12000
> +#define GEN8_BSD2_RING_BASE			0x1c000
> +#define GEN11_BSD_RING_BASE			0x1c0000
> +#define GEN11_BSD2_RING_BASE			0x1c4000
> +#define GEN11_BSD3_RING_BASE			0x1d0000
> +#define GEN11_BSD4_RING_BASE			0x1d4000
> +#define XEHP_BSD5_RING_BASE			0x1e0000
> +#define XEHP_BSD6_RING_BASE			0x1e4000
> +#define XEHP_BSD7_RING_BASE			0x1f0000
> +#define XEHP_BSD8_RING_BASE			0x1f4000
> +#define VEBOX_RING_BASE				0x1a000
> +#define GEN11_VEBOX_RING_BASE			0x1c8000
> +#define GEN11_VEBOX2_RING_BASE			0x1d8000
> +#define XEHP_VEBOX3_RING_BASE			0x1e8000
> +#define XEHP_VEBOX4_RING_BASE			0x1f8000
> +#define GEN12_COMPUTE0_RING_BASE		0x1a000
> +#define GEN12_COMPUTE1_RING_BASE		0x1c000
> +#define GEN12_COMPUTE2_RING_BASE		0x1e000
> +#define GEN12_COMPUTE3_RING_BASE		0x26000
> +#define BLT_RING_BASE				0x22000
> +#define XEHPC_BCS1_RING_BASE			0x3e0000
> +#define XEHPC_BCS2_RING_BASE			0x3e2000
> +#define XEHPC_BCS3_RING_BASE			0x3e4000
> +#define XEHPC_BCS4_RING_BASE			0x3e6000
> +#define XEHPC_BCS5_RING_BASE			0x3e8000
> +#define XEHPC_BCS6_RING_BASE			0x3ea000
> +#define XEHPC_BCS7_RING_BASE			0x3ec000
> +#define XEHPC_BCS8_RING_BASE			0x3ee000
> +#define   GT_WAIT_SEMAPHORE_INTERRUPT		REG_BIT(11)
> +#define   GT_CONTEXT_SWITCH_INTERRUPT		(1 <<  8)
> +#define   GT_RENDER_PIPECTL_NOTIFY_INTERRUPT	(1 <<  4)
> +#define   GT_CS_MASTER_ERROR_INTERRUPT		REG_BIT(3)
> +#define   GT_RENDER_USER_INTERRUPT		(1 <<  0)
> +
> +#define GEN7_FF_THREAD_MODE			_MMIO(0x20a0)
> +#define   GEN12_FF_TESSELATION_DOP_GATE_DISABLE BIT(19)
> +
> +#define PVC_RP_STATE_CAP			_MMIO(0x281014)
> +#define MTL_RP_STATE_CAP			_MMIO(0x138000)
> +
> +#define MTL_MEDIAP_STATE_CAP			_MMIO(0x138020)
> +#define   MTL_RP0_CAP_MASK			REG_GENMASK(8, 0)
> +#define   MTL_RPN_CAP_MASK			REG_GENMASK(24, 16)
> +
> +#define MTL_GT_RPE_FREQUENCY			_MMIO(0x13800c)
> +#define MTL_MPE_FREQUENCY			_MMIO(0x13802c)
> +#define   MTL_RPE_MASK				REG_GENMASK(8, 0)
> +
> +#define TRANSCODER_A_OFFSET			0x60000
> +#define TRANSCODER_B_OFFSET			0x61000
> +#define TRANSCODER_C_OFFSET			0x62000
> +#define TRANSCODER_D_OFFSET			0x63000
> +#define TRANSCODER_DSI0_OFFSET			0x6b000
> +#define TRANSCODER_DSI1_OFFSET			0x6b800
> +#define PIPE_A_OFFSET				0x70000
> +#define PIPE_B_OFFSET				0x71000
> +#define PIPE_C_OFFSET				0x72000
> +#define PIPE_D_OFFSET				0x73000
> +#define PIPE_DSI0_OFFSET			0x7b000
> +#define PIPE_DSI1_OFFSET			0x7b800
> +
> +#define GEN8_PCU_ISR				_MMIO(0x444e0)
> +#define GEN8_PCU_IMR				_MMIO(0x444e4)
> +#define GEN8_PCU_IIR				_MMIO(0x444e8)
> +#define GEN8_PCU_IER				_MMIO(0x444ec)
> +
> +#define GEN11_GU_MISC_ISR			_MMIO(0x444f0)
> +#define GEN11_GU_MISC_IMR			_MMIO(0x444f4)
> +#define GEN11_GU_MISC_IIR			_MMIO(0x444f8)
> +#define GEN11_GU_MISC_IER			_MMIO(0x444fc)
> +#define   GEN11_GU_MISC_GSE			(1 << 27)
> +
> +#define GEN11_GFX_MSTR_IRQ			_MMIO(0x190010)
> +#define   GEN11_MASTER_IRQ			(1 << 31)
> +#define   GEN11_GU_MISC_IRQ			(1 << 29)
> +#define   GEN11_DISPLAY_IRQ			(1 << 16)
> +#define   GEN11_GT_DW_IRQ(x)			(1 << (x))
> +
> +#define DG1_MSTR_TILE_INTR			_MMIO(0x190008)
> +#define   DG1_MSTR_IRQ				REG_BIT(31)
> +#define   DG1_MSTR_TILE(t)			REG_BIT(t)
> +
> +#define GEN9_TIMESTAMP_OVERRIDE					_MMIO(0x44074)
> +#define   GEN9_TIMESTAMP_OVERRIDE_US_COUNTER_DIVIDER_SHIFT	0
> +#define   GEN9_TIMESTAMP_OVERRIDE_US_COUNTER_DIVIDER_MASK	0x3ff
> +#define   GEN9_TIMESTAMP_OVERRIDE_US_COUNTER_DENOMINATOR_SHIFT	12
> +#define   GEN9_TIMESTAMP_OVERRIDE_US_COUNTER_DENOMINATOR_MASK	(0xf << 12)
> +
> +#define GGC					_MMIO(0x108040)
> +#define   GMS_MASK				REG_GENMASK(15, 8)
> +#define   GGMS_MASK				REG_GENMASK(7, 6)

one of the things that I'm always confused is with these REG_{BIT,GENMASK,FIELD_PREP}.
Why can't we simply use the regular "non-REG_" versions like every other driver
(aside of i915 of course).

> +
> +#define GEN12_GSMBASE				_MMIO(0x108100)
> +#define GEN12_DSMBASE				_MMIO(0x1080C0)
> +#define   GEN12_BDSM_MASK			REG_GENMASK64(63, 20)
> +
> +#endif
> diff --git a/drivers/gpu/drm/xe/xe_execlist.c b/drivers/gpu/drm/xe/xe_execlist.c
> index c3c52bdc70b1..ad097918b496 100644
> --- a/drivers/gpu/drm/xe/xe_execlist.c
> +++ b/drivers/gpu/drm/xe/xe_execlist.c
> @@ -10,6 +10,7 @@
>  #include "regs/xe_lrc_regs.h"
>  #include "regs/xe_gpu_commands.h"
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_bo.h"
>  #include "xe_device.h"
>  #include "xe_engine.h"
> @@ -22,8 +23,6 @@
>  #include "xe_ring_ops_types.h"
>  #include "xe_sched_job.h"
>  
> -#include "i915_reg.h"
> -
>  #define XE_EXECLIST_HANG_LIMIT 1
>  
>  #define GEN11_SW_CTX_ID_SHIFT 37
> diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
> index 2c9b2175c5bb..6674297fa25e 100644
> --- a/drivers/gpu/drm/xe/xe_ggtt.c
> +++ b/drivers/gpu/drm/xe/xe_ggtt.c
> @@ -10,6 +10,7 @@
>  #include <drm/i915_drm.h>
>  
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_bo.h"
>  #include "xe_device.h"
>  #include "xe_gt.h"
> @@ -18,8 +19,6 @@
>  #include "xe_mmio.h"
>  #include "xe_wopcm.h"
>  
> -#include "i915_reg.h"
> -
>  /* FIXME: Common file, preferably auto-gen */
>  #define MTL_GGTT_PTE_PAT0	BIT_ULL(52)
>  #define MTL_GGTT_PTE_PAT1	BIT_ULL(53)
> diff --git a/drivers/gpu/drm/xe/xe_gt_clock.c b/drivers/gpu/drm/xe/xe_gt_clock.c
> index e9aa7c5452af..cbf9c96c142c 100644
> --- a/drivers/gpu/drm/xe/xe_gt_clock.c
> +++ b/drivers/gpu/drm/xe/xe_gt_clock.c
> @@ -5,13 +5,12 @@
>  #include "xe_gt_clock.h"
>  
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_device.h"
>  #include "xe_gt.h"
>  #include "xe_macros.h"
>  #include "xe_mmio.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);
> diff --git a/drivers/gpu/drm/xe/xe_guc_pc.c b/drivers/gpu/drm/xe/xe_guc_pc.c
> index f7aaf4826f00..0df82defa7e4 100644
> --- a/drivers/gpu/drm/xe/xe_guc_pc.c
> +++ b/drivers/gpu/drm/xe/xe_guc_pc.c
> @@ -9,6 +9,7 @@
>  #include <drm/drm_managed.h>
>  
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_bo.h"
>  #include "xe_device.h"
>  #include "xe_gt.h"
> @@ -19,8 +20,6 @@
>  #include "xe_mmio.h"
>  #include "xe_pcode.h"
>  
> -#include "i915_reg.h"
> -#include "i915_reg_defs.h"
>  #include "intel_mchbar_regs.h"
>  
>  /* For GEN6_RP_STATE_CAP.reg to be merged when the definition moves to Xe */
> diff --git a/drivers/gpu/drm/xe/xe_hw_engine.c b/drivers/gpu/drm/xe/xe_hw_engine.c
> index caae4f897644..2b49ccba0ce9 100644
> --- a/drivers/gpu/drm/xe/xe_hw_engine.c
> +++ b/drivers/gpu/drm/xe/xe_hw_engine.c
> @@ -8,6 +8,7 @@
>  
>  #include "regs/xe_engine_regs.h"
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_bo.h"
>  #include "xe_device.h"
>  #include "xe_execlist.h"
> @@ -22,8 +23,6 @@
>  #include "xe_sched_job.h"
>  #include "xe_wa.h"
>  
> -#include "i915_reg.h"
> -
>  #define MAX_MMIO_BASES 3
>  struct engine_info {
>  	const char *name;
> diff --git a/drivers/gpu/drm/xe/xe_irq.c b/drivers/gpu/drm/xe/xe_irq.c
> index d7756c14b4e2..f4f84a16dd83 100644
> --- a/drivers/gpu/drm/xe/xe_irq.c
> +++ b/drivers/gpu/drm/xe/xe_irq.c
> @@ -9,6 +9,7 @@
>  #include <drm/drm_managed.h>
>  
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_device.h"
>  #include "xe_display.h"
>  #include "xe_drv.h"
> @@ -17,8 +18,6 @@
>  #include "xe_hw_engine.h"
>  #include "xe_mmio.h"
>  
> -#include "i915_reg.h"
> -
>  static void gen3_assert_iir_is_zero(struct xe_gt *gt, i915_reg_t reg)
>  {
>  	u32 val = xe_mmio_read32(gt, reg.reg);
> diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
> index c1df76c2cf78..261eabaae369 100644
> --- a/drivers/gpu/drm/xe/xe_lrc.c
> +++ b/drivers/gpu/drm/xe/xe_lrc.c
> @@ -8,6 +8,7 @@
>  #include "regs/xe_lrc_regs.h"
>  #include "regs/xe_gpu_commands.h"
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_bo.h"
>  #include "xe_device.h"
>  #include "xe_engine_types.h"
> @@ -16,8 +17,6 @@
>  #include "xe_map.h"
>  #include "xe_vm.h"
>  
> -#include "i915_reg.h"
> -
>  #define GEN8_CTX_VALID				(1 << 0)
>  #define GEN8_CTX_L3LLC_COHERENT			(1 << 5)
>  #define GEN8_CTX_PRIVILEGE			(1 << 8)
> diff --git a/drivers/gpu/drm/xe/xe_mmio.c b/drivers/gpu/drm/xe/xe_mmio.c
> index d0cd9e920d34..262d33e59d26 100644
> --- a/drivers/gpu/drm/xe/xe_mmio.c
> +++ b/drivers/gpu/drm/xe/xe_mmio.c
> @@ -9,14 +9,13 @@
>  
>  #include "regs/xe_engine_regs.h"
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_device.h"
>  #include "xe_gt.h"
>  #include "xe_gt_mcr.h"
>  #include "xe_macros.h"
>  #include "xe_module.h"
>  
> -#include "i915_reg.h"
> -
>  #define XEHP_MTCFG_ADDR		_MMIO(0x101800)
>  #define TILE_COUNT		REG_GENMASK(15, 8)
>  #define GEN12_LMEM_BAR		2
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index b0e5c402190c..4ecde7c2e619 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -13,6 +13,7 @@
>  #include <drm/drm_drv.h>
>  #include <drm/xe_pciids.h>
>  
> +#include "regs/xe_regs.h"
>  #include "xe_device.h"
>  #include "xe_drv.h"
>  #include "xe_macros.h"
> @@ -20,8 +21,6 @@
>  #include "xe_pm.h"
>  #include "xe_step.h"
>  
> -#include "i915_reg.h"
> -
>  #define DEV_INFO_FOR_EACH_FLAG(func) \
>  	func(require_force_probe); \
>  	func(is_dgfx); \
> diff --git a/drivers/gpu/drm/xe/xe_ring_ops.c b/drivers/gpu/drm/xe/xe_ring_ops.c
> index 104f96658e1f..ba3bde117af1 100644
> --- a/drivers/gpu/drm/xe/xe_ring_ops.c
> +++ b/drivers/gpu/drm/xe/xe_ring_ops.c
> @@ -7,6 +7,7 @@
>  #include "regs/xe_lrc_regs.h"
>  #include "regs/xe_gpu_commands.h"
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_engine_types.h"
>  #include "xe_gt.h"
>  #include "xe_lrc.h"
> @@ -14,8 +15,6 @@
>  #include "xe_sched_job.h"
>  #include "xe_vm_types.h"
>  
> -#include "i915_reg.h"
> -
>  static u32 preparser_disable(bool state)
>  {
>  	return MI_ARB_CHECK | BIT(8) | state;
> diff --git a/drivers/gpu/drm/xe/xe_ttm_stolen_mgr.c b/drivers/gpu/drm/xe/xe_ttm_stolen_mgr.c
> index 1c3783becefd..19828e003283 100644
> --- a/drivers/gpu/drm/xe/xe_ttm_stolen_mgr.c
> +++ b/drivers/gpu/drm/xe/xe_ttm_stolen_mgr.c
> @@ -11,8 +11,7 @@
>  #include <drm/ttm/ttm_placement.h>
>  #include <drm/ttm/ttm_range_manager.h>
>  
> -#include "../i915/i915_reg.h"
> -
> +#include "regs/xe_regs.h"
>  #include "xe_bo.h"
>  #include "xe_device.h"
>  #include "xe_gt.h"
> diff --git a/drivers/gpu/drm/xe/xe_wa.c b/drivers/gpu/drm/xe/xe_wa.c
> index 155cfd1dcc50..df72b15dfeb0 100644
> --- a/drivers/gpu/drm/xe/xe_wa.c
> +++ b/drivers/gpu/drm/xe/xe_wa.c
> @@ -9,6 +9,7 @@
>  
>  #include "regs/xe_engine_regs.h"
>  #include "regs/xe_gt_regs.h"
> +#include "regs/xe_regs.h"
>  #include "xe_device_types.h"
>  #include "xe_force_wake.h"
>  #include "xe_gt.h"
> @@ -18,8 +19,6 @@
>  #include "xe_rtp.h"
>  #include "xe_step.h"
>  
> -#include "i915_reg.h"
> -
>  /**
>   * DOC: Hardware workarounds
>   *
> -- 
> 2.39.0
> 

  reply	other threads:[~2023-02-17 20:27 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-17  0:52 [Intel-xe] [PATCH v2 00/11] Start register cleanup Lucas De Marchi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 01/11] drm/xe: Remove outdated build workaround Lucas De Marchi
2023-02-17 20:13   ` Rodrigo Vivi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 02/11] drm/xe: Sort includes Lucas De Marchi
2023-02-17 11:20   ` Matthew Auld
2023-02-17 15:03     ` Lucas De Marchi
2023-02-17 11:39   ` Jani Nikula
2023-02-17 15:09     ` Lucas De Marchi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 03/11] drm/xe/guc: Remove i915_regs.h include Lucas De Marchi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 04/11] drm/xe: Remove dependency on intel_engine_regs.h Lucas De Marchi
2023-02-24 18:02   ` Matt Roper
2023-02-24 18:05     ` Lucas De Marchi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 05/11] drm/xe: Remove dependency on intel_gt_regs.h Lucas De Marchi
2023-02-17 20:20   ` Rodrigo Vivi
2023-02-17 22:08     ` Lucas De Marchi
2023-02-24 18:06   ` Matt Roper
2023-02-24 18:29     ` Lucas De Marchi
2023-02-24 18:34       ` Matt Roper
2023-02-24 19:47         ` Vivi, Rodrigo
2023-02-24 20:25           ` Lucas De Marchi
2023-02-24 21:22             ` Vivi, Rodrigo
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 06/11] drm/xe: Remove dependency on intel_lrc_reg.h Lucas De Marchi
2023-02-24 18:20   ` Matt Roper
2023-02-24 18:54     ` Lucas De Marchi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 07/11] drm/xe: Remove dependency on intel_gpu_commands.h Lucas De Marchi
2023-02-24 18:26   ` Matt Roper
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 08/11] drm/xe: Remove dependency on i915_reg.h Lucas De Marchi
2023-02-17 20:27   ` Rodrigo Vivi [this message]
2023-02-17 22:22     ` Lucas De Marchi
2023-02-27 14:19       ` Jani Nikula
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 09/11] drm/xe/guc_pc: Move gt register to the proper place Lucas De Marchi
2023-02-17 20:23   ` Rodrigo Vivi
2023-02-17 22:11     ` Lucas De Marchi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 10/11] drm/xe: Remove dependency on intel_mchbar_regs.h Lucas De Marchi
2023-02-17 20:18   ` Rodrigo Vivi
2023-02-17  0:52 ` [Intel-xe] [PATCH v2 11/11] drm/xe: Use relative includes for i915_reg_defs.h Lucas De Marchi
2023-02-17 16:33   ` Lucas De Marchi
2023-02-17 20:28   ` Rodrigo Vivi
2023-02-17 22:30     ` Lucas De Marchi
2023-02-21 21:42       ` Rodrigo Vivi
2023-02-21 21:50         ` Lucas De Marchi
2023-02-17  6:19 ` [Intel-xe] [PATCH v2 00/11] Start register cleanup Lucas De Marchi

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=Y+/jICXsAkeW0Onw@intel.com \
    --to=rodrigo.vivi@intel.com \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=lucas.demarchi@intel.com \
    --cc=maarten.lankhorst@intel.com \
    --cc=matthew.auld@intel.com \
    /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