[Intel-gfx] [PATCH 14/17] drm/i915: Add debugfs function to check connector status for compliance testing

[Paulo Zanoni]

2014-12-10 21:53 GMT-02:00 Todd Previte <tprevite at gmail.com>:
> Adds a function to check the status of a Displayport connector. This function
> simplifies the Displayport compliance testing interface in debugfs by providing
> a single method for determining the status of a connector during Displayport
> compliance testing. This function checks whether or not the connector is a
> Displayport connector and whether or not that connector is active.
>

I see patches 14 and 15 rewrite some code that was added by patches 7
and 10. Instead of doing this, please make sure patch 7 already
contains your dp_connector_is_valid() function, then patch 10 just
calls it.

Also, please remove those "goto" and just use plain "return".

> Signed-off-by: Todd Previte <tprevite at gmail.com>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 29 +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 5eb6c24..978ddd1 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -3920,6 +3920,35 @@ static int displayport_parse_config(char *input_buffer,
>         return status;
>  }
>
> +static bool dp_connector_is_valid(struct drm_connector *connector,
> +                                 bool check_active)
> +{
> +       struct intel_encoder *intel_encoder;
> +       struct intel_connector *intel_connector;
> +
> +       intel_connector = to_intel_connector(connector);
> +       intel_encoder = intel_connector->encoder;
> +
> +       if (connector->connector_type != DRM_MODE_CONNECTOR_DisplayPort)
> +               goto no;
> +
> +       if (intel_encoder->type != INTEL_OUTPUT_DISPLAYPORT &&
> +           intel_encoder->type != INTEL_OUTPUT_UNKNOWN)
> +               goto no;
> +
> +       if (check_active) {
> +               if (connector->status == connector_status_connected)
> +                       goto yes;
> +               else
> +                       goto no;
> +       }
> +
> +yes:
> +       return true;
> +no:
> +       return false;
> +}
> +
>  static int displayport_config_ctl_show(struct seq_file *m, void *data)
>  {
>         struct drm_device *dev = m->private;
> --
> 1.9.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Paulo Zanoni