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 <[email protected]> --- 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/[email protected] 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/[email protected] 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/[email protected] 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/[email protected] --- 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 <[email protected]>
