| Message ID | 20200911135413.3654800-2-m.tretter@pengutronix.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/exynos: Convert driver to drm bridge | expand |
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. 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
On Fri, Sep 11, 2020 at 03:53:58PM +0200, Michael Tretter 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> Hi Michael. Applied to drm-misc-next, thanks. The rest of this patch-set is exynos stuff, that the maintainer needs to deal with but I will try to take a look at some of the patches. Sam
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 >
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 *
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> --- v2: none --- include/drm/drm_encoder.h | 1 - 1 file changed, 1 deletion(-)