Messages in this thread | | | Subject | Re: [PATCH v4 00/10] Introduce new mode validation callbacks | From | Jose Abreu <> | Date | Mon, 22 May 2017 11:01:43 +0100 |
| |
Hi Daniel,
On 22-05-2017 08:56, Daniel Vetter wrote: > On Fri, May 19, 2017 at 01:52:09AM +0100, Jose Abreu wrote: >> This series is a follow up from the discussion at [1]. We start by >> introducing crtc->mode_valid(), encoder->mode_valid() and >> bridge->mode_valid() callbacks which will be used in followup >> patches and also by cleaning the documentation a little bit. This >> patch is available at [2] and the series depends on it. >> >> We proceed by introducing new helpers to call this new callbacks >> at 1/10. >> >> At 2/10 a helper function is introduced that calls all mode_valid() >> from a set of bridges. >> >> Next, at 3/10 we modify the connector probe helper so that only modes >> which are supported by a given bridge+encoder+crtc combination are >> probbed. >> >> At 4/10 we call all the mode_valid() callbacks for a given pipeline, >> except the connector->mode_valid one, so that the mode is validated. >> This is done before calling mode_fixup(). >> >> Finally, from 5/10 to 10/10 we use the new callbacks in drivers that >> can implement it. >> >> [1] https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.kernel.org_patch_9702233_&d=DwIDAw&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=k-nrzbop60GvJmHyy9k4CfcDFfpSPQEZVA4lpNurIXo&s=rkPEIZmYcjgdVLR6jC_5SMwjNRc_Ye9r4RO95axhtwQ&e= >> [2] https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop.org_archives_dri-2Ddevel_2017-2DMay_141761.html&d=DwIDAw&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=k-nrzbop60GvJmHyy9k4CfcDFfpSPQEZVA4lpNurIXo&s=z5gY8e9Yii5F4HDTa-MQojgFHenigkUt2aKPaGGVQLg&e= >> >> Jose Abreu (10): >> drm: Add drm_{crtc/encoder/connector}_mode_valid() >> drm: Introduce drm_bridge_mode_valid() >> drm: Use new mode_valid() helpers in connector probe helper >> drm: Use mode_valid() in atomic modeset >> drm: arc: Use crtc->mode_valid() callback >> drm/bridge: analogix-anx78xx: Use bridge->mode_valid() callback >> drm/bridge/synopsys: dw-hdmi: Use bridge->mode_valid() callback >> drm/arm: malidp: Use crtc->mode_valid() callback >> drm/atmel-hlcdc: Use crtc->mode_valid() callback >> drm: vc4: Use crtc->mode_valid() and encoder->mode_valid() callbacks > Looks all real nice, I think a bit more time to get acks/reviews/tested-by > for the driver conversions and I'll go and vacuum this all up.
Yeah, I would really like someone to test these, especially the drivers part.
> > Thanks a lot for doing this.
No problem :) Thanks!
Best regards, Jose Miguel Abreu
> -Daniel > >> Cc: Carlos Palminha <palminha@synopsys.com> >> Cc: Alexey Brodkin <abrodkin@synopsys.com> >> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> >> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> >> Cc: Dave Airlie <airlied@linux.ie> >> Cc: Andrzej Hajda <a.hajda@samsung.com> >> Cc: Archit Taneja <architt@codeaurora.org> >> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >> >> drivers/gpu/drm/arc/arcpgu_crtc.c | 29 ++++--- >> drivers/gpu/drm/arm/malidp_crtc.c | 11 ++- >> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 9 +-- >> drivers/gpu/drm/bridge/analogix-anx78xx.c | 13 ++- >> drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 40 +++------- >> drivers/gpu/drm/drm_atomic_helper.c | 76 +++++++++++++++++- >> drivers/gpu/drm/drm_bridge.c | 33 ++++++++ >> drivers/gpu/drm/drm_crtc_helper_internal.h | 13 +++ >> drivers/gpu/drm/drm_probe_helper.c | 105 ++++++++++++++++++++++++- >> drivers/gpu/drm/imx/dw_hdmi-imx.c | 4 +- >> drivers/gpu/drm/meson/meson_dw_hdmi.c | 2 +- >> drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 2 +- >> drivers/gpu/drm/vc4/vc4_crtc.c | 13 ++- >> drivers/gpu/drm/vc4/vc4_dpi.c | 13 ++- >> include/drm/bridge/dw_hdmi.h | 2 +- >> include/drm/drm_bridge.h | 2 + >> 16 files changed, 280 insertions(+), 87 deletions(-) >> >> -- >> 1.9.1 >> >>
| |