Re: [PATCH 1/7] drm/bridge: analogix_dp: drop extra calls to analogix_dp_prepare_panel()

2025-04-13 Thread Damon Ding
Hi Dmitry, On 2025/4/1 13:11, Dmitry Baryshkov wrote: The analogix_dp_prepare_panel() returns immediately if there is no attached panel. Drop several calls to this function which are performed when dp->plat_data->panel is NULL. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/bridge/analo

Re: [PATCH 1/7] drm/bridge: analogix_dp: drop extra calls to analogix_dp_prepare_panel()

2025-04-13 Thread Damon Ding
On 2025/4/14 9:20, Damon Ding wrote: Hi Dmitry, On 2025/4/1 13:11, Dmitry Baryshkov wrote: The analogix_dp_prepare_panel() returns immediately if there is no attached panel. Drop several calls to this function which are performed when dp->plat_data->panel is NULL. Signed-off-by:

Re: [PATCH 6/7] drm/bridge: analogix_dp: ignore return values of drm_panel_* calls

2025-04-13 Thread Damon Ding
DRM_ERROR("failed to turnoff the panel\n"); - } + drm_panel_unprepare(dp->plat_data->panel); drm_dp_aux_unregister(&dp->aux); The patch has been verified with the eDP panel "lg,lp079qx1-sp0v" in RK3588S EVB1 board. Tested-by: Damon Ding Best regards, Damon

Re: [PATCH 5/7] drm/bridge: analogix_dp: inline analogix_dp_prepare_panel()

2025-04-13 Thread Damon Ding
} dp->fast_train_enable = false; dp->psr_supported = false; The patch has been verified with the eDP panel "lg,lp079qx1-sp0v" in RK3588S EVB1 board. Tested-by: Damon Ding Best regards, Damon

Re: [PATCH 4/7] drm/bridge: analogic_dp: drop panel_lock

2025-04-13 Thread Damon Ding
utex panel_lock; - struct analogix_dp_plat_data *plat_data; }; The patch has been verified with the eDP panel "lg,lp079qx1-sp0v" in RK3588S EVB1 board. Tested-by: Damon Ding Best regards, Damon

Re: [PATCH 3/7] drm/bridge: analogic_dp: drop panel_is_modeset

2025-04-13 Thread Damon Ding
panel_is_modeset; struct analogix_dp_plat_data *plat_data; }; The patch has been verified with the eDP panel "lg,lp079qx1-sp0v" in RK3588S EVB1 board. Tested-by: Damon Ding Best regards, Damon

Re: [PATCH 2/7] drm/bridge: analogix_dp: drop unused argument to analogix_dp_prepare_panel()

2025-04-13 Thread Damon Ding
lse); if (ret) DRM_ERROR("failed to setup the panel ret = %d\n", ret); The patch has been verified with the eDP panel "lg,lp079qx1-sp0v" in RK3588S EVB1 board. Tested-by: Damon Ding Best regards, Damon

[PATCH v8 01/13] drm/bridge: analogix_dp: Add irq flag IRQF_NO_AUTOEN instead of calling disable_irq()

2025-03-15 Thread Damon Ding
flag. Suggested-by: Douglas Anderson Reviewed-by: Douglas Anderson Signed-off-by: Damon Ding --- drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/ana

[PATCH v8 06/13] drm/bridge: analogix_dp: Add support to get panel from the DP AUX bus

2025-03-11 Thread Damon Ding
-by: Damon Ding --- Changes in v4: - Use done_probing() to call drm_of_find_panel_or_bridge() and component_add() when getting panel from the DP AUX bus Changes in v5: - Advance PM operations to make eDP AUX work well Changes in v6: - Use devm_pm_runtime_enable() instead of

[PATCH v8 08/13] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-03-11 Thread Damon Ding
get the panel from the DP AUX bus, it will then try the other way to get panel information through the platform bus. Reviewed-by: Dmitry Baryshkov Signed-off-by: Damon Ding --- Changes in v4: - Use done_probing() to call drm_of_find_panel_or_bridge() and component_add() when getting panel from

Re: [PATCH v7 10/15] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-03-11 Thread Damon Ding
Hi Doug, On 2025/2/25 9:42, Doug Anderson wrote: Hi, On Mon, Feb 24, 2025 at 12:14 AM Damon Ding wrote: @@ -392,11 +393,27 @@ static const struct component_ops rockchip_dp_component_ops = { .unbind = rockchip_dp_unbind, }; +static int rockchip_dp_link_panel(struct drm_dp_aux

[PATCH v8 00/13] Add eDP support for RK3588

2025-03-11 Thread Damon Ding
enable the eDP0 display on RK3588S EVB1 board. Damon Ding (13): drm/bridge: analogix_dp: Add irq flag IRQF_NO_AUTOEN instead of calling disable_irq() drm/bridge: analogix_dp: Remove CONFIG_PM related check in analogix_dp_bind()/analogix_dp_unbind() drm/bridge: analogix_dp: Add

[PATCH v8 12/13] arm64: dts: rockchip: Add eDP0 node for RK3588

2025-03-10 Thread Damon Ding
Add support for the eDP0 output on RK3588 SoC. Signed-off-by: Damon Ding --- Changes in v3: - Remove currently unsupported property '#sound-dai-cells' Changes in v4: - Remove currently unsupported clock 'spdif' --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi | 28 ++

dri-devel@lists.freedesktop.org

2025-03-10 Thread Damon Ding
igned-off-by: Damon Ding --- Changes in v7: - Use the forward-declare struct drm_dp_aux instead of including the relevant header file. --- drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 14 ++ include/drm/bridge/analogix_dp.h | 4 2 files changed, 1

[PATCH v8 09/13] dt-bindings: display: rockchip: analogix-dp: Add support for RK3588

2025-03-10 Thread Damon Ding
eviewed-by: Rob Herring (Arm) Signed-off-by: Damon Ding --- Changes in v2: - Add the main defferences of the RK3588 eDP and the previous versions in commit message Changes in v3: - Expand the property clock-names, resets and reset-names Changes in v4: - Remove 'spdif' clock which a

[PATCH v8 11/13] drm/rockchip: analogix_dp: Add support for RK3588

2025-03-10 Thread Damon Ding
#x27;. And the newly added 'apb' reset control is to ensure the APB bus of eDP controller works well on the RK3588 SoC. Acked-by: Dmitry Baryshkov Signed-off-by: Damon Ding --- Changes in v2: - Add support for the other eDP output edp1 Changes in v3: - Fix the unexpected use of alia

[PATCH v8 13/13] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board

2025-03-10 Thread Damon Ding
202.02MHz. Additionally, the 'force-hpd' is set for edp0 because the HPD pin on the panel side is not connected to the eDP HPD pin on the SoC side according to the RK3588S EVB1 hardware design. Signed-off-by: Damon Ding --- Changes in v2: - Remove brightness-levels and default-brightness

[PATCH v8 10/13] drm/bridge: analogix_dp: Add support for RK3588

2025-03-10 Thread Damon Ding
Expand enum analogix_dp_devtype with RK3588_EDP, and add max_link_rate and max_lane_count configs for it. Reviewed-by: Dmitry Baryshkov Signed-off-by: Damon Ding --- Changes in v5: - Add the RK3588_EDP related modification in analogix_dp.h - Move this commit above related commit on the

dri-devel@lists.freedesktop.org

2025-03-10 Thread Damon Ding
Add analogix_dpaux_wait_hpd_asserted() to help confirm the HPD state before doing AUX transfers. Signed-off-by: Damon Ding --- .../drm/bridge/analogix/analogix_dp_core.c| 21 +++ 1 file changed, 21 insertions(+) diff --git a/drivers/gpu/drm/bridge/analogix

[PATCH v8 04/13] dt-bindings: display: rockchip: analogix-dp: Add support to get panel from the DP AUX bus

2025-03-10 Thread Damon Ding
According to Documentation/devicetree/bindings/display/dp-aux-bus.yaml, it is a good way to get panel through the DP AUX bus. Acked-by: Krzysztof Kozlowski Signed-off-by: Damon Ding --- Changes in v4: - Move the dt-bindings commit before related driver commits Changes in v5: - Remove the

[PATCH v8 03/13] drm/bridge: analogix_dp: Add support for phy configuration.

2025-03-10 Thread Damon Ding
Add support to configurate link rate, lane count, voltage swing and pre-emphasis with phy_configure(). It is helpful in application scenarios where analogix controller is mixed with the phy of other vendors. Reviewed-by: Dmitry Baryshkov Signed-off-by: Damon Ding --- Changes in v2: - remove

[PATCH v8 02/13] drm/bridge: analogix_dp: Remove CONFIG_PM related check in analogix_dp_bind()/analogix_dp_unbind()

2025-03-10 Thread Damon Ding
-by: Douglas Anderson Reviewed-by: Douglas Anderson Signed-off-by: Damon Ding --- .../drm/bridge/analogix/analogix_dp_core.c| 30 --- 1 file changed, 6 insertions(+), 24 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge

Re: [PATCH v7 09/15] drm/bridge: analogix_dp: Add support to get panel from the DP AUX bus

2025-03-02 Thread Damon Ding
Hi Doug, On 2025/2/25 9:41, Doug Anderson wrote: Hi, On Mon, Feb 24, 2025 at 12:14 AM Damon Ding wrote: The main modification is moving the DP AUX initialization from function analogix_dp_bind() to analogix_dp_probe(). In order to get the EDID of eDP panel during probing, it is also needed

[PATCH v8] drm/bridge: analogix_dp: Remove the unnecessary calls to clk_disable_unprepare() during probing

2025-03-02 Thread Damon Ding
ng probing. Fixes: f37952339cc2 ("drm/bridge: analogix_dp: handle clock via runtime PM") Suggested-by: Douglas Anderson Reviewed-by: Douglas Anderson Signed-off-by: Damon Ding --- Picked from: https://patchwork.kernel.org/project/linux-rockchip/list/?series=936932 Changes in v8: - Fix

Re: [PATCH v7 04/15] drm/bridge: analogix_dp: Remove the unnecessary calls to clk_disable_unprepare() during probing

2025-03-01 Thread Damon Ding
Hi Doug, On 2025/2/25 9:40, Doug Anderson wrote: Hi, On Mon, Feb 24, 2025 at 12:14 AM Damon Ding wrote: With the commit f37952339cc2 ("drm/bridge: analogix_dp: handle clock via runtime PM"), the PM operations can help enable/disable the clock. The err_disable_clk

dri-devel@lists.freedesktop.org

2025-02-24 Thread Damon Ding
Add two new functions: one to find &analogix_dp_device.plat_data via &drm_dp_aux, and the other to get &analogix_dp_device.aux. Both of them serve for the function of getting panel from DP AUX bus, which is why they are included in a single commit. Signed-off-by: Damon Ding Reviewe

[PATCH v7 02/15] drm/rockchip: analogix_dp: Expand device data to support multiple edp display

2025-02-24 Thread Damon Ding
There are two main modifications: one is expanding struct rockchip_dp_chip_data to an array, and the other is adding &rockchip_dp_chip_data.reg to separate different edp devices. Signed-off-by: Damon Ding --- Changes in v6: - Add the description of &rockchip_dp_chip_data.reg -

[PATCH v7 04/15] drm/bridge: analogix_dp: Remove the unnecessary calls to clk_disable_unprepare() during probing

2025-02-24 Thread Damon Ding
ng probing. Fixes: f37952339cc2 ("drm/bridge: analogix_dp: handle clock via runtime PM") Signed-off-by: Damon Ding --- .../gpu/drm/bridge/analogix/analogix_dp_core.c | 18 +- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/a

[PATCH v7 05/15] drm/bridge: analogix_dp: Remove CONFIG_PM related check in analogix_dp_bind()/analogix_dp_unbind()

2025-02-24 Thread Damon Ding
-by: Damon Ding --- .../drm/bridge/analogix/analogix_dp_core.c| 30 --- 1 file changed, 6 insertions(+), 24 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c index d9dafb038e7a..174e7011bba9

[PATCH v7 09/15] drm/bridge: analogix_dp: Add support to get panel from the DP AUX bus

2025-02-24 Thread Damon Ding
, add support for &drm_dp_aux.wait_hpd_asserted() to help confirm the HPD state before doing AUX transfers. Signed-off-by: Damon Ding --- Changes in v4: - Use done_probing() to call drm_of_find_panel_or_bridge() and component_add() when getting panel from the DP AUX bus Changes i

[PATCH v7 00/15] Add eDP support for RK3588

2025-02-24 Thread Damon Ding
. Patch 15is to enable the eDP0 display on RK3588S EVB1 board. Damon Ding (15): drm/rockchip: analogix_dp: Use formalized struct definition for grf field drm/rockchip: analogix_dp: Expand device data to support multiple edp display drm/bridge: analogix_dp: Add irq flag

[PATCH v7 01/15] drm/rockchip: analogix_dp: Use formalized struct definition for grf field

2025-02-24 Thread Damon Ding
The formalized struct definition will makes grf field operations more concise and easier to extend. Signed-off-by: Damon Ding --- Changes in v2: - Initialize struct rockchip_dp_chip_data rk3399_edp/rk3288_dp in order of its members Changes in v6: - Pass 'dp' in drm_...() rathe

[PATCH v7 03/15] drm/bridge: analogix_dp: Add irq flag IRQF_NO_AUTOEN instead of calling disable_irq()

2025-02-24 Thread Damon Ding
igned-off-by: Damon Ding --- drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c index bfa88409a7ff..e23af674d91c 100644

[PATCH v7 06/15] drm/bridge: analogix_dp: Add support for phy configuration.

2025-02-24 Thread Damon Ding
Add support to configurate link rate, lane count, voltage swing and pre-emphasis with phy_configure(). It is helpful in application scenarios where analogix controller is mixed with the phy of other vendors. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- Changes in v2: - remove

[PATCH v7 10/15] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-02-24 Thread Damon Ding
get the panel from the DP AUX bus, it will then try the other way to get panel information through the platform bus. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- Changes in v4: - Use done_probing() to call drm_of_find_panel_or_bridge() and component_add() when getting panel from

Re: [PATCH v6 00/14] Add eDP support for RK3588

2025-02-24 Thread Damon Ding
Hi Piotr, On 2025/2/14 20:00, Piotr Oniszczuk wrote: Wiadomość napisana przez Sebastian Reichel w dniu 13 lut 2025, o godz. 18:26: Hello Piotr, The RK3588 has two different DP controllers. The one handled in this series should get the eDP port running, which is routed to the "eDP Panel"

[PATCH v7 07/15] dt-bindings: display: rockchip: analogix-dp: Add support to get panel from the DP AUX bus

2025-02-24 Thread Damon Ding
According to Documentation/devicetree/bindings/display/dp-aux-bus.yaml, it is a good way to get panel through the DP AUX bus. Acked-by: Krzysztof Kozlowski Signed-off-by: Damon Ding --- Changes in v4: - Move the dt-bindings commit before related driver commits Changes in v5: - Remove the

[PATCH v7 15/15] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board

2025-02-24 Thread Damon Ding
202.02MHz. Additionally, the 'force-hpd' is set for edp0 because the HPD pin on the panel side is not connected to the eDP HPD pin on the SoC side according to the RK3588S EVB1 hardware design. Signed-off-by: Damon Ding --- Changes in v2: - Remove brightness-levels and default-brightness

[PATCH v7 12/15] drm/bridge: analogix_dp: Add support for RK3588

2025-02-24 Thread Damon Ding
Expand enum analogix_dp_devtype with RK3588_EDP, and add max_link_rate and max_lane_count configs for it. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- Changes in v5: - Add the RK3588_EDP related modification in analogix_dp.h - Move this commit above related commit on the

[PATCH v7 11/15] dt-bindings: display: rockchip: analogix-dp: Add support for RK3588

2025-02-24 Thread Damon Ding
ned-off-by: Damon Ding --- Changes in v2: - Add the main defferences of the RK3588 eDP and the previous versions in commit message Changes in v3: - Expand the property clock-names, resets and reset-names Changes in v4: - Remove 'spdif' clock which added in v3 - Add the comment

[PATCH v7 14/15] arm64: dts: rockchip: Add eDP0 node for RK3588

2025-02-24 Thread Damon Ding
Add support for the eDP0 output on RK3588 SoC. Signed-off-by: Damon Ding --- Changes in v3: - Remove currently unsupported property '#sound-dai-cells' Changes in v4: - Remove currently unsupported clock 'spdif' --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi | 28 ++

[PATCH v7 13/15] drm/rockchip: analogix_dp: Add support for RK3588

2025-02-24 Thread Damon Ding
#x27;. And the newly added 'apb' reset control is to ensure the APB bus of eDP controller works well on the RK3588 SoC. Signed-off-by: Damon Ding Acked-by: Dmitry Baryshkov --- Changes in v2: - Add support for the other eDP output edp1 Changes in v3: - Fix the unexpected use of alia

Re: [PATCH v6 08/14] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-02-22 Thread Damon Ding
Hi Heiko, On 2025/1/31 4:33, Heiko Stübner wrote: Hi Damon, Am Donnerstag, 23. Januar 2025, 11:07:41 MEZ schrieb Damon Ding: Move drm_of_find_panel_or_bridge() a little later and combine it with component_add() into a new function rockchip_dp_link_panel(). The function will serve as

Re: [PATCH v6 08/14] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-02-22 Thread Damon Ding
Hi Lucas, On 2025/2/8 3:42, Lucas Stach wrote: Hi Damon, Am Donnerstag, dem 23.01.2025 um 18:07 +0800 schrieb Damon Ding: Move drm_of_find_panel_or_bridge() a little later and combine it with component_add() into a new function rockchip_dp_link_panel(). The function will serve as done_probing

Re: [PATCH v6 08/14] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-02-22 Thread Damon Ding
On 2025/2/22 19:25, Damon Ding wrote: Hi Heiko, On 2025/1/31 4:33, Heiko Stübner wrote: Hi Damon, Am Donnerstag, 23. Januar 2025, 11:07:41 MEZ schrieb Damon Ding: Move drm_of_find_panel_or_bridge() a little later and combine it with component_add() into a new function rockchip_dp_link_panel

Re: [PATCH v6 07/14] drm/bridge: analogix_dp: Add support to get panel from the DP AUX bus

2025-02-16 Thread Damon Ding
Hi Doug, On 2025/1/24 11:13, Doug Anderson wrote: Hi, On Thu, Jan 23, 2025 at 3:25 AM Dmitry Baryshkov wrote: On Thu, Jan 23, 2025 at 06:07:40PM +0800, Damon Ding wrote: The main modification is moving the DP AUX initialization from function analogix_dp_bind() to analogix_dp_probe(). In

[PATCH v7 0/2] Fix label name of hdptxphy for RK3588

2025-02-05 Thread Damon Ding
. Damon Ding (2): dt-bindings: display: rockchip: Fix label name of hdptxphy for RK3588 HDMI TX Controller arm64: dts: rockchip: Fix label name of hdptxphy for RK3588 .../bindings/display/rockchip/rockchip,rk3588-dw-hdmi-qp.yaml | 2 +- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi

[PATCH v7 2/2] arm64: dts: rockchip: Fix label name of hdptxphy for RK3588

2025-02-05 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Signed-off-by: Damon Ding --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi

[PATCH v7 1/2] dt-bindings: display: rockchip: Fix label name of hdptxphy for RK3588 HDMI TX Controller

2025-02-05 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Acked-by: Rob Herring (Arm) Signed-off-by: Damon Ding --- .../bindings/display

[PATCH v7] drm/edp-panel: Add LG Display panel model LP079QX1-SP0V

2025-02-05 Thread Damon Ding
00 00 00 00 00 00 00 00 00 00 fe 00 4c 50 30 37 39 51 58 31 2d 53 50 30 56 00 00 00 fc 00 43 6f 6c 6f 72 20 4c 43 44 0a 20 20 20 00 3f Signed-off-by: Damon Ding --- Picked from: https://patchwork.kernel.org/project/linux-rockchip/list/?series=927765 changes in v7: - Add &panel_delay.disable

Re: [PATCH v6 12/14] drm/edp-panel: Add LG Display panel model LP079QX1-SP0V

2025-01-24 Thread Damon Ding
Hi Doug, On 2025/1/24 8:30, Doug Anderson wrote: Hi, On Thu, Jan 23, 2025 at 3:31 AM Dmitry Baryshkov wrote: On Thu, Jan 23, 2025 at 06:07:45PM +0800, Damon Ding wrote: The raw edid for LP079QX1-SP0V panel model is: 00 ff ff ff ff ff ff 00 16 83 00 00 00 00 00 00 04 17 01 00 a5 10 0c 78

Re: [PATCH v6 14/14] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board

2025-01-24 Thread Damon Ding
Hi Dmitry, On 2025/1/23 19:32, Dmitry Baryshkov wrote: On Thu, Jan 23, 2025 at 06:07:47PM +0800, Damon Ding wrote: Add the necessary DT changes to enable eDP0 on RK3588S EVB1 board: - Set pinctrl of pwm12 for backlight - Enable edp0/hdptxphy0/vp2 - Assign the parent of DCLK_VOP2_SRC to

Re: [PATCH v6 12/14] drm/edp-panel: Add LG Display panel model LP079QX1-SP0V

2025-01-24 Thread Damon Ding
Hi Dmitry, On 2025/1/23 19:31, Dmitry Baryshkov wrote: On Thu, Jan 23, 2025 at 06:07:45PM +0800, Damon Ding wrote: The raw edid for LP079QX1-SP0V panel model is: 00 ff ff ff ff ff ff 00 16 83 00 00 00 00 00 00 04 17 01 00 a5 10 0c 78 06 ef 05 a3 54 4c 99 26 0f 50 54 00 00 00 01 01 01 01 01 01

Re: [PATCH v6 09/14] dt-bindings: display: rockchip: analogix-dp: Add support for RK3588

2025-01-24 Thread Damon Ding
Hi Rob, On 2025/1/23 19:22, Rob Herring (Arm) wrote: On Thu, 23 Jan 2025 18:07:42 +0800, Damon Ding wrote: Compared with RK3288/RK3399, the HBR2 link rate support is the main improvement of RK3588 eDP TX controller, and there are also two independent eDP display interfaces on RK3588 Soc. The

dri-devel@lists.freedesktop.org

2025-01-24 Thread Damon Ding
Hi Dmitry, On 2025/1/23 19:19, Dmitry Baryshkov wrote: On Thu, Jan 23, 2025 at 06:07:39PM +0800, Damon Ding wrote: Add two new functions: one to find &analogix_dp_device.plat_data via &drm_dp_aux, and the other to get &analogix_dp_device.aux. Both of them serve for the functio

[PATCH v6 13/14] arm64: dts: rockchip: Add eDP0 node for RK3588

2025-01-23 Thread Damon Ding
Add support for the eDP0 output on RK3588 SoC. Signed-off-by: Damon Ding --- Changes in v3: - Remove currently unsupported property '#sound-dai-cells' Changes in v4: - Remove currently unsupported clock 'spdif' --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi | 28 ++

[PATCH v6 12/14] drm/edp-panel: Add LG Display panel model LP079QX1-SP0V

2025-01-23 Thread Damon Ding
00 00 00 00 00 00 00 00 00 00 fe 00 4c 50 30 37 39 51 58 31 2d 53 50 30 56 00 00 00 fc 00 43 6f 6c 6f 72 20 4c 43 44 0a 20 20 20 00 3f Signed-off-by: Damon Ding --- drivers/gpu/drm/panel/panel-edp.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-edp.c b

[PATCH v6 09/14] dt-bindings: display: rockchip: analogix-dp: Add support for RK3588

2025-01-23 Thread Damon Ding
ned-off-by: Damon Ding --- Changes in v2: - Add the main defferences of the RK3588 eDP and the previous versions in commit message Changes in v3: - Expand the property clock-names, resets and reset-names Changes in v4: - Remove 'spdif' clock which added in v3 - Add the comment

[PATCH v6 14/14] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board

2025-01-23 Thread Damon Ding
z. Signed-off-by: Damon Ding --- Changes in v2: - Remove brightness-levels and default-brightness-level properties in backlight node. - Add the detail DT changes to commit message. Changes in v3: - Use aux-bus instead of platform bus for edp-panel. Changes in v4: - Add comments related to

[PATCH v6 11/14] drm/rockchip: analogix_dp: Add support for RK3588

2025-01-23 Thread Damon Ding
#x27;. And the newly added 'apb' reset control is to ensure the APB bus of eDP controller works well on the RK3588 SoC. Signed-off-by: Damon Ding --- Changes in v2: - Add support for the other eDP output edp1 Changes in v3: - Fix the unexpected use of alias - Add more details in commit

[PATCH v6 08/14] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-01-23 Thread Damon Ding
get the panel from the DP AUX bus, it will then try the other way to get panel information through the platform bus. Signed-off-by: Damon Ding --- Changes in v4: - Use done_probing() to call drm_of_find_panel_or_bridge() and component_add() when getting panel from the DP AUX bus Changes in v5

[PATCH v6 02/14] drm/rockchip: analogix_dp: Use formalized struct definition for grf field

2025-01-23 Thread Damon Ding
The formalized struct definition will makes grf field operations more concise and easier to extend. Signed-off-by: Damon Ding --- Changes in v2: - Initialize struct rockchip_dp_chip_data rk3399_edp/rk3288_dp in order of its members Changes in v6: - Pass 'dp' in drm_...() rathe

[PATCH v6 00/14] Add eDP support for RK3588

2025-01-23 Thread Damon Ding
for panel model LP079QX1-SP0V. Patch 13is the addition of RK3588 eDP0 node. Patch 14is to enable the eDP0 display on RK3588S EVB1 board. Damon Ding (14): drm/rockchip: analogix_dp: Replace DRM_...() functions with drm_...() drm/rockchip: analogix_dp: Use formalized struct

[PATCH v6 10/14] drm/bridge: analogix_dp: Add support for RK3588

2025-01-23 Thread Damon Ding
Expand enum analogix_dp_devtype with RK3588_EDP, and add max_link_rate and max_lane_count configs for it. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- Changes in v5: - Add the RK3588_EDP related modification in analogix_dp.h - Move this commit above related commit on the

[PATCH v6 07/14] drm/bridge: analogix_dp: Add support to get panel from the DP AUX bus

2025-01-23 Thread Damon Ding
, add a new function analogix_dp_remove() to ensure symmetry for PM operations. Signed-off-by: Damon Ding --- Changes in v4: - Use done_probing() to call drm_of_find_panel_or_bridge() and component_add() when getting panel from the DP AUX bus Changes in v5: - Advance PM operations to make eDP

[PATCH v6 05/14] dt-bindings: display: rockchip: analogix-dp: Add support to get panel from the DP AUX bus

2025-01-23 Thread Damon Ding
According to Documentation/devicetree/bindings/display/dp-aux-bus.yaml, it is a good way to get panel through the DP AUX bus. Acked-by: Krzysztof Kozlowski Signed-off-by: Damon Ding --- Changes in v4: - Move the dt-bindings commit before related driver commits Changes in v5: - Remove the

[PATCH v6 03/14] drm/rockchip: analogix_dp: Expand device data to support multiple edp display

2025-01-23 Thread Damon Ding
There are two main modifications: one is expanding struct rockchip_dp_chip_data to an array, and the other is adding &rockchip_dp_chip_data.reg to separate different edp devices. Signed-off-by: Damon Ding --- Changes in v6: - Add the description of &rockchip_dp_chip_data.reg -

[PATCH v6 04/14] drm/bridge: analogix_dp: Add support for phy configuration.

2025-01-23 Thread Damon Ding
Add support to configurate link rate, lane count, voltage swing and pre-emphasis with phy_configure(). It is helpful in application scenarios where analogix controller is mixed with the phy of other vendors. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- Changes in v2: - remove

dri-devel@lists.freedesktop.org

2025-01-23 Thread Damon Ding
Add two new functions: one to find &analogix_dp_device.plat_data via &drm_dp_aux, and the other to get &analogix_dp_device.aux. Both of them serve for the function of getting panel from DP AUX bus, which is why they are included in a single commit. Signed-off-by: Damon Ding Reviewe

[PATCH v6 01/14] drm/rockchip: analogix_dp: Replace DRM_...() functions with drm_...()

2025-01-23 Thread Damon Ding
According to the comments in include/drm/drm_print.h, the DRM_...() functions are deprecated in favor of drm_...() or dev_...() functions. Use drm_err()/drm_dbg_core()/drm_dbg_kms() instead of DRM_DEV_ERROR()/DRM_ERROR()/DRM_DEV_DEBUG()/DRM_DEBUG_KMS(). Signed-off-by: Damon Ding --- Changes

Re: [PATCH v5 12/20] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-01-22 Thread Damon Ding
Hi Dmitry, On 2025/1/22 16:17, Damon Ding wrote: Hi Dmitry, On 2025/1/9 20:48, Dmitry Baryshkov wrote: On Thu, Jan 09, 2025 at 11:27:17AM +0800, Damon Ding wrote: Move drm_of_find_panel_or_bridge() a little later and combine it with component_add() into a new function rockchip_dp_link_panel

Re: [PATCH v5 20/20] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board

2025-01-22 Thread Damon Ding
Hi Andy, On 2025/1/11 18:28, Andy Yan wrote: Hi Damon, At 2025-01-09 11:27:25, "Damon Ding" wrote: Add the necessary DT changes to enable eDP0 on RK3588S EVB1 board: - Set pinctrl of pwm12 for backlight - Enable edp0/hdptxphy0/vp2 - Add aux-bus/panel nodes Signed-off-by:

Re: [PATCH v5 05/20] drm/rockchip: analogix_dp: Replace DRM_...() functions with drm_...() or dev_...()

2025-01-22 Thread Damon Ding
Hi Andy, On 2025/1/9 14:28, Andy Yan wrote: Hi Damon, At 2025-01-09 11:27:10, "Damon Ding" wrote: According to the comments in include/drm/drm_print.h, the DRM_...() functions are deprecated in favor of drm_...() or dev_...() functions. Use drm_err()/drm_dbg_core()/drm_dbg_kms()

Re: [PATCH v5 12/20] drm/rockchip: analogix_dp: Add support to get panel from the DP AUX bus

2025-01-22 Thread Damon Ding
Hi Dmitry, On 2025/1/9 20:48, Dmitry Baryshkov wrote: On Thu, Jan 09, 2025 at 11:27:17AM +0800, Damon Ding wrote: Move drm_of_find_panel_or_bridge() a little later and combine it with component_add() into a new function rockchip_dp_link_panel(). The function will serve as done_probing

Re: [PATCH v5 11/20] drm/bridge: analogix_dp: Add support to get panel from the DP AUX bus

2025-01-22 Thread Damon Ding
Hi Dmitry, On 2025/1/9 20:45, Dmitry Baryshkov wrote: On Thu, Jan 09, 2025 at 11:27:16AM +0800, Damon Ding wrote: The main modification is moving the DP AUX initialization from function analogix_dp_bind() to analogix_dp_probe(). In order to get the EDID of eDP panel during probing, it is also

[PATCH v6 6/6] arm64: dts: rockchip: Fix label name of hdptxphy for RK3588

2025-01-20 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Signed-off-by: Damon Ding --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi

[PATCH v6 5/6] dt-bindings: display: rockchip: Fix label name of hdptxphy for RK3588 HDMI TX Controller

2025-01-20 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Acked-by: Rob Herring (Arm) Signed-off-by: Damon Ding --- .../bindings/display

[PATCH v6 4/6] phy: phy-rockchip-samsung-hdptx: Add eDP mode support for RK3588

2025-01-20 Thread Damon Ding
link rate are set according to the eDP 1.3 requirements. Signed-off-by: Damon Ding --- Changes in v2: - Add the module author Changes in v3: - Split this patch into two, one for correction and the other for extension Changes in v4: - Add link_rate and lanes parameters in struct rk_hdptx_phy

[PATCH v6 3/6] phy: phy-rockchip-samsung-hdptx: Add the '_MASK' suffix to all registers

2025-01-20 Thread Damon Ding
Adding the '_MASK' suffix to all registers in order to ensures consistency in the naming convention for register macros throughout the file. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- Changes in v4: - Split the older patch related to the renaming of registers

[PATCH v6 2/6] phy: phy-rockchip-samsung-hdptx: Supplement some register names with their full version

2025-01-20 Thread Damon Ding
Complete the register names of CMN_REG(0081) and CMN_REG(0087) to their full version, and it can help to better match the datasheet. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH v6 1/6] phy: phy-rockchip-samsung-hdptx: Swap the definitions of LCPLL_REF and ROPLL_REF

2025-01-20 Thread Damon Ding
According to the datasheet, setting the dig_clk_sel bit of CMN_REG(0097) to 1'b1 selects LCPLL as the reference clock, while setting it to 1'b0 selects the ROPLL. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c | 4 ++

[PATCH v6 0/6] Add eDP mode support for Rockchip Samsung HDPTX PHY

2025-01-20 Thread Damon Ding
Picked from: https://patchwork.kernel.org/project/linux-rockchip/list/?series=923593 These patchs have been tested with a 1536x2048p60 eDP panel on RK3588S EVB1 board, and HDMI 1080P/4K display also has been verified on RK3588 EVB1 board. Damon Ding (6): phy: phy-rockchip-samsung-hdptx: Swap

Re: [PATCH v1 0/6] Add eDP mode support for Rockchip Samsung HDPTX PHY

2025-01-19 Thread Damon Ding
Hi Dmitry, On 2025/1/13 17:10, Dmitry Baryshkov wrote: On Sun, Jan 12, 2025 at 05:07:08PM +0800, Damon Ding wrote: Picked from: https://patchwork.kernel.org/project/linux-rockchip/list/?series=923593 Then it should have been v6, not v1. I will update a patch series to fix it. These

Re: [PATCH v5 13/20] dt-bindings: display: rockchip: analogix-dp: Add support for RK3588

2025-01-19 Thread Damon Ding
Hi Krzysztof, On 2025/1/9 16:54, Krzysztof Kozlowski wrote: On Thu, Jan 09, 2025 at 11:27:18AM +0800, Damon Ding wrote: Compared with RK3288/RK3399, the HBR2 link rate support is the main improvement of RK3588 eDP TX controller, and there are also two independent eDP display interfaces on

[PATCH v1 6/6] arm64: dts: rockchip: Fix label name of hdptxphy for RK3588

2025-01-12 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Signed-off-by: Damon Ding --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi

[PATCH v1 5/6] dt-bindings: display: rockchip: Fix label name of hdptxphy for RK3588 HDMI TX Controller

2025-01-12 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Acked-by: Rob Herring (Arm) Signed-off-by: Damon Ding --- .../bindings/display

[PATCH v1 4/6] phy: phy-rockchip-samsung-hdptx: Add eDP mode support for RK3588

2025-01-12 Thread Damon Ding
link rate are set according to the eDP 1.3 requirements. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- .../phy/rockchip/phy-rockchip-samsung-hdptx.c | 879 +- 1 file changed, 869 insertions(+), 10 deletions(-) diff --git a/drivers/phy/rockchip/phy-rockchip-samsung

[PATCH v1 2/6] phy: phy-rockchip-samsung-hdptx: Supplement some register names with their full version

2025-01-12 Thread Damon Ding
Complete the register names of CMN_REG(0081) and CMN_REG(0087) to their full version, and it can help to better match the datasheet. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH v1 3/6] phy: phy-rockchip-samsung-hdptx: Add the '_MASK' suffix to all registers

2025-01-12 Thread Damon Ding
Adding the '_MASK' suffix to all registers in order to ensures consistency in the naming convention for register macros throughout the file. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- .../phy/rockchip/phy-rockchip-samsung-hdptx.c | 88 +-- 1 file c

[PATCH v1 1/6] phy: phy-rockchip-samsung-hdptx: Swap the definitions of LCPLL_REF and ROPLL_REF

2025-01-12 Thread Damon Ding
According to the datasheet, setting the dig_clk_sel bit of CMN_REG(0097) to 1'b1 selects LCPLL as the reference clock, while setting it to 1'b0 selects the ROPLL. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c | 4 ++

[PATCH v1 0/6] Add eDP mode support for Rockchip Samsung HDPTX PHY

2025-01-12 Thread Damon Ding
Picked from: https://patchwork.kernel.org/project/linux-rockchip/list/?series=923593 These patchs have been tested with a 1536x2048p60 eDP panel on RK3588S EVB1 board, and HDMI 1080P/4K display also has been verified on RK3588 EVB1 board. Damon Ding (6): phy: phy-rockchip-samsung-hdptx: Swap

[PATCH v5 01/20] phy: phy-rockchip-samsung-hdptx: Swap the definitions of LCPLL_REF and ROPLL_REF

2025-01-08 Thread Damon Ding
According to the datasheet, setting the dig_clk_sel bit of CMN_REG(0097) to 1'b1 selects LCPLL as the reference clock, while setting it to 1'b0 selects the ROPLL. Signed-off-by: Damon Ding Reviewed-by: Dmitry Baryshkov --- drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c | 4 ++

[PATCH v5 17/20] dt-bindings: display: rockchip: Fix label name of hdptxphy for RK3588 HDMI TX Controller

2025-01-08 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Acked-by: Rob Herring (Arm) Signed-off-by: Damon Ding --- .../bindings/display

dri-devel@lists.freedesktop.org

2025-01-08 Thread Damon Ding
Add two new functions: one to find &analogix_dp_device.plat_data via &drm_dp_aux, and the other to get &analogix_dp_device.aux. Both of them serve for the function of getting panel from DP AUX bus, which is why they are included in a single commit. Signed-off-by: Damon Ding --- dri

[PATCH v5 18/20] arm64: dts: rockchip: Fix label name of hdptxphy for RK3588

2025-01-08 Thread Damon Ding
The hdptxphy is a combo transmit-PHY for HDMI2.1 TMDS Link, FRL Link, DP and eDP Link. Therefore, it is better to name it hdptxphy0 other than hdptxphy_hdmi0, which will be referenced by both hdmi0 and edp0 nodes. Signed-off-by: Damon Ding --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi

[PATCH v5 20/20] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board

2025-01-08 Thread Damon Ding
Add the necessary DT changes to enable eDP0 on RK3588S EVB1 board: - Set pinctrl of pwm12 for backlight - Enable edp0/hdptxphy0/vp2 - Add aux-bus/panel nodes Signed-off-by: Damon Ding --- Changes in v2: - Remove brightness-levels and default-brightness-level properties in backlight node

[PATCH v5 19/20] arm64: dts: rockchip: Add eDP0 node for RK3588

2025-01-08 Thread Damon Ding
Add support for the eDP0 output on RK3588 SoC. Signed-off-by: Damon Ding --- Changes in v3: - Remove currently unsupported property '#sound-dai-cells' Changes in v4: - Remove currently unsupported clock 'spdif' --- arch/arm64/boot/dts/rockchip/rk3588-base.dtsi | 28 ++

[PATCH v5 14/20] drm/bridge: analogix_dp: Add support for RK3588

2025-01-08 Thread Damon Ding
Expand enum analogix_dp_devtype with RK3588_EDP, and add max_link_rate and max_lane_count configs for it. Signed-off-by: Damon Ding --- Changes in v5: - Add the RK3588_EDP related modification in analogix_dp.h - Move this commit above related commit on the Rockchip side --- drivers/gpu/drm

[PATCH v5 16/20] drm/edp-panel: Add LG Display panel model LP079QX1-SP0V

2025-01-08 Thread Damon Ding
00 00 00 00 00 00 00 00 00 00 fe 00 4c 50 30 37 39 51 58 31 2d 53 50 30 56 00 00 00 fc 00 43 6f 6c 6f 72 20 4c 43 44 0a 20 20 20 00 3f Signed-off-by: Damon Ding --- drivers/gpu/drm/panel/panel-edp.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-edp.c b

[PATCH v5 15/20] drm/rockchip: analogix_dp: Add support for RK3588

2025-01-08 Thread Damon Ding
#x27;. And the newly added 'apb' reset control is to ensure the APB bus of eDP controller works well on the RK3588 SoC. Signed-off-by: Damon Ding --- Changes in v2: - Add support for the other eDP output edp1 Changes in v3: - Fix the unexpected use of alias - Add more details in commit

  1   2   >