Skip to content

drm/bridge: ps8640 and ti-sn65dsi86 updates

Sam Ravnborg dri-devel llvm oe-kbuild-all 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest9 patches 20 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9drm/bridge: add DRM_BRIDGE_STATE_OPS macroinclude/drm/drm_bridge.h+12 -03 replies
2/9drm: add drm specific media-bus-format header fileinclude/drm/media-bus-format.h+53 -02 replies
3/9drm: add drm_atomic_helper_bridge_dsi_input_bus_fmtdrivers/gpu/drm/drm_atomic_helper.c include/drm/drm_atomic_helper.h+48 -03 replies
4/9drm/bridge: ti-sn65dsi86: Use atomic variants of drm_bridge_funcsdrivers/gpu/drm/bridge/ti-sn65dsi86.c+13 -82 replies
5/9drm/bridge: ti-sn65dsi86: Fetch bpc via drm_bridge_statedrivers/gpu/drm/bridge/ti-sn65dsi86.c+10 -62 replies
6/9drm/bridge: ti-sn65dsi86: Add NO_CONNECTOR supportdrivers/gpu/drm/bridge/ti-sn65dsi86.c+9 -114 replies
7/9drm/bridge: ps8640: Use atomic variants of drm_bridge_funcsdrivers/gpu/drm/bridge/parade-ps8640.c+10 -60 replies
8/9drm/bridge: ps8640: plug atomic_get_input_bus_fmtsdrivers/gpu/drm/bridge/parade-ps8640.c+2 -00 replies
9/9drm/bridge: Drop unused drm_bridge_chain functionsdrivers/gpu/drm/drm_bridge.c include/drm/drm_bridge.h+0 -1381 reply

Reviews trailers per patch

5 of 9 patches have a Reviewed-by or Acked-by.

PatchMaxime RipardDouglas AndersonLaurent Pinchart
1/9R-b R-b
2/9
3/9
4/9R-b
5/9
6/9R-b R-b
7/9R-b
8/9
9/9R-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH v1 0/9] drm/bridge: ps8640 and ti-sn65dsi86 updates

Add a new macro DRM_BRIDGE_STATE_OPS that can be used to assign atomic_reset and atomic_{duplicate,destroy}_state to the default implementations. They will be valid in most cases.

Add a drm variant of media-bus-format.h to hold a single function to get the bpc from the bus format.

Also add a small drm_atomic_helper_bridge_dsi_input_bus_fmt helper.

Update ti-sn65dsi86 to support atomic and NO_CONNECTOR. The NO_CONNECTOR support was from Rob Clark - I just rebased it. To support NO_CONNECTOR use the newly introduced function to lokk up bpc from the bus format.

Update parade-ps8640 to support atomic. To do this just migrate to the atomic variants of the operations and add the few mandatry missing callbacks.

A few of the patches are migrated from a patchset I posted several months ago and I decided to add them here for now, which explains why there is a v4 of a patch in a v1 submission.

For the output bus fmt stuff I did what I think is correct - but as I have paged out all my memory of this it may be all wrong.

The code builds - but needs testing.

I was temped to move bridge helpers to a new drm_bridge_helper.c but that will wait until next time.

Sam

Rob Clark (1):
drm/bridge: ti-sn65dsi86: Add NO_CONNECTOR support

Sam Ravnborg (8):
drm/bridge: add DRM_BRIDGE_STATE_OPS macro
drm: add drm specific media-bus-format header file
drm: add drm_atomic_helper_bridge_dsi_input_bus_fmt drm/bridge: ti-sn65dsi86: Use atomic variants of drm_bridge_funcs drm/bridge: ti-sn65dsi86: Fetch bpc via drm_bridge_state drm/bridge: ps8640: Use atomic variants of drm_bridge_funcs drm/bridge: ps8640: plug atomic_get_input_bus_fmts drm/bridge: Drop unused drm_bridge_chain functions

 drivers/gpu/drm/bridge/parade-ps8640.c |  18 ++++--
 drivers/gpu/drm/bridge/ti-sn65dsi86.c  |  57 +++++++++--------
 drivers/gpu/drm/drm_atomic_helper.c    |  41 ++++++++++++
 drivers/gpu/drm/drm_bridge.c           | 110 ---------------------------------
 include/drm/drm_atomic_helper.h        |   7 +++
 include/drm/drm_bridge.h               |  40 ++++--------
 include/drm/media-bus-format.h         |  53 ++++++++++++++++
 7 files changed, 157 insertions(+), 169 deletions(-)