Hi Laurent/all. On Sun, Jul 26, 2020 at 10:33:09PM +0200, Sam Ravnborg wrote: > The objective is that all bridge drivers shall support a chained setup > connector creation is moved to the display drivers. > This is just one step on this path. > > The general approach for the bridge drivers: > - Introduce bridge operations > - Introduce use of panel bridge and make connector creation optional > > v4: > - Dropped patch for ti-sn65dsi86. Await full conversion. > - Dropped patch for ti-tpd12s015. It was wrong (Laurent) > - Drop boe,hv070wsa-100 patch, it was applied > - Combined a few patches to fix connector created twice (Laurent) > - Fix memory leak in get_edid in several drivers (Laurent) > - Added patch to validate panel descriptions in panel-simple > - Set bridge.type in relevant drivers > > v3: > - Rebase on top of drm-misc-next > - Address kbuild test robot feedback > > v2: > - Updated bus_flags for boe,hv070wsa-100 > - Collected r-b, but did not apply patches yet > - On the panel side the panel-simple driver gained a default > connector type for all the dumb panels that do not > include so in their description. > With this change panels always provide a connector type, > and we have the potential to drop most uses of > devm_drm_panel_bridge_add_typed(). > - Added conversion of a few more bridge drivers > > Patches can build but no run-time testing. > So both test and review feedback appreciated! > > Sam > > Sam Ravnborg (15): > drm/panel: panel-simple: validate panel description > drm/panel: panel-simple: add default connector_type > drm/bridge: tc358764: drop drm_connector_(un)register > drm/bridge: tc358764: add drm_panel_bridge support > drm/bridge: tc358767: add detect bridge operation > drm/bridge: tc358767: add get_edid bridge operation > drm/bridge: tc358767: add drm_panel_bridge support > drm/bridge: parade-ps8622: add drm_panel_bridge support > drm/bridge: megachips: add helper to create connector > drm/bridge: megachips: get drm_device from bridge > drm/bridge: megachips: enable detect bridge operation > drm/bridge: megachips: add get_edid bridge operation > drm/bridge: megachips: make connector creation optional > drm/bridge: nxp-ptn3460: add get_edid bridge operation > drm/bridge: nxp-ptn3460: add drm_panel_bridge support Laurent reviewed the full series - thanks! I went ahead and applied the patches for drivers where all patches was reviewed. I will send a v5 soon for tc358767 and nxp-ptn3460 where I have fixed my brown paper bag bugs . I am very happy Laurent spotted these before we applied the patches. This also gives a good indication of the quality of the review. Sam > > .../drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 97 +++++++++------- > drivers/gpu/drm/bridge/nxp-ptn3460.c | 103 ++++++++--------- > drivers/gpu/drm/bridge/parade-ps8622.c | 100 +++------------- > drivers/gpu/drm/bridge/tc358764.c | 110 +++--------------- > drivers/gpu/drm/bridge/tc358767.c | 126 +++++++++++---------- > drivers/gpu/drm/panel/panel-simple.c | 48 +++++++- > 6 files changed, 242 insertions(+), 342 deletions(-) > > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
