Re: [Freedreno] [PATCH v2 26/26] drm/bridge: establish a link between the bridge supplier and consumer

2018-05-16 Thread Daniel Vetter
On Tue, May 15, 2018 at 01:09:59PM +0200, Peter Rosin wrote: > On 2018-05-15 12:22, Daniel Vetter wrote: > > On Mon, May 14, 2018 at 10:40 PM, Peter Rosin wrote: > >> On 2018-05-14 18:28, Daniel Vetter wrote: > >>> On Fri, May 11, 2018 at 09:37:47AM +0200, Peter Rosin wrote: >

[Freedreno] [PATCH v3 24/26] drm/bridge: remove the .of_node member

2018-05-16 Thread Peter Rosin
It is unused. Acked-by: Daniel Vetter Signed-off-by: Peter Rosin --- drivers/gpu/drm/drm_bridge.c | 3 +-- drivers/gpu/drm/rockchip/rockchip_lvds.c | 2 -- include/drm/drm_bridge.h | 4 3 files changed, 1 insertion(+), 8

[Freedreno] [PATCH v3 25/26] drm/bridge: require the owner .odev to be filled in on drm_bridge_add/attach

2018-05-16 Thread Peter Rosin
The .odev owner device will be handy to have around. Reviewed-by: Andrzej Hajda Acked-by: Daniel Vetter Signed-off-by: Peter Rosin --- drivers/gpu/drm/drm_bridge.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

[Freedreno] [PATCH v3 26/26] drm/bridge: establish a link between the bridge supplier and consumer

2018-05-16 Thread Peter Rosin
If the bridge supplier is unbound, this will bring the bridge consumer down along with the bridge. Thus, there will no longer linger any dangling pointers from the bridge consumer (the drm_device) to some non-existent bridge supplier. Reviewed-by: Andrzej Hajda Acked-by:

[Freedreno] [PATCH v3 21/26] drm/msm: specify the owner .odev of the bridges

2018-05-16 Thread Peter Rosin
This will become mandatory. Signed-off-by: Peter Rosin --- drivers/gpu/drm/msm/dsi/dsi_manager.c | 1 + drivers/gpu/drm/msm/edp/edp_bridge.c | 1 + drivers/gpu/drm/msm/hdmi/hdmi_bridge.c | 1 + 3 files changed, 3 insertions(+) diff --git

Re: [Freedreno] [PATCH v6 2/4] dt-bindings: drm/bridge: Document sn65dsi86 bridge bindings

2018-05-16 Thread Stephen Boyd
Quoting Sandeep Panda (2018-05-14 22:52:42) > diff --git > a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.txt > b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.txt > new file mode 100644 > index 000..b82bb56 > --- /dev/null > +++

[Freedreno] [PATCH v3 01/26] drm/bridge: allow optionally specifying an owner .odev device

2018-05-16 Thread Peter Rosin
Bridge drivers can now (temporarily, in a transition phase) select if they want to provide a full owner device or keep just providing an of_node. By providing a full owner device, the bridge drivers no longer need to provide an of_node since that node is available via the owner device. When all

[Freedreno] [PATCH v3 00/26] device link, bridge supplier <-> drm device

2018-05-16 Thread Peter Rosin
Hi! It was noted by Russell King [1] that bridges (not using components) might disappear unexpectedly if the owner of the bridge was unbound. Jyri Sarha had previously noted the same thing with panels [2]. Jyri came up with using device links to resolve the panel issue, which was also my