Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/trace: split out display trace to a separate file

2021-12-09 Thread Lucas De Marchi
On Wed, Dec 08, 2021 at 06:39:55PM +0200, Jani Nikula wrote: On Wed, 08 Dec 2021, Lucas De Marchi wrote: On Wed, Dec 08, 2021 at 01:05:17PM +0200, Jani Nikula wrote: Add display/intel_display_trace.[ch] for defining display tracepoints. The main goal is to reduce cross-includes between gem and

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/trace: split out display trace to a separate file

2021-12-09 Thread Jani Nikula
On Wed, 08 Dec 2021, Jani Nikula wrote: > On Wed, 08 Dec 2021, Lucas De Marchi wrote: >> On Wed, Dec 08, 2021 at 01:05:17PM +0200, Jani Nikula wrote: >>>Add display/intel_display_trace.[ch] for defining display >>>tracepoints. The main goal is to reduce cross-includes between gem and >>>display.

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/trace: split out display trace to a separate file

2021-12-08 Thread Jani Nikula
On Wed, 08 Dec 2021, Lucas De Marchi wrote: > On Wed, Dec 08, 2021 at 01:05:17PM +0200, Jani Nikula wrote: >>Add display/intel_display_trace.[ch] for defining display >>tracepoints. The main goal is to reduce cross-includes between gem and >>display. It would be possible split up tracing even furt

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/trace: split out display trace to a separate file

2021-12-08 Thread Lucas De Marchi
On Wed, Dec 08, 2021 at 01:05:17PM +0200, Jani Nikula wrote: Add display/intel_display_trace.[ch] for defining display tracepoints. The main goal is to reduce cross-includes between gem and display. It would be possible split up tracing even further, but that would lead to more boilerplate. We e

[Intel-gfx] [PATCH v3 2/2] drm/i915/trace: split out display trace to a separate file

2021-12-08 Thread Jani Nikula
Add display/intel_display_trace.[ch] for defining display tracepoints. The main goal is to reduce cross-includes between gem and display. It would be possible split up tracing even further, but that would lead to more boilerplate. We end up having to include intel_crtc.h in a few places because it