On Sat, 07 Nov 2020 09:07:19 -0600, Adam Ford wrote:
On Fri, Sep 11, 2020 at 8:54 AM Michael Tretter m.tretter@pengutronix.de wrote:
In commit 05193dc38197 ("drm/bridge: Make the bridge chain a double-linked list") the bridge has been removed and replaced by a private field. Remove the leftover documentation of the removed field.
Signed-off-by: Michael Tretter m.tretter@pengutronix.de Reviewed-by: Laurent Pinchart laurent.pinchart@ideasonboard.com
What is the status of this series? I know of at least one other patch series depending on this.
Sorry for that. I lately didn't have time to work on the series.
There are two big open TODOs:
- How should this bridge react if there is no out bridge available, yet? This series implements a static approach by returning EPROBE_DEFER from bridge_attach if there isn't a next bridge, connector or display. Andrezej suggested a dynamic approach which allows to attach this bridge without a next bridge and dynamically add further bridges/panels. The latter approach didn't work with the mxsfb driver, but I didn't have time to look into this.
- The component framework stuff that allows to use the bridge with the Exynos driver should not go into the bridge driver, but stay in the platform part.
Michael
adam
v2: none
include/drm/drm_encoder.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/drm/drm_encoder.h b/include/drm/drm_encoder.h index a60f5f1555ac..5dfa5f7a80a7 100644 --- a/include/drm/drm_encoder.h +++ b/include/drm/drm_encoder.h @@ -89,7 +89,6 @@ struct drm_encoder_funcs {
- @head: list management
- @base: base KMS object
- @name: human readable name, can be overwritten by the driver
- @bridge: bridge associated to the encoder
- @funcs: control functions
- @helper_private: mid-layer private data
-- 2.20.1
dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel