[Intel-xe] [PATCH v2 11/11] drm/xe: Use relative includes for i915_reg_defs.h

[Lucas De Marchi]

On Fri, Feb 17, 2023 at 03:28:24PM -0500, Rodrigo Vivi wrote:
>On Thu, Feb 16, 2023 at 04:52:26PM -0800, Lucas De Marchi wrote:
>> With the goal of eliminating i915 includes from xe outside of the
>> display area, it's better to use relative includes for the only shared
>> header, i915_reg_defs.h.  With this the build system can be changed to
>> stop adding i915 as an include dir.
>>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
>> ---
>>  drivers/gpu/drm/xe/display/Makefile      | 120 +++++++++++++++++++++++
>>  drivers/gpu/drm/xe/regs/xe_engine_regs.h |   2 +-
>>  drivers/gpu/drm/xe/regs/xe_gt_regs.h     |   2 +-
>>  drivers/gpu/drm/xe/regs/xe_regs.h        |   2 +-
>>  drivers/gpu/drm/xe/xe_gt_mcr.h           |   2 +-
>>  drivers/gpu/drm/xe/xe_guc.c              |   2 -
>>  drivers/gpu/drm/xe/xe_guc_reg.h          |   2 +-
>>  drivers/gpu/drm/xe/xe_reg_sr_types.h     |   2 -
>>  drivers/gpu/drm/xe/xe_rtp.h              |   2 -
>>  drivers/gpu/drm/xe/xe_rtp_types.h        |   2 -
>>  10 files changed, 125 insertions(+), 13 deletions(-)
>>  create mode 100644 drivers/gpu/drm/xe/display/Makefile
>>
>> diff --git a/drivers/gpu/drm/xe/display/Makefile b/drivers/gpu/drm/xe/display/Makefile
>> new file mode 100644
>> index 000000000000..eee576972427
>> --- /dev/null
>> +++ b/drivers/gpu/drm/xe/display/Makefile
>
>shouldn't this patch also be modifying the drm/xe/Makefile?

sorry, while trying to resolve the display situation I squashed this by
mistake here. Next version I'm giving up on the idea in this patch
and I will just create a regs/xe_reg_defs.h. That will be the only file
with the relative include.

Lucas De Marchi