mbox series

[0/4] drm/i915: deconflate display disable from no display

Message ID cover.1567446845.git.jani.nikula@intel.com (mailing list archive)
Headers show
Series drm/i915: deconflate display disable from no display | expand

Message

Jani Nikula Sept. 2, 2019, 6:08 p.m. UTC
Deconflate not having display hardware from having disabled display
hardware, with some collateral improvements.

This doesn't actually fix any of the issues resulting from the two being
conflated, but unblocks fixing both independently.

Read the commit messages for details.

BR,
Jani.

Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: José Roberto de Souza <jose.souza@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>

Jani Nikula (4):
  drm/i915: add INTEL_NUM_PIPES() and use it
  drm/i915: convert device info num_pipes to pipe_mask
  drm/i915: introduce INTEL_DISPLAY_ENABLED()
  drm/i915: stop conflating HAS_DISPLAY() and disabled display

 drivers/gpu/drm/i915/display/intel_bios.c     |  2 +-
 drivers/gpu/drm/i915/display/intel_display.c  | 30 ++++++++++---------
 drivers/gpu/drm/i915/display/intel_display.h  |  4 +--
 drivers/gpu/drm/i915/display/intel_fbdev.c    |  2 +-
 drivers/gpu/drm/i915/display/intel_gmbus.c    |  2 +-
 .../gpu/drm/i915/display/intel_lpe_audio.c    |  2 +-
 drivers/gpu/drm/i915/i915_drv.c               | 10 +++----
 drivers/gpu/drm/i915/i915_drv.h               |  7 ++++-
 drivers/gpu/drm/i915/i915_pci.c               | 24 +++++++--------
 drivers/gpu/drm/i915/intel_device_info.c      | 16 ++++------
 drivers/gpu/drm/i915/intel_device_info.h      |  2 +-
 drivers/gpu/drm/i915/intel_pch.c              |  2 +-
 drivers/gpu/drm/i915/intel_pm.c               |  6 ++--
 13 files changed, 56 insertions(+), 53 deletions(-)

Comments

Ville Syrjala Sept. 5, 2019, 12:28 p.m. UTC | #1
On Mon, Sep 02, 2019 at 09:08:12PM +0300, Jani Nikula wrote:
> Deconflate not having display hardware from having disabled display
> hardware, with some collateral improvements.
> 
> This doesn't actually fix any of the issues resulting from the two being
> conflated, but unblocks fixing both independently.
> 
> Read the commit messages for details.
> 
> BR,
> Jani.
> 
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>

Looks good based on a cursor scan. Series is
Acked-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> 
> Jani Nikula (4):
>   drm/i915: add INTEL_NUM_PIPES() and use it
>   drm/i915: convert device info num_pipes to pipe_mask
>   drm/i915: introduce INTEL_DISPLAY_ENABLED()
>   drm/i915: stop conflating HAS_DISPLAY() and disabled display
> 
>  drivers/gpu/drm/i915/display/intel_bios.c     |  2 +-
>  drivers/gpu/drm/i915/display/intel_display.c  | 30 ++++++++++---------
>  drivers/gpu/drm/i915/display/intel_display.h  |  4 +--
>  drivers/gpu/drm/i915/display/intel_fbdev.c    |  2 +-
>  drivers/gpu/drm/i915/display/intel_gmbus.c    |  2 +-
>  .../gpu/drm/i915/display/intel_lpe_audio.c    |  2 +-
>  drivers/gpu/drm/i915/i915_drv.c               | 10 +++----
>  drivers/gpu/drm/i915/i915_drv.h               |  7 ++++-
>  drivers/gpu/drm/i915/i915_pci.c               | 24 +++++++--------
>  drivers/gpu/drm/i915/intel_device_info.c      | 16 ++++------
>  drivers/gpu/drm/i915/intel_device_info.h      |  2 +-
>  drivers/gpu/drm/i915/intel_pch.c              |  2 +-
>  drivers/gpu/drm/i915/intel_pm.c               |  6 ++--
>  13 files changed, 56 insertions(+), 53 deletions(-)
> 
> -- 
> 2.20.1