Hi Maxime,
On 2025/7/29 15:30, Maxime Ripard wrote:
Hi,
On Thu, Jul 24, 2025 at 04:02:54PM +0800, Damon Ding wrote:
In order to move the parnel/bridge parsing and attachmenet to the
Analogix side, add component struct drm_bridge *bridge to platform
data struct analogix_dp_plat_data.
The movemenet makes sense because the panel/bridge should logically
be positioned behind the Analogix bridge in the display pipeline.
Signed-off-by: Damon Ding <damon.d...@rock-chips.com>
---
include/drm/bridge/analogix_dp.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/drm/bridge/analogix_dp.h b/include/drm/bridge/analogix_dp.h
index cf17646c1310..15cb6b706e9f 100644
--- a/include/drm/bridge/analogix_dp.h
+++ b/include/drm/bridge/analogix_dp.h
@@ -27,6 +27,7 @@ static inline bool is_rockchip(enum analogix_dp_devtype type)
struct analogix_dp_plat_data {
enum analogix_dp_devtype dev_type;
struct drm_panel *panel;
+ struct drm_bridge *bridge;
struct drm_encoder *encoder;
struct drm_connector *connector;
bool skip_connector;
So it's not the analogix_dp bridge, but the bridge after that if any?
You should probably change it to next_bridge to avoid any confusion.
Yes, &analogix_dp_plat_data.bridge is intended to be the next bridge
after analogix_dp bridge.
I will rename it to &analogix_dp_plat_data.next_bridge as suggested and
update it in the next version.
Best regards,
Damon
