Displaying 2 results from an estimated 2 matches for "unset_edid".
Did you mean:
set_edid
2018 Aug 20
2
[PATCH (repost) 1/5] drm_dp_cec: check that aux has a transfer function
...on was set, so not a DP connector */
> + if (!aux->transfer)
> + return;
> +
> #ifndef CONFIG_MEDIA_CEC_RC
> /*
> * CEC_CAP_RC is part of CEC_CAP_DEFAULTS, but it is stripped by
> @@ -361,6 +369,10 @@ EXPORT_SYMBOL(drm_dp_cec_set_edid);
> */
> void drm_dp_cec_unset_edid(struct drm_dp_aux *aux)
> {
> + /* No transfer function was set, so not a DP connector */
> + if (!aux->transfer)
> + return;
> +
> cancel_delayed_work_sync(&aux->cec.unregister_work);
>
> mutex_lock(&aux->cec.lock);
> @@ -404,6 +416,8 @@ void drm...
2018 Aug 20
0
[PATCH (repost) 1/5] drm_dp_cec: check that aux has a transfer function
...gt; amdgpu? It doesn't make a whole ton of sense to have connectors registering DP
> aux busses if they aren't actually DP, that should probably just be fixed...
The difference between the i915 driver and the nouveau (and amdgpu) driver is
that in the i915 driver the drm_dp_cec_set_edid/unset_edid/irq functions are
called from code that is exclusively for DisplayPort connectors.
For nouveau/amdgpu they are called from code that is shared between DisplayPort
and HDMI, so aux->transfer may be NULL.
Rather than either testing for the connector type or for a non-NULL aux->transfer
every...