Commit 998bd66a authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915: Set .domains=POWER_DOMAIN_MASK for the always-on well

The always-on well is the same as runtime PM, so we should just
"enable" it for any power domain. Throw out the usless
FOO_ALWAYS_ON_DOMAINS defines and just use POWER_DOMAIN_MASK.
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1460977348-32260-2-git-send-email-ville.syrjala@linux.intel.comReviewed-by: default avatarImre Deak <imre.deak@intel.com>
parent 187a1c07
...@@ -397,11 +397,6 @@ static void hsw_set_power_well(struct drm_i915_private *dev_priv, ...@@ -397,11 +397,6 @@ static void hsw_set_power_well(struct drm_i915_private *dev_priv,
BIT(POWER_DOMAIN_MODESET) | \ BIT(POWER_DOMAIN_MODESET) | \
BIT(POWER_DOMAIN_AUX_A) | \ BIT(POWER_DOMAIN_AUX_A) | \
BIT(POWER_DOMAIN_INIT)) BIT(POWER_DOMAIN_INIT))
#define SKL_DISPLAY_ALWAYS_ON_POWER_DOMAINS ( \
(POWER_DOMAIN_MASK & ~( \
SKL_DISPLAY_POWERWELL_2_POWER_DOMAINS | \
SKL_DISPLAY_DC_OFF_POWER_DOMAINS)) | \
BIT(POWER_DOMAIN_INIT))
#define BXT_DISPLAY_POWERWELL_2_POWER_DOMAINS ( \ #define BXT_DISPLAY_POWERWELL_2_POWER_DOMAINS ( \
BIT(POWER_DOMAIN_TRANSCODER_A) | \ BIT(POWER_DOMAIN_TRANSCODER_A) | \
...@@ -424,10 +419,6 @@ static void hsw_set_power_well(struct drm_i915_private *dev_priv, ...@@ -424,10 +419,6 @@ static void hsw_set_power_well(struct drm_i915_private *dev_priv,
BIT(POWER_DOMAIN_MODESET) | \ BIT(POWER_DOMAIN_MODESET) | \
BIT(POWER_DOMAIN_AUX_A) | \ BIT(POWER_DOMAIN_AUX_A) | \
BIT(POWER_DOMAIN_INIT)) BIT(POWER_DOMAIN_INIT))
#define BXT_DISPLAY_ALWAYS_ON_POWER_DOMAINS ( \
(POWER_DOMAIN_MASK & ~( \
BXT_DISPLAY_POWERWELL_2_POWER_DOMAINS)) | \
BIT(POWER_DOMAIN_INIT))
static void assert_can_enable_dc9(struct drm_i915_private *dev_priv) static void assert_can_enable_dc9(struct drm_i915_private *dev_priv)
{ {
...@@ -1627,7 +1618,6 @@ void intel_display_power_put(struct drm_i915_private *dev_priv, ...@@ -1627,7 +1618,6 @@ void intel_display_power_put(struct drm_i915_private *dev_priv,
(POWER_DOMAIN_MASK & ~BDW_ALWAYS_ON_POWER_DOMAINS) | \ (POWER_DOMAIN_MASK & ~BDW_ALWAYS_ON_POWER_DOMAINS) | \
BIT(POWER_DOMAIN_INIT)) BIT(POWER_DOMAIN_INIT))
#define VLV_ALWAYS_ON_POWER_DOMAINS BIT(POWER_DOMAIN_INIT)
#define VLV_DISPLAY_POWER_DOMAINS POWER_DOMAIN_MASK #define VLV_DISPLAY_POWER_DOMAINS POWER_DOMAIN_MASK
#define VLV_DPIO_CMN_BC_POWER_DOMAINS ( \ #define VLV_DPIO_CMN_BC_POWER_DOMAINS ( \
...@@ -1725,7 +1715,7 @@ static struct i915_power_well hsw_power_wells[] = { ...@@ -1725,7 +1715,7 @@ static struct i915_power_well hsw_power_wells[] = {
{ {
.name = "always-on", .name = "always-on",
.always_on = 1, .always_on = 1,
.domains = HSW_ALWAYS_ON_POWER_DOMAINS, .domains = POWER_DOMAIN_MASK,
.ops = &i9xx_always_on_power_well_ops, .ops = &i9xx_always_on_power_well_ops,
}, },
{ {
...@@ -1739,7 +1729,7 @@ static struct i915_power_well bdw_power_wells[] = { ...@@ -1739,7 +1729,7 @@ static struct i915_power_well bdw_power_wells[] = {
{ {
.name = "always-on", .name = "always-on",
.always_on = 1, .always_on = 1,
.domains = BDW_ALWAYS_ON_POWER_DOMAINS, .domains = POWER_DOMAIN_MASK,
.ops = &i9xx_always_on_power_well_ops, .ops = &i9xx_always_on_power_well_ops,
}, },
{ {
...@@ -1774,7 +1764,7 @@ static struct i915_power_well vlv_power_wells[] = { ...@@ -1774,7 +1764,7 @@ static struct i915_power_well vlv_power_wells[] = {
{ {
.name = "always-on", .name = "always-on",
.always_on = 1, .always_on = 1,
.domains = VLV_ALWAYS_ON_POWER_DOMAINS, .domains = POWER_DOMAIN_MASK,
.ops = &i9xx_always_on_power_well_ops, .ops = &i9xx_always_on_power_well_ops,
.data = PUNIT_POWER_WELL_ALWAYS_ON, .data = PUNIT_POWER_WELL_ALWAYS_ON,
}, },
...@@ -1832,7 +1822,7 @@ static struct i915_power_well chv_power_wells[] = { ...@@ -1832,7 +1822,7 @@ static struct i915_power_well chv_power_wells[] = {
{ {
.name = "always-on", .name = "always-on",
.always_on = 1, .always_on = 1,
.domains = VLV_ALWAYS_ON_POWER_DOMAINS, .domains = POWER_DOMAIN_MASK,
.ops = &i9xx_always_on_power_well_ops, .ops = &i9xx_always_on_power_well_ops,
}, },
{ {
...@@ -1876,7 +1866,7 @@ static struct i915_power_well skl_power_wells[] = { ...@@ -1876,7 +1866,7 @@ static struct i915_power_well skl_power_wells[] = {
{ {
.name = "always-on", .name = "always-on",
.always_on = 1, .always_on = 1,
.domains = SKL_DISPLAY_ALWAYS_ON_POWER_DOMAINS, .domains = POWER_DOMAIN_MASK,
.ops = &i9xx_always_on_power_well_ops, .ops = &i9xx_always_on_power_well_ops,
.data = SKL_DISP_PW_ALWAYS_ON, .data = SKL_DISP_PW_ALWAYS_ON,
}, },
...@@ -1936,7 +1926,7 @@ static struct i915_power_well bxt_power_wells[] = { ...@@ -1936,7 +1926,7 @@ static struct i915_power_well bxt_power_wells[] = {
{ {
.name = "always-on", .name = "always-on",
.always_on = 1, .always_on = 1,
.domains = BXT_DISPLAY_ALWAYS_ON_POWER_DOMAINS, .domains = POWER_DOMAIN_MASK,
.ops = &i9xx_always_on_power_well_ops, .ops = &i9xx_always_on_power_well_ops,
}, },
{ {
......
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