Commit 8834e365 authored by Jani Nikula's avatar Jani Nikula

drm/i915: extract intel_fifo_underrun.h from intel_drv.h

It used to be handy that we only had a couple of headers, but over time
intel_drv.h has become unwieldy. Extract declarations to a separate
header file corresponding to the implementation module, clarifying the
modularity of the driver.

Ensure the new header is self-contained, and do so with minimal further
includes, using forward declarations as needed. Include the new header
only where needed, and sort the modified include directives while at it
and as needed.

No functional changes.
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/0507c5523d1f07a48e6679a04db75246ce8ba766.1556540889.git.jani.nikula@intel.com
parent 3db9de3b
...@@ -29,6 +29,7 @@ header_test := \ ...@@ -29,6 +29,7 @@ header_test := \
intel_dvo_dev.h \ intel_dvo_dev.h \
intel_fbc.h \ intel_fbc.h \
intel_fbdev.h \ intel_fbdev.h \
intel_fifo_underrun.h \
intel_frontbuffer.h \ intel_frontbuffer.h \
intel_hdcp.h \ intel_hdcp.h \
intel_hdmi.h \ intel_hdmi.h \
......
...@@ -40,6 +40,7 @@ ...@@ -40,6 +40,7 @@
#include "i915_drv.h" #include "i915_drv.h"
#include "i915_trace.h" #include "i915_trace.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_fifo_underrun.h"
#include "intel_psr.h" #include "intel_psr.h"
/** /**
......
...@@ -39,6 +39,7 @@ ...@@ -39,6 +39,7 @@
#include "intel_crt.h" #include "intel_crt.h"
#include "intel_ddi.h" #include "intel_ddi.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_fifo_underrun.h"
/* Here's the desired hotplug mode */ /* Here's the desired hotplug mode */
#define ADPA_HOTPLUG_BITS (ADPA_CRT_HOTPLUG_PERIOD_128 | \ #define ADPA_HOTPLUG_BITS (ADPA_CRT_HOTPLUG_PERIOD_128 | \
......
...@@ -34,6 +34,7 @@ ...@@ -34,6 +34,7 @@
#include "intel_dp.h" #include "intel_dp.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_dsi.h" #include "intel_dsi.h"
#include "intel_fifo_underrun.h"
#include "intel_hdcp.h" #include "intel_hdcp.h"
#include "intel_hdmi.h" #include "intel_hdmi.h"
#include "intel_lspcon.h" #include "intel_lspcon.h"
......
...@@ -58,6 +58,7 @@ ...@@ -58,6 +58,7 @@
#include "intel_dvo.h" #include "intel_dvo.h"
#include "intel_fbc.h" #include "intel_fbc.h"
#include "intel_fbdev.h" #include "intel_fbdev.h"
#include "intel_fifo_underrun.h"
#include "intel_frontbuffer.h" #include "intel_frontbuffer.h"
#include "intel_hdcp.h" #include "intel_hdcp.h"
#include "intel_hdmi.h" #include "intel_hdmi.h"
......
...@@ -48,6 +48,7 @@ ...@@ -48,6 +48,7 @@
#include "intel_ddi.h" #include "intel_ddi.h"
#include "intel_dp.h" #include "intel_dp.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_fifo_underrun.h"
#include "intel_hdcp.h" #include "intel_hdcp.h"
#include "intel_hdmi.h" #include "intel_hdmi.h"
#include "intel_lspcon.h" #include "intel_lspcon.h"
......
...@@ -1581,19 +1581,6 @@ intel_atomic_get_new_crtc_state(struct intel_atomic_state *state, ...@@ -1581,19 +1581,6 @@ intel_atomic_get_new_crtc_state(struct intel_atomic_state *state,
&crtc->base)); &crtc->base));
} }
/* intel_fifo_underrun.c */
bool intel_set_cpu_fifo_underrun_reporting(struct drm_i915_private *dev_priv,
enum pipe pipe, bool enable);
bool intel_set_pch_fifo_underrun_reporting(struct drm_i915_private *dev_priv,
enum pipe pch_transcoder,
bool enable);
void intel_cpu_fifo_underrun_irq_handler(struct drm_i915_private *dev_priv,
enum pipe pipe);
void intel_pch_fifo_underrun_irq_handler(struct drm_i915_private *dev_priv,
enum pipe pch_transcoder);
void intel_check_cpu_fifo_underruns(struct drm_i915_private *dev_priv);
void intel_check_pch_fifo_underruns(struct drm_i915_private *dev_priv);
/* i915_irq.c */ /* i915_irq.c */
void gen5_enable_gt_irq(struct drm_i915_private *dev_priv, u32 mask); void gen5_enable_gt_irq(struct drm_i915_private *dev_priv, u32 mask);
void gen5_disable_gt_irq(struct drm_i915_private *dev_priv, u32 mask); void gen5_disable_gt_irq(struct drm_i915_private *dev_priv, u32 mask);
......
...@@ -28,6 +28,7 @@ ...@@ -28,6 +28,7 @@
#include "i915_drv.h" #include "i915_drv.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_fbc.h" #include "intel_fbc.h"
#include "intel_fifo_underrun.h"
/** /**
* DOC: fifo underrun handling * DOC: fifo underrun handling
......
/* SPDX-License-Identifier: MIT */
/*
* Copyright © 2019 Intel Corporation
*/
#ifndef __INTEL_FIFO_UNDERRUN_H__
#define __INTEL_FIFO_UNDERRUN_H__
#include <linux/types.h>
#include "intel_display.h"
struct drm_i915_private;
bool intel_set_cpu_fifo_underrun_reporting(struct drm_i915_private *dev_priv,
enum pipe pipe, bool enable);
bool intel_set_pch_fifo_underrun_reporting(struct drm_i915_private *dev_priv,
enum pipe pch_transcoder,
bool enable);
void intel_cpu_fifo_underrun_irq_handler(struct drm_i915_private *dev_priv,
enum pipe pipe);
void intel_pch_fifo_underrun_irq_handler(struct drm_i915_private *dev_priv,
enum pipe pch_transcoder);
void intel_check_cpu_fifo_underruns(struct drm_i915_private *dev_priv);
void intel_check_pch_fifo_underruns(struct drm_i915_private *dev_priv);
#endif /* __INTEL_FIFO_UNDERRUN_H__ */
...@@ -45,6 +45,7 @@ ...@@ -45,6 +45,7 @@
#include "intel_ddi.h" #include "intel_ddi.h"
#include "intel_dp.h" #include "intel_dp.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_fifo_underrun.h"
#include "intel_hdcp.h" #include "intel_hdcp.h"
#include "intel_hdmi.h" #include "intel_hdmi.h"
#include "intel_lspcon.h" #include "intel_lspcon.h"
......
...@@ -39,6 +39,7 @@ ...@@ -39,6 +39,7 @@
#include "i915_drv.h" #include "i915_drv.h"
#include "intel_connector.h" #include "intel_connector.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_fifo_underrun.h"
#include "intel_hdmi.h" #include "intel_hdmi.h"
#include "intel_panel.h" #include "intel_panel.h"
#include "intel_sdvo.h" #include "intel_sdvo.h"
......
...@@ -35,6 +35,7 @@ ...@@ -35,6 +35,7 @@
#include "intel_connector.h" #include "intel_connector.h"
#include "intel_drv.h" #include "intel_drv.h"
#include "intel_dsi.h" #include "intel_dsi.h"
#include "intel_fifo_underrun.h"
#include "intel_panel.h" #include "intel_panel.h"
#include "intel_sideband.h" #include "intel_sideband.h"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment