On Mon, 10 Nov 2025, Ville Syrjälä <[email protected]> wrote:
> On Mon, Nov 10, 2025 at 09:31:39PM +0200, Jani Nikula wrote:
>> Shorten the intel_display_irq_regs_* function names to the underlying
>> function names with a _wl suffix for clarity.
>> 
>> - intel_display_irq_regs_init() -> irq_init_wl()
>> - intel_display_irq_regs_reset() -> irq_reset_wl()
>> - intel_display_irq_regs_assert_irr_is_zero() -> assert_iir_is_zero_wl()
>> 
>> This emphasizes the difference is the wakelock. Platforms without the
>> DMC wakelock mechanism can use the non-wl versions.
>
> These things should just get nuked. You already switched to
> intel_de_{read,write}() in the previous patch, and those already
> handle the wakelock stuff for you.

Oh where's my brown paper bag when I need it.


-- 
Jani Nikula, Intel

Reply via email to