[PATCH 2/2] drm/xe: drop display/ subdir from include directories

Jani Nikula jani.nikula at intel.com
Mon Jan 22 17:39:56 UTC 2024


On Mon, 22 Jan 2024, Lucas De Marchi <lucas.demarchi at intel.com> wrote:
> the downside of patch 1 is now that core xe code can include any of the
> display/ headers, but only xe_display.h is acceptable to keep the
> interface sane.

I'd say xe_display.h remains the only interface towards xe core, I don't
think this patch changes that, and its location doesn't really give any
guarantees. It's been a matter of sticking display/ in the include
anyway, and enforcing that is a matter of maintainer vigilance.

> Or are you thinking about changing the interface?

I agree the interface should be in one file only, but changing the
interface is an orthogonal matter (I have no plans atm).

BR,
Jani.


-- 
Jani Nikula, Intel


More information about the Intel-xe mailing list