Commit 40560e26 authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915: Use drm_plane_mask() & co.

Use drm_{plane,crtc,encoder,connector}_mask() where appropriate.
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180626194716.12522-5-ville.syrjala@linux.intel.comReviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent 73705732
...@@ -2756,10 +2756,10 @@ intel_set_plane_visible(struct intel_crtc_state *crtc_state, ...@@ -2756,10 +2756,10 @@ intel_set_plane_visible(struct intel_crtc_state *crtc_state,
/* FIXME pre-g4x don't work like this */ /* FIXME pre-g4x don't work like this */
if (visible) { if (visible) {
crtc_state->base.plane_mask |= BIT(drm_plane_index(&plane->base)); crtc_state->base.plane_mask |= drm_plane_mask(&plane->base);
crtc_state->active_planes |= BIT(plane->id); crtc_state->active_planes |= BIT(plane->id);
} else { } else {
crtc_state->base.plane_mask &= ~BIT(drm_plane_index(&plane->base)); crtc_state->base.plane_mask &= ~drm_plane_mask(&plane->base);
crtc_state->active_planes &= ~BIT(plane->id); crtc_state->active_planes &= ~BIT(plane->id);
} }
...@@ -11884,7 +11884,7 @@ verify_single_dpll_state(struct drm_i915_private *dev_priv, ...@@ -11884,7 +11884,7 @@ verify_single_dpll_state(struct drm_i915_private *dev_priv,
struct drm_crtc_state *new_state) struct drm_crtc_state *new_state)
{ {
struct intel_dpll_hw_state dpll_hw_state; struct intel_dpll_hw_state dpll_hw_state;
unsigned crtc_mask; unsigned int crtc_mask;
bool active; bool active;
memset(&dpll_hw_state, 0, sizeof(dpll_hw_state)); memset(&dpll_hw_state, 0, sizeof(dpll_hw_state));
...@@ -11911,7 +11911,7 @@ verify_single_dpll_state(struct drm_i915_private *dev_priv, ...@@ -11911,7 +11911,7 @@ verify_single_dpll_state(struct drm_i915_private *dev_priv,
return; return;
} }
crtc_mask = 1 << drm_crtc_index(crtc); crtc_mask = drm_crtc_mask(crtc);
if (new_state->active) if (new_state->active)
I915_STATE_WARN(!(pll->active_mask & crtc_mask), I915_STATE_WARN(!(pll->active_mask & crtc_mask),
...@@ -11946,7 +11946,7 @@ verify_shared_dpll_state(struct drm_device *dev, struct drm_crtc *crtc, ...@@ -11946,7 +11946,7 @@ verify_shared_dpll_state(struct drm_device *dev, struct drm_crtc *crtc,
if (old_state->shared_dpll && if (old_state->shared_dpll &&
old_state->shared_dpll != new_state->shared_dpll) { old_state->shared_dpll != new_state->shared_dpll) {
unsigned crtc_mask = 1 << drm_crtc_index(crtc); unsigned int crtc_mask = drm_crtc_mask(crtc);
struct intel_shared_dpll *pll = old_state->shared_dpll; struct intel_shared_dpll *pll = old_state->shared_dpll;
I915_STATE_WARN(pll->active_mask & crtc_mask, I915_STATE_WARN(pll->active_mask & crtc_mask,
...@@ -15608,9 +15608,9 @@ static void intel_modeset_readout_hw_state(struct drm_device *dev) ...@@ -15608,9 +15608,9 @@ static void intel_modeset_readout_hw_state(struct drm_device *dev)
* rely on the connector_mask being accurate. * rely on the connector_mask being accurate.
*/ */
encoder->base.crtc->state->connector_mask |= encoder->base.crtc->state->connector_mask |=
1 << drm_connector_index(&connector->base); drm_connector_mask(&connector->base);
encoder->base.crtc->state->encoder_mask |= encoder->base.crtc->state->encoder_mask |=
1 << drm_encoder_index(&encoder->base); drm_encoder_mask(&encoder->base);
} }
} else { } else {
......
...@@ -261,7 +261,7 @@ struct intel_link_m_n { ...@@ -261,7 +261,7 @@ struct intel_link_m_n {
&(dev)->mode_config.plane_list, \ &(dev)->mode_config.plane_list, \
base.head) \ base.head) \
for_each_if((plane_mask) & \ for_each_if((plane_mask) & \
BIT(drm_plane_index(&intel_plane->base))) drm_plane_mask(&intel_plane->base)))
#define for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) \ #define for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) \
list_for_each_entry(intel_plane, \ list_for_each_entry(intel_plane, \
...@@ -278,7 +278,7 @@ struct intel_link_m_n { ...@@ -278,7 +278,7 @@ struct intel_link_m_n {
list_for_each_entry(intel_crtc, \ list_for_each_entry(intel_crtc, \
&(dev)->mode_config.crtc_list, \ &(dev)->mode_config.crtc_list, \
base.head) \ base.head) \
for_each_if((crtc_mask) & BIT(drm_crtc_index(&intel_crtc->base))) for_each_if((crtc_mask) & drm_crtc_mask(&intel_crtc->base))
#define for_each_intel_encoder(dev, intel_encoder) \ #define for_each_intel_encoder(dev, intel_encoder) \
list_for_each_entry(intel_encoder, \ list_for_each_entry(intel_encoder, \
......
...@@ -163,8 +163,8 @@ void intel_enable_shared_dpll(struct intel_crtc *crtc) ...@@ -163,8 +163,8 @@ void intel_enable_shared_dpll(struct intel_crtc *crtc)
struct drm_device *dev = crtc->base.dev; struct drm_device *dev = crtc->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev); struct drm_i915_private *dev_priv = to_i915(dev);
struct intel_shared_dpll *pll = crtc->config->shared_dpll; struct intel_shared_dpll *pll = crtc->config->shared_dpll;
unsigned crtc_mask = 1 << drm_crtc_index(&crtc->base); unsigned int crtc_mask = drm_crtc_mask(&crtc->base);
unsigned old_mask; unsigned int old_mask;
if (WARN_ON(pll == NULL)) if (WARN_ON(pll == NULL))
return; return;
...@@ -207,7 +207,7 @@ void intel_disable_shared_dpll(struct intel_crtc *crtc) ...@@ -207,7 +207,7 @@ void intel_disable_shared_dpll(struct intel_crtc *crtc)
{ {
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
struct intel_shared_dpll *pll = crtc->config->shared_dpll; struct intel_shared_dpll *pll = crtc->config->shared_dpll;
unsigned crtc_mask = 1 << drm_crtc_index(&crtc->base); unsigned int crtc_mask = drm_crtc_mask(&crtc->base);
/* PCH only available on ILK+ */ /* PCH only available on ILK+ */
if (INTEL_GEN(dev_priv) < 5) if (INTEL_GEN(dev_priv) < 5)
......
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