[PATCH 1/3] drm/nouveau: include drm/drm_edid.h only where needed

Jani Nikula jani.nikula at intel.com
Wed Jan 10 09:18:19 UTC 2024


On Tue, 09 Jan 2024, Danilo Krummrich <dakr at redhat.com> wrote:
> On 1/9/24 10:59, Jani Nikula wrote:
>> On Mon, 08 Jan 2024, Danilo Krummrich <dakr at redhat.com> wrote:
>>> On 1/4/24 21:16, Jani Nikula wrote:
>>>> Including drm_edid.h from nouveau_connector.h causes the rebuild of 15
>>>> files when drm_edid.h is modified, while there are only a few files that
>>>> actually need to include drm_edid.h.
>>>>
>>>> Cc: Karol Herbst <kherbst at redhat.com>
>>>> Cc: Lyude Paul <lyude at redhat.com>
>>>> Cc: Danilo Krummrich <dakr at redhat.com>
>>>> Cc: nouveau at lists.freedesktop.org
>>>> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
>>>
>>> Reviewed-by: Danilo Krummrich <dakr at redhat.com>
>> 
>> Are you going to pick this up via the nouveau tree, or shall I apply it
>> to drm-misc-next?
>
> We don't maintain a separate tree, hence feel free to apply it to drm-misc-next.

Thanks for the review, pushed to drm-misc-next.

You do still have

T:	git https://gitlab.freedesktop.org/drm/nouveau.git

MAINTAINERS entry, so I thought that's where you apply patches.

BR,
Jani.


-- 
Jani Nikula, Intel


More information about the dri-devel mailing list