[Intel-gfx] [PATCH v2 3/5] drm/i915/opregion: handle missing connector types for acpi display types

[Jani Nikula]

On Mon, 13 Jun 2016, Jani Nikula <jani.nikula at intel.com> wrote:
> Most notably eDP, DSI, and TV. Add MISSING_CASE handling so we won't
> miss this in the future.
>
> Reviewed-and-tested-by: Peter Wu <peter at lekensteyn.nl>
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>

Pushed up to and including this patch to drm-intel-next-queued, as
they're fairly benign changes. Patches 4 and 5 should probably go in
hand-in-hand.

BR,
Jani.


> ---
>  drivers/gpu/drm/i915/intel_opregion.c | 13 ++++++++++++-
>  1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_opregion.c b/drivers/gpu/drm/i915/intel_opregion.c
> index 108cfbf65931..82e687dd09c3 100644
> --- a/drivers/gpu/drm/i915/intel_opregion.c
> +++ b/drivers/gpu/drm/i915/intel_opregion.c
> @@ -676,7 +676,7 @@ static void set_did(struct intel_opregion *opregion, int i, u32 val)
>  
>  static u32 acpi_display_type(struct drm_connector *connector)
>  {
> -	u32 display_type = ACPI_DISPLAY_TYPE_OTHER;
> +	u32 display_type;
>  
>  	switch (connector->connector_type) {
>  	case DRM_MODE_CONNECTOR_VGA:
> @@ -687,6 +687,7 @@ static u32 acpi_display_type(struct drm_connector *connector)
>  	case DRM_MODE_CONNECTOR_SVIDEO:
>  	case DRM_MODE_CONNECTOR_Component:
>  	case DRM_MODE_CONNECTOR_9PinDIN:
> +	case DRM_MODE_CONNECTOR_TV:
>  		display_type = ACPI_DISPLAY_TYPE_TV;
>  		break;
>  	case DRM_MODE_CONNECTOR_DVII:
> @@ -697,8 +698,18 @@ static u32 acpi_display_type(struct drm_connector *connector)
>  		display_type = ACPI_DISPLAY_TYPE_EXTERNAL_DIGITAL;
>  		break;
>  	case DRM_MODE_CONNECTOR_LVDS:
> +	case DRM_MODE_CONNECTOR_eDP:
> +	case DRM_MODE_CONNECTOR_DSI:
>  		display_type = ACPI_DISPLAY_TYPE_INTERNAL_DIGITAL;
>  		break;
> +	case DRM_MODE_CONNECTOR_Unknown:
> +	case DRM_MODE_CONNECTOR_VIRTUAL:
> +		display_type = ACPI_DISPLAY_TYPE_OTHER;
> +		break;
> +	default:
> +		MISSING_CASE(connector->connector_type);
> +		display_type = ACPI_DISPLAY_TYPE_OTHER;
> +		break;
>  	}
>  
>  	return display_type;

-- 
Jani Nikula, Intel Open Source Technology Center