Re: [PATCH v2 0/3] drm: replace drm_print.h includes from headers with a forward declaration

[Thomas Zimmermann <tzimmermann@suse.de>]

Hi

Am 27.10.25 um 13:04 schrieb Jani Nikula:
> On Mon, 27 Oct 2025, Thomas Zimmermann <tzimmermann@suse.de> wrote:
>> Hi,
>>
>> for the series
>>
>> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
> Thanks!
>
>> Some thoughts: Consider merging patches 1 and 2 OR split off the change
>> to ttm_resource.h from patch 3.  Doing the latter would make a revert of
>> the forward declaration easier in case some trees stop building. Feel
>> free to put everything into drm-misc-next if that still compiles then.
> I'm sorry, I don't understand what you're getting at. The series is
> bisectable, and in the first patch the indirect source of drm_print.h is
> drm_mm.h, and in the third patch it's ttm_resource.h.

Sorry, I misread that. Then maybe consider splitting off those header 
changes into separate patches. They could be reverted by themselves 
easily without touching all the drivers. Your choice.

Best regards
Thomas

>
> BR,
> Jani.
>
>

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)