[PATCH v2 01/16] drm/encoder: remove obsolete documentation of bridge
Michael Tretter
m.tretter at pengutronix.de
Tue Nov 10 08:46:01 UTC 2020
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 at 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 at pengutronix.de>
> > Reviewed-by: Laurent Pinchart <laurent.pinchart at 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 at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
More information about the dri-devel
mailing list