Re: [PATCH 1/2] drm/probe_helper: Add drm_connector_helper_funcs.mode_valid_ctx

2020-07-08 Thread Lyude Paul
On Wed, 2020-07-08 at 21:45 +0300, Imre Deak wrote: > On Wed, Jul 08, 2020 at 01:25:14PM -0400, Lyude Paul wrote: > > JFYI - found an issue with this patch that wouldn't have shown up on i915, > > info > > down below: > > > > On Wed, 2020-07-08 at 01:40 +0300, Imre Deak wrote: > > > Sorry for the

Re: [PATCH 1/2] drm/probe_helper: Add drm_connector_helper_funcs.mode_valid_ctx

2020-07-08 Thread Imre Deak
On Wed, Jul 08, 2020 at 01:25:14PM -0400, Lyude Paul wrote: > > JFYI - found an issue with this patch that wouldn't have shown up on i915, > info > down below: > > On Wed, 2020-07-08 at 01:40 +0300, Imre Deak wrote: > > Sorry for the delay, the review as I promised: > > > > On Tue, May 26, 202

Re: [PATCH 1/2] drm/probe_helper: Add drm_connector_helper_funcs.mode_valid_ctx

2020-07-08 Thread Lyude Paul
JFYI - found an issue with this patch that wouldn't have shown up on i915, info down below: On Wed, 2020-07-08 at 01:40 +0300, Imre Deak wrote: > Sorry for the delay, the review as I promised: > > On Tue, May 26, 2020 at 02:23:09PM -0400, Lyude Paul wrote: > > This is just an atomic version of

Re: [PATCH 1/2] drm/probe_helper: Add drm_connector_helper_funcs.mode_valid_ctx

2020-07-07 Thread Imre Deak
Sorry for the delay, the review as I promised: On Tue, May 26, 2020 at 02:23:09PM -0400, Lyude Paul wrote: > This is just an atomic version of mode_valid, which is intended to be > used for situations where a driver might need to check the atomic state > of objects other than the connector itself.

[PATCH 1/2] drm/probe_helper: Add drm_connector_helper_funcs.mode_valid_ctx

2020-05-26 Thread Lyude Paul
This is just an atomic version of mode_valid, which is intended to be used for situations where a driver might need to check the atomic state of objects other than the connector itself. One such example is with MST, where the maximum possible bandwidth on a connector can change dynamically irregard