diff options
author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2013-10-09 10:24:58 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-10-10 06:47:14 -0400 |
commit | 4c445e0ebc648ee42c0d21713b8f76597854d47a (patch) | |
tree | d18626390d5324a79a05113f578c1bf8dc20f26c /drivers | |
parent | e5b611fd4493d09eb5164f5244ac0a5325346895 (diff) |
drm/i915: Rename primary_disabled to primary_enabled
Let's try to avoid these confusing negated booleans.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 10 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_drv.h | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_pm.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_sprite.c | 8 |
4 files changed, 11 insertions, 11 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index b54a4cf899f9..ebe5d0840a86 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -1840,9 +1840,9 @@ static void intel_enable_primary_plane(struct drm_i915_private *dev_priv, | |||
1840 | /* If the pipe isn't enabled, we can't pump pixels and may hang */ | 1840 | /* If the pipe isn't enabled, we can't pump pixels and may hang */ |
1841 | assert_pipe_enabled(dev_priv, pipe); | 1841 | assert_pipe_enabled(dev_priv, pipe); |
1842 | 1842 | ||
1843 | WARN(!intel_crtc->primary_disabled, "Primary plane already enabled\n"); | 1843 | WARN(intel_crtc->primary_enabled, "Primary plane already enabled\n"); |
1844 | 1844 | ||
1845 | intel_crtc->primary_disabled = false; | 1845 | intel_crtc->primary_enabled = true; |
1846 | 1846 | ||
1847 | reg = DSPCNTR(plane); | 1847 | reg = DSPCNTR(plane); |
1848 | val = I915_READ(reg); | 1848 | val = I915_READ(reg); |
@@ -1870,9 +1870,9 @@ static void intel_disable_primary_plane(struct drm_i915_private *dev_priv, | |||
1870 | int reg; | 1870 | int reg; |
1871 | u32 val; | 1871 | u32 val; |
1872 | 1872 | ||
1873 | WARN(intel_crtc->primary_disabled, "Primary plane already disabled\n"); | 1873 | WARN(!intel_crtc->primary_enabled, "Primary plane already disabled\n"); |
1874 | 1874 | ||
1875 | intel_crtc->primary_disabled = true; | 1875 | intel_crtc->primary_enabled = false; |
1876 | 1876 | ||
1877 | reg = DSPCNTR(plane); | 1877 | reg = DSPCNTR(plane); |
1878 | val = I915_READ(reg); | 1878 | val = I915_READ(reg); |
@@ -10706,7 +10706,7 @@ static void intel_modeset_readout_hw_state(struct drm_device *dev) | |||
10706 | &crtc->config); | 10706 | &crtc->config); |
10707 | 10707 | ||
10708 | crtc->base.enabled = crtc->active; | 10708 | crtc->base.enabled = crtc->active; |
10709 | crtc->primary_disabled = !crtc->active; | 10709 | crtc->primary_enabled = crtc->active; |
10710 | 10710 | ||
10711 | DRM_DEBUG_KMS("[CRTC:%d] hw state readout: %s\n", | 10711 | DRM_DEBUG_KMS("[CRTC:%d] hw state readout: %s\n", |
10712 | crtc->base.base.id, | 10712 | crtc->base.base.id, |
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h index ce80289de69e..b497a96af082 100644 --- a/drivers/gpu/drm/i915/intel_drv.h +++ b/drivers/gpu/drm/i915/intel_drv.h | |||
@@ -321,7 +321,7 @@ struct intel_crtc { | |||
321 | */ | 321 | */ |
322 | bool active; | 322 | bool active; |
323 | bool eld_vld; | 323 | bool eld_vld; |
324 | bool primary_disabled; /* is the crtc obscured by a plane? */ | 324 | bool primary_enabled; /* is the primary plane (partially) visible? */ |
325 | bool lowfreq_avail; | 325 | bool lowfreq_avail; |
326 | struct intel_overlay *overlay; | 326 | struct intel_overlay *overlay; |
327 | struct intel_unpin_work *unpin_work; | 327 | struct intel_unpin_work *unpin_work; |
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index 9534e72fdbcb..c91087a542ec 100644 --- a/drivers/gpu/drm/i915/intel_pm.c +++ b/drivers/gpu/drm/i915/intel_pm.c | |||
@@ -475,7 +475,7 @@ void intel_update_fbc(struct drm_device *dev) | |||
475 | */ | 475 | */ |
476 | list_for_each_entry(tmp_crtc, &dev->mode_config.crtc_list, head) { | 476 | list_for_each_entry(tmp_crtc, &dev->mode_config.crtc_list, head) { |
477 | if (intel_crtc_active(tmp_crtc) && | 477 | if (intel_crtc_active(tmp_crtc) && |
478 | !to_intel_crtc(tmp_crtc)->primary_disabled) { | 478 | to_intel_crtc(tmp_crtc)->primary_enabled) { |
479 | if (crtc) { | 479 | if (crtc) { |
480 | if (set_no_fbc_reason(dev_priv, FBC_MULTIPLE_PIPES)) | 480 | if (set_no_fbc_reason(dev_priv, FBC_MULTIPLE_PIPES)) |
481 | DRM_DEBUG_KMS("more than one pipe active, disabling compression\n"); | 481 | DRM_DEBUG_KMS("more than one pipe active, disabling compression\n"); |
diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c index e001d2c35c39..8afaad6bcc48 100644 --- a/drivers/gpu/drm/i915/intel_sprite.c +++ b/drivers/gpu/drm/i915/intel_sprite.c | |||
@@ -521,10 +521,10 @@ intel_enable_primary(struct drm_crtc *crtc) | |||
521 | struct intel_crtc *intel_crtc = to_intel_crtc(crtc); | 521 | struct intel_crtc *intel_crtc = to_intel_crtc(crtc); |
522 | int reg = DSPCNTR(intel_crtc->plane); | 522 | int reg = DSPCNTR(intel_crtc->plane); |
523 | 523 | ||
524 | if (!intel_crtc->primary_disabled) | 524 | if (intel_crtc->primary_enabled) |
525 | return; | 525 | return; |
526 | 526 | ||
527 | intel_crtc->primary_disabled = false; | 527 | intel_crtc->primary_enabled = true; |
528 | 528 | ||
529 | I915_WRITE(reg, I915_READ(reg) | DISPLAY_PLANE_ENABLE); | 529 | I915_WRITE(reg, I915_READ(reg) | DISPLAY_PLANE_ENABLE); |
530 | intel_flush_primary_plane(dev_priv, intel_crtc->plane); | 530 | intel_flush_primary_plane(dev_priv, intel_crtc->plane); |
@@ -553,10 +553,10 @@ intel_disable_primary(struct drm_crtc *crtc) | |||
553 | struct intel_crtc *intel_crtc = to_intel_crtc(crtc); | 553 | struct intel_crtc *intel_crtc = to_intel_crtc(crtc); |
554 | int reg = DSPCNTR(intel_crtc->plane); | 554 | int reg = DSPCNTR(intel_crtc->plane); |
555 | 555 | ||
556 | if (intel_crtc->primary_disabled) | 556 | if (!intel_crtc->primary_enabled) |
557 | return; | 557 | return; |
558 | 558 | ||
559 | intel_crtc->primary_disabled = true; | 559 | intel_crtc->primary_enabled = false; |
560 | 560 | ||
561 | mutex_lock(&dev->struct_mutex); | 561 | mutex_lock(&dev->struct_mutex); |
562 | if (dev_priv->fbc.plane == intel_crtc->plane) | 562 | if (dev_priv->fbc.plane == intel_crtc->plane) |