Re: [PATCH] drm/i915/display: remove unused for_each_crtc()

[Jani Nikula <jani.nikula@intel.com>]

On Mon, 02 Dec 2024, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> Quoting Jani Nikula (2024-12-02 09:54:37-03:00)
>>This is essentially a duplicate of drm_for_each_crtc() anyway. Remove.
>
> And it looks like commit 08dfd243abac ("drm/i915: Clean up
> intel_fbdev_init_bios() a bit") was the one that made it unused.
>
>>
>>Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>
> Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>

Thanks for the review, pushed to din.

BR,
Jani.

>
>>---
>> drivers/gpu/drm/i915/display/intel_display.h | 3 ---
>> 1 file changed, 3 deletions(-)
>>
>>diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h
>>index caef04f655c5..49a246feb1ae 100644
>>--- a/drivers/gpu/drm/i915/display/intel_display.h
>>+++ b/drivers/gpu/drm/i915/display/intel_display.h
>>@@ -238,9 +238,6 @@ enum phy_fia {
>>         for ((__phy) = PHY_A; (__phy) < I915_MAX_PHYS; (__phy)++)        \
>>                 for_each_if((__phys_mask) & BIT(__phy))
>> 
>>-#define for_each_crtc(dev, crtc) \
>>-        list_for_each_entry(crtc, &(dev)->mode_config.crtc_list, head)
>>-
>> #define for_each_intel_plane(dev, intel_plane) \
>>         list_for_each_entry(intel_plane,                        \
>>                             &(dev)->mode_config.plane_list,        \
>>-- 
>>2.39.5
>>

-- 
Jani Nikula, Intel