All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v5 00/33] drm: Implement state readout support
@ 2026-09-07 13:23 Maxime Ripard
  2026-09-07 13:23 ` [PATCH v5 01/33] drm/atomic: Fix unused but set warning in state iterator macros Maxime Ripard
                   ` (34 more replies)
  0 siblings, 35 replies; 41+ messages in thread
From: Maxime Ripard @ 2026-09-07 13:23 UTC (permalink / raw)
  To: Maarten Lankhorst, Thomas Zimmermann, David Airlie, Simona Vetter,
	Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart,
	Jonas Karlman, Jernej Skrabec, Jyri Sarha, Tomi Valkeinen
  Cc: Devarsh Thakkar, dri-devel, Maxime Ripard, Simona Vetter,
	Laurent Pinchart

Hi,

Here's a series that implement what i915 calls "fastboot", ie,
initializing the initial KMS state from the hardware state at boot, to
skip the first modeset if the firmware already set up the display.

This series creates the infrastructure in KMS to create that state by
relying on driver specific hooks. It also implements some infrastructure
to check during non-blocking commits that the readout helpers work
properly by reading out the state that was just committed and comparing
it to what was supposed to be committed.

This relies on another set of driver hooks to compare the entities
states, with helpers providing the default implementation.

It then implements the readout support in the TIDSS driver, and was
tested with the SK-AM62 board. This board in particular is pretty
interesting, since it relies on an DPI to HDMI bridge, and uses the
drm_bridge_connector infrastructure.

So the readout works with the current state of the art on embedded-ish
platforms.

Unlike the earlier RFCs, it now works fine: the state is picked up
properly, doesn't trigger a modeset if what was programmed is the one
the first modeset tries to pick as well, will switch properly if it
isn't, etc.

Let me know what you think,
Maxime

Signed-off-by: Maxime Ripard <mripard@kernel.org>
---
Changes in v5:
- Implement readout for bridge/display-connector
- Implement readout for tidss encoders
- Fix sii902 readout implementation
- Add atomic_print_state implementation to tidss crtc
- Rebase over latest changes. In particular, the CRTC conversion to
  atomic_create_state and the introduction of crtc's hw_reset made me
  create a new helper and hook to trigger a device-wide hw_reset 
- Add a bunch of NULL pointer checks everywhere
- Fix compare macros calls to __stringify
- Create documentation section for sro
- Fix error path in drm_atomic_sro_state_init
- Improve bisectability
- Link to v4: https://lore.kernel.org/r/20260629-drm-state-readout-v4-0-5966657980ed@kernel.org

Changes in v4:
- Dropped the HACK patch that went into the v3 resend
- Hopefully the kernel.org smtp works this time
- Link to v3: https://lore.kernel.org/r/20260626-drm-state-readout-v3-0-d5825e3299a8@kernel.org

Changes in v3:
- Rebase on top of the current drm-misc-next, with rename of
  drm_atomic_state and the various preliminary series merged.
- Use drm_printf_indent when possible
- Change the access mode for the atomic_readout parameter from 000 to
  400
- Use drm_drv_uses_atomic_modeset() instead of drm_core_check_feature()
- Introduce a hw_reset hook to deal with device hardware reset when
  needed (ie, SRO fails or is disabled)
- Improve mode flags readout
- Fix plane alpha readout
- Add sii902x state readout

- Link to v2: https://lore.kernel.org/r/20260423-drm-state-readout-v2-0-8549f87cb978@kernel.org

Changes in v2:
- Get rid of patches already applied
- Rebase on top of the current atomic_create_state work
- Use the SRO prefix everywhere
- Create our own states container structure instead of trying to plumb
  it into a drm_atomic_state
- Make a lot more use of helpers
- Add a hook to enable the hardware resources when the readout state
  becomes active
- Move all the tidss readout code into tidss_dispc.c
- Write documentation
- Add drm_private_obj and drm_bridge name for easier debugging
- Add drm_private_obj_is_bridge()
- Link to v1: https://lore.kernel.org/r/20250902-drm-state-readout-v1-0-14ad5315da3f@kernel.org

---
Maxime Ripard (33):
      drm/atomic: Fix unused but set warning in state iterator macros
      drm/atomic_helper: Skip over NULL private_obj pointers
      drm/atomic: Convert drm_priv_to_bridge_state to container_of_const
      drm/atomic: Add drm_private_obj name
      drm/bridge: Add drm_private_obj_is_bridge()
      drm/bridge: Implement atomic_print_state
      drm/atomic: Export drm_atomic_*_print_state
      drm/atomic: Only call atomic_destroy_state on a !NULL pointer
      drm/atomic_sro: Create drm_atomic_sro_state container
      drm/atomic_sro: Create kernel parameter to force or disable readout
      drm/atomic_sro: Add atomic state readout infrastructure
      drm/atomic_sro: Add function to install state into drm objects
      drm/atomic_sro: Create documentation
      drm/bridge: Handle bridges with hardware state readout
      drm/mode-config: Read out hardware state in drm_mode_config_create_state
      drm/atomic_sro: Provide helpers to implement hardware state readout
      drm/mode-config: Introduce hw_reset hook
      drm/modeset-helper: Add drm_mode_config_helper_hw_reset()
      drm/atomic_helper: Pass nonblock to commit_tail
      drm/atomic_helper: Compare actual and readout states once the commit is done
      drm/atomic-state-helper: Provide comparison macros
      drm/atomic-state-helper: Provide atomic_compare_state helpers
      drm/atomic-helper: Document State Read-Out as a state initialization approach
      drm/encoder: Create atomic_sro_get_current_crtc hook
      drm/bridge: display-connector: Implement readout support
      drm/bridge_connector: Implement hw readout for connector
      drm/tidss: crtc: Add atomic_print_state implementation
      drm/tidss: dispc: Improve mode checking logs
      drm/tidss: Switch to drm_mode_config_create_initial_state()
      drm/tidss: Implement hw_reset
      drm/tidss: Implement readout support
      drm/tidss: encoder: Implement atomic_sro_get_current_crtc
      drm/bridge: sii902x: Implement hw state readout

 Documentation/gpu/drm-kms-helpers.rst              |  26 +
 drivers/gpu/drm/Makefile                           |   2 +
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c  |   1 +
 .../drm/arm/display/komeda/komeda_private_obj.c    |   8 +
 drivers/gpu/drm/bridge/display-connector.c         |  64 ++
 drivers/gpu/drm/bridge/sii902x.c                   |  82 +-
 drivers/gpu/drm/display/drm_bridge_connector.c     |  35 +
 drivers/gpu/drm/display/drm_dp_mst_topology.c      |   2 +-
 drivers/gpu/drm/display/drm_dp_tunnel.c            |   1 +
 drivers/gpu/drm/drm_atomic.c                       |  46 +-
 drivers/gpu/drm/drm_atomic_helper.c                |  46 +-
 drivers/gpu/drm/drm_atomic_sro.c                   | 932 +++++++++++++++++++++
 drivers/gpu/drm/drm_atomic_sro_helper.c            | 679 +++++++++++++++
 drivers/gpu/drm/drm_atomic_state_helper.c          |  20 +-
 drivers/gpu/drm/drm_bridge.c                       | 132 ++-
 drivers/gpu/drm/drm_internal.h                     |  12 +
 drivers/gpu/drm/drm_mode_config.c                  |  55 +-
 drivers/gpu/drm/drm_modeset_helper.c               |  24 +
 drivers/gpu/drm/ingenic/ingenic-drm-drv.c          |   1 +
 drivers/gpu/drm/ingenic/ingenic-ipu.c              |   1 +
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c            |   1 +
 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c           |   1 +
 drivers/gpu/drm/omapdrm/omap_drv.c                 |   1 +
 drivers/gpu/drm/tegra/hub.c                        |   1 +
 drivers/gpu/drm/tidss/tidss_crtc.c                 | 104 +++
 drivers/gpu/drm/tidss/tidss_dispc.c                | 334 +++++++-
 drivers/gpu/drm/tidss/tidss_dispc.h                |  16 +
 drivers/gpu/drm/tidss/tidss_drv.c                  |   6 +-
 drivers/gpu/drm/tidss/tidss_encoder.c              |  57 ++
 drivers/gpu/drm/tidss/tidss_kms.c                  |  20 +-
 drivers/gpu/drm/tidss/tidss_plane.c                | 154 ++++
 drivers/gpu/drm/vc4/vc4_kms.c                      |   3 +
 include/drm/drm_atomic.h                           | 144 +++-
 include/drm/drm_atomic_sro.h                       |  59 ++
 include/drm/drm_atomic_sro_helper.h                | 276 ++++++
 include/drm/drm_bridge.h                           |  87 ++
 include/drm/drm_connector.h                        |  69 ++
 include/drm/drm_crtc.h                             |  69 ++
 include/drm/drm_encoder.h                          |  18 +
 include/drm/drm_mode_config.h                      |  50 ++
 include/drm/drm_modeset_helper.h                   |   1 +
 include/drm/drm_modeset_helper_vtables.h           |  23 +
 include/drm/drm_plane.h                            |  69 ++
 43 files changed, 3644 insertions(+), 88 deletions(-)
---
base-commit: 17c6b88e85ffe5de5720004ae8c2176d52944aca
change-id: 20250730-drm-state-readout-108f089c1c30
prerequisite-change-id: 20260709-drm-no-more-crtc-reset-14a4d5bc8d41:v3
prerequisite-patch-id: 58c15cfdaac5894112d6b55152cee55444e59c2c
prerequisite-patch-id: f1d676df498396acf15592ca7ead5be61138f47a
prerequisite-patch-id: 0b4247d6f5e7d16522473260c92ad1b4a235e5d6
prerequisite-patch-id: 454462ebf2b4cc237d83aa8b3631252e158d16d0
prerequisite-patch-id: ffa6afc89be0c75a3b3565d204e998841fcd7c15
prerequisite-patch-id: c20225c4fc6139ca606ca94d44d973b22d94ad94
prerequisite-patch-id: a0b461fed6e896d3ea179caff28a00d912aee47a
prerequisite-patch-id: ae43b98d635078d5d2d76a9dee7dffba082c3dd4
prerequisite-patch-id: 91c7bc55447b4c4fc97000bc257a72d1694d8aa6
prerequisite-patch-id: 527b8258f3d80c3dafc1833732f66d7283e7bbb5
prerequisite-patch-id: 9ef306a7e4c68ac38a6ed26c0f18b02dd9e3beeb
prerequisite-patch-id: 576654f19b1d56754bdb115f673438140f242fda
prerequisite-patch-id: 166fb879fa7744d42ee56d6735cc344585a74b2a

Best regards,
-- 
Maxime Ripard <mripard@kernel.org>


^ permalink raw reply	[flat|nested] 41+ messages in thread

end of thread, other threads:[~2026-09-15 14:14 UTC | newest]

Thread overview: 41+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-07 13:23 [PATCH v5 00/33] drm: Implement state readout support Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 01/33] drm/atomic: Fix unused but set warning in state iterator macros Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 02/33] drm/atomic_helper: Skip over NULL private_obj pointers Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 03/33] drm/atomic: Convert drm_priv_to_bridge_state to container_of_const Maxime Ripard
2026-09-15  9:29   ` Javier Martinez Canillas
2026-09-15 14:14   ` (subset) " Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 04/33] drm/atomic: Add drm_private_obj name Maxime Ripard
2026-09-15 14:13   ` Laurent Pinchart
2026-09-07 13:23 ` [PATCH v5 05/33] drm/bridge: Add drm_private_obj_is_bridge() Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 06/33] drm/bridge: Implement atomic_print_state Maxime Ripard
2026-09-15  9:26   ` Geert Uytterhoeven
2026-09-15 10:08     ` Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 07/33] drm/atomic: Export drm_atomic_*_print_state Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 08/33] drm/atomic: Only call atomic_destroy_state on a !NULL pointer Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 09/33] drm/atomic_sro: Create drm_atomic_sro_state container Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 10/33] drm/atomic_sro: Create kernel parameter to force or disable readout Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 11/33] drm/atomic_sro: Add atomic state readout infrastructure Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 12/33] drm/atomic_sro: Add function to install state into drm objects Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 13/33] drm/atomic_sro: Create documentation Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 14/33] drm/bridge: Handle bridges with hardware state readout Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 15/33] drm/mode-config: Read out hardware state in drm_mode_config_create_state Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 16/33] drm/atomic_sro: Provide helpers to implement hardware state readout Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 17/33] drm/mode-config: Introduce hw_reset hook Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 18/33] drm/modeset-helper: Add drm_mode_config_helper_hw_reset() Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 19/33] drm/atomic_helper: Pass nonblock to commit_tail Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 20/33] drm/atomic_helper: Compare actual and readout states once the commit is done Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 21/33] drm/atomic-state-helper: Provide comparison macros Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 22/33] drm/atomic-state-helper: Provide atomic_compare_state helpers Maxime Ripard
2026-09-07 13:23 ` [PATCH v5 23/33] drm/atomic-helper: Document State Read-Out as a state initialization approach Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 24/33] drm/encoder: Create atomic_sro_get_current_crtc hook Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 25/33] drm/bridge: display-connector: Implement readout support Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 26/33] drm/bridge_connector: Implement hw readout for connector Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 27/33] drm/tidss: crtc: Add atomic_print_state implementation Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 28/33] drm/tidss: dispc: Improve mode checking logs Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 29/33] drm/tidss: Switch to drm_mode_config_create_initial_state() Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 30/33] drm/tidss: Implement hw_reset Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 31/33] drm/tidss: Implement readout support Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 32/33] drm/tidss: encoder: Implement atomic_sro_get_current_crtc Maxime Ripard
2026-09-07 13:24 ` [PATCH v5 33/33] drm/bridge: sii902x: Implement hw state readout Maxime Ripard
2026-09-15  8:06 ` [PATCH v5 00/33] drm: Implement state readout support Maxime Ripard
2026-09-15 12:34 ` Tomi Valkeinen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.