Commit 8e695444 authored by Imre Deak's avatar Imre Deak Committed by Jani Nikula

drm/i915: add MISSING_CASE to a few port/aux power domain helpers

MISSING_CASE() would have been useful to track down a recent problem in
intel_display_port_aux_power_domain(), so add it there and a few related
helpers. This was also suggested by Ville in his review of the latest
DMC/DC changes, we forgot to address that.
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: default avatarPatrik Jakobsson <patrik.jakobsson@linux.intel.com>
[Cherry-picked from drm-intel-next-queued b9fec167 (Imre)]
Signed-off-by: default avatarImre Deak <imre.deak@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1448643329-18675-5-git-send-email-imre.deak@intel.comSigned-off-by: default avatarJani Nikula <jani.nikula@intel.com>
parent 88747f13
...@@ -5194,7 +5194,7 @@ static enum intel_display_power_domain port_to_power_domain(enum port port) ...@@ -5194,7 +5194,7 @@ static enum intel_display_power_domain port_to_power_domain(enum port port)
case PORT_E: case PORT_E:
return POWER_DOMAIN_PORT_DDI_E_2_LANES; return POWER_DOMAIN_PORT_DDI_E_2_LANES;
default: default:
WARN_ON_ONCE(1); MISSING_CASE(port);
return POWER_DOMAIN_PORT_OTHER; return POWER_DOMAIN_PORT_OTHER;
} }
} }
...@@ -5214,7 +5214,7 @@ static enum intel_display_power_domain port_to_aux_power_domain(enum port port) ...@@ -5214,7 +5214,7 @@ static enum intel_display_power_domain port_to_aux_power_domain(enum port port)
/* FIXME: Check VBT for actual wiring of PORT E */ /* FIXME: Check VBT for actual wiring of PORT E */
return POWER_DOMAIN_AUX_D; return POWER_DOMAIN_AUX_D;
default: default:
WARN_ON_ONCE(1); MISSING_CASE(port);
return POWER_DOMAIN_AUX_A; return POWER_DOMAIN_AUX_A;
} }
} }
...@@ -5275,7 +5275,7 @@ intel_display_port_aux_power_domain(struct intel_encoder *intel_encoder) ...@@ -5275,7 +5275,7 @@ intel_display_port_aux_power_domain(struct intel_encoder *intel_encoder)
intel_dig_port = enc_to_mst(&intel_encoder->base)->primary; intel_dig_port = enc_to_mst(&intel_encoder->base)->primary;
return port_to_aux_power_domain(intel_dig_port->port); return port_to_aux_power_domain(intel_dig_port->port);
default: default:
WARN_ON_ONCE(1); MISSING_CASE(intel_encoder->type);
return POWER_DOMAIN_AUX_A; return POWER_DOMAIN_AUX_A;
} }
} }
......
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