On Fri, 05 Sep 2025, Ville Syrjälä <[email protected]> wrote: > On Wed, Sep 03, 2025 at 12:04:08PM +0300, Jani Nikula wrote: >> Add a proper function for display && HAS_DISPLAY(display) to hide >> indirect struct intel_display access via the macro from a number of >> places outside of display. This makes struct intel_display * an opaque >> pointer in these places. All HAS_DISPLAY() usage is now constrained >> within display. > > Looks all right. Though a lot of code where this gets used really > shouldn't exist outside the display code in the first place. But > cleaning all that up is going to take some effort.
Agreed. > Reviewed-by: Ville Syrjälä <[email protected]> Pushed to din, thanks for the review. BR, Jani. -- Jani Nikula, Intel
