diff options
| author | Jani Nikula <jani.nikula@intel.com> | 2015-08-20 10:47:38 +0300 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-08-26 11:00:33 +0200 |
| commit | 196cabd4a39496038ee81c2755496955e7355bd5 (patch) | |
| tree | b05165f318ba6989d6674b1058c659b704861c91 /drivers/gpu/drm/i915/intel_dp.c | |
| parent | 1d245987754f8cd0bf21c77ecda2023ca13ff8c1 (diff) | |
| download | linux-196cabd4a39496038ee81c2755496955e7355bd5.tar.bz2 | |
drm/i915: add MISSING_CASE annotation to ibx_digital_port_connected
With the case added for eDP on port A (always connected from this
function's point of view), we should not be hitting any of the default
cases in ibx_digital_port_connected, so add MISSING_CASE annotation.
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Durgadoss R <durgadoss.r@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_dp.c')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_dp.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index be53e36499b8..8c8070ee26d7 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -4494,6 +4494,8 @@ static bool ibx_digital_port_connected(struct drm_i915_private *dev_priv, if (HAS_PCH_IBX(dev_priv->dev)) { switch (port->port) { + case PORT_A: + return true; case PORT_B: bit = SDE_PORTB_HOTPLUG; break; @@ -4504,10 +4506,13 @@ static bool ibx_digital_port_connected(struct drm_i915_private *dev_priv, bit = SDE_PORTD_HOTPLUG; break; default: - return true; + MISSING_CASE(port->port); + return false; } } else { switch (port->port) { + case PORT_A: + return true; case PORT_B: bit = SDE_PORTB_HOTPLUG_CPT; break; @@ -4518,7 +4523,8 @@ static bool ibx_digital_port_connected(struct drm_i915_private *dev_priv, bit = SDE_PORTD_HOTPLUG_CPT; break; default: - return true; + MISSING_CASE(port->port); + return false; } } |