On Thu, 2025-06-26 at 13:16 +0300, Jani Nikula wrote:
> With the switch to an unordered workqueue dedicated to display, we've
> stopped using struct drm_i915_private in a number of places, and can
> drop the dependencies on i915_drv.h.
>
> Cc: Luca Coelho <luciano.coe...@intel.com>
> Signed-off-by: Jani Nikula <jani.nik...@intel.com>
> ---
Nice! I had removed some of these, but apparently I forgot and lost
those changes.
[...]
> @@ -32,7 +31,7 @@ void intel_encoder_link_check_flush_work(struct
> intel_encoder *encoder)
>
> void intel_encoder_link_check_queue_work(struct intel_encoder *encoder, int
> delay_ms)
> {
> - struct intel_display *display = to_i915(encoder->base.dev)->display;
> + struct intel_display *display = to_intel_display(encoder);
>
> mod_delayed_work(display->wq.unordered,
> &encoder->link_check_work, msecs_to_jiffies(delay_ms));
This looks much nicer. I should have looked more for a better macro.
Reviewed-by: Luca Coelho <luciano.coe...@intel.com>
--
Cheers,
Luca.
