[PATCH v5 0/3] SII902X HDMI Bridge fixups

From: Jayesh Choudhary
Date: Thu Jun 13 2024 - 04:40:36 EST


- Move the mode_valid hook to drm_bridge_funcs structure to take care
of the case when the encoder attaches the bridge chain with the
DRM_BRIDGE_ATTACH_NO_CONNECTOR flag in which case, the connector is
not initialized in the bridge's attach call and mode_valid is not
called.

- Also add this check to the atomic_check call as suggested by Maxime in
v1 patch.

- Move from deprecated enable()/disable() hooks to atomic hooks as
suggested in v4 by Sam

Testing has been done on AM62X platform which have SII902X HDMI bridge.
>From the logs we can see the propagated modes and flip test for 1080p
resolution.

Testlog:
<https://gist.github.com/Jayesh2000/9bea6840672869337039296bc1145df4>

Changelog v4->v5:
- Add followup patch to replace deprecated bridge hooks
- Fix atomic check to return negative value
- Made commit message for [1/2] in v4 more clear while keeping the R-by and
Acked-by tags.
- Fix commit header for [2/2] in v4

v4 patch:
<https://lore.kernel.org/all/20240530092930.434026-1-j-choudhary@xxxxxx/>

Changelog v3->v4:
- Remove mode_valid hook from connector_helper_funcs as it is not needed.

v3 patch:
<https://lore.kernel.org/all/20240524093509.127189-1-j-choudhary@xxxxxx/>

Changelog v2->v3:
- Remove newline that was introduced in [1/2] and later deleted in [2/2]
in v2

v2 patch:
<https://lore.kernel.org/all/20240524073305.107293-1-j-choudhary@xxxxxx/>

Changelog v1->v2:
- Add KHZ suffix in the macros to be more clear
- Add the hook for drm_bridge_funcs as well
- Add check in atomic_check dunction call (in a separate patch)

v1 patch:
<https://lore.kernel.org/all/20240408081435.216927-1-j-choudhary@xxxxxx/>

Jayesh Choudhary (3):
drm/bridge: sii902x: Fix mode_valid hook
drm/bridge: sii902x: Support atomic bridge APIs
drm/bridge: sii902x: Add pixel clock check in atomic_check

drivers/gpu/drm/bridge/sii902x.c | 46 +++++++++++++++++++++++---------
1 file changed, 33 insertions(+), 13 deletions(-)

--
2.25.1