From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 539E2C02181 for ; Sat, 25 Jan 2025 03:27:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=c7MldcMAKTGJgYwoCbGLzVzEW02VyfkoG5ILQhtNfYE=; b=cM+B84pqAOriDqh3VQGmhFBkxz TILxPGwF8kiWb17sR5LV11S4dAcKGHY/beKfglQtEb2zjGL0f+GTM310F+9wpFTqxRok1zC/NE/Uj EVPfH+mhBnbXiqsWuppsrtlWdEqfn3Vnl3TJYgm+4ExuxuR2Qu+ouFy7qiubIqakGwMYbVoGT9mw+ UGW2FY7iLeuaiU5VdBOk3Ab82YxGlU7ox/8bZZoYYqWwpsWOmCmdBAdSZAl9nWDBOE1TavHD64dGj ginQuhVrNRESIAVXOiTy1sppL8hXT6wtvVnHIgjZ2qX2xD9kvLLqW8Lor/0SLz/tKoyWXF78HA5b+ oLExJexQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tbWos-0000000FxFn-0CYd; Sat, 25 Jan 2025 03:26:50 +0000 Received: from mail-m15579.qiye.163.com ([101.71.155.79]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tbWnZ-0000000FxAy-05iT; Sat, 25 Jan 2025 03:25:30 +0000 Received: from [172.16.12.26] (unknown [58.22.7.114]) by smtp.qiye.163.com (Hmail) with ESMTP id 9a41508b; Sat, 25 Jan 2025 11:25:23 +0800 (GMT+08:00) Message-ID: Date: Sat, 25 Jan 2025 11:25:23 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v6 14/14] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board To: Dmitry Baryshkov Cc: heiko@sntech.de, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, rfoss@kernel.org, vkoul@kernel.org, sebastian.reichel@collabora.com, cristian.ciocaltea@collabora.com, l.stach@pengutronix.de, andy.yan@rock-chips.com, hjc@rock-chips.com, algea.cao@rock-chips.com, kever.yang@rock-chips.com, dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org References: <20250123100747.1841357-1-damon.ding@rock-chips.com> <20250123100747.1841357-15-damon.ding@rock-chips.com> Content-Language: en-US From: Damon Ding In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-HM-Spam-Status: e1kfGhgUHx5ZQUpXWQgPGg8OCBgUHx5ZQUlOS1dZFg8aDwILHllBWSg2Ly tZV1koWUFDSUNOT01LS0k3V1ktWUFJV1kPCRoVCBIfWUFZQ0pPGVYYQxpOHUtJHkpOHU9WFRQJFh oXVRMBExYaEhckFA4PWVdZGBILWUFZTkNVSUlVTFVKSk9ZV1kWGg8SFR0UWUFZT0tIVUpLSU9PT0 hVSktLVUpCS0tZBg++ X-HM-Tid: 0a949b7e27fe03a3kunm9a41508b X-HM-MType: 1 X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6Pkk6ISo4MjIWKBkrEjEvST4N LSJPFA9VSlVKTEhMTExOTklOT05PVTMWGhIXVR8aFhQVVR8SFRw7CRQYEFYYExILCFUYFBZFWVdZ EgtZQVlOQ1VJSVVMVUpKT1lXWQgBWUFIQkpINwY+ DKIM-Signature: a=rsa-sha256; b=i90GOSOVKov9ch4IdolcO1Z29iwfGzmWMLjaVWh3YmbO8GUwW36x+gQAI5kP9tEdHEvR3Q8mJGc35GOfiMNX0r7+RDd9pp7Ld3YLRyK9lQOQSrQFtqSiIQtjnPX6sA39CHdTklmYML+XFUUO1NkmMp5F1C4rSJG1SdOv1vRLp1U=; s=default; c=relaxed/relaxed; d=rock-chips.com; v=1; bh=c7MldcMAKTGJgYwoCbGLzVzEW02VyfkoG5ILQhtNfYE=; h=date:mime-version:subject:message-id:from; X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250124_192529_597928_A7AABE92 X-CRM114-Status: GOOD ( 17.68 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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 PLL_V0PLL >> - Add aux-bus/panel nodes >> >> For RK3588, the PLL_V0PLL is specifically designed for the VOP2. This >> means the clock rate of PLL_V0PLL can be adjusted according to the dclk >> rate of relevant VP. It is typically assigned as the dclk source of a >> specific VP when the clock of relevant display mode is unusual, such as >> the eDP panel 'lg,lp079qx1-sp0v' paired with RK3588S EVB1, which has a >> clock rate of 202.02MHz. >> >> 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 the use of panel compatible "lg,lp079qx1-sp0v" >> in the commit message. >> >> Changes in v5: >> - Use "edp-panel" instead of "lg,lp079qx1-sp0v" >> - Remove unnecessary comments in commit message >> - Assign the parent of DCLK_VOP2_SRC to PLL_V0PLL >> >> Changes in v6: >> - Add PLL_V0PLL related descriptions in commit message >> --- >> .../boot/dts/rockchip/rk3588s-evb1-v10.dts | 54 +++++++++++++++++++ >> 1 file changed, 54 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-evb1-v10.dts b/arch/arm64/boot/dts/rockchip/rk3588s-evb1-v10.dts >> index bc4077575beb..a8c151b41e21 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588s-evb1-v10.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-evb1-v10.dts >> @@ -9,6 +9,7 @@ >> #include >> #include >> #include >> +#include >> #include >> #include "rk3588s.dtsi" >> >> @@ -238,6 +239,41 @@ &combphy2_psu { >> status = "okay"; >> }; >> >> +&edp0 { >> + force-hpd; > > Why? Please mention the reason in the commit message. > 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. And I will add this description to the commit message in the next version. >> + status = "okay"; >> + >> + aux-bus { >> + panel { >> + compatible = "edp-panel"; >> + backlight = <&backlight>; >> + power-supply = <&vcc3v3_lcd_edp>; >> + >> + port { >> + panel_in_edp: endpoint { >> + remote-endpoint = <&edp_out_panel>; >> + }; >> + }; >> + }; >> + }; >> +}; >> + >> +&edp0_in { >> + edp0_in_vp2: endpoint { >> + remote-endpoint = <&vp2_out_edp0>; >> + }; >> +}; >> + >> +&edp0_out { >> + edp_out_panel: endpoint { >> + remote-endpoint = <&panel_in_edp>; >> + }; >> +}; >> + >> +&hdptxphy0 { >> + status = "okay"; >> +}; >> + >> &i2c3 { >> status = "okay"; >> >> @@ -399,6 +435,7 @@ usbc0_int: usbc0-int { >> }; >> >> &pwm12 { >> + pinctrl-0 = <&pwm12m1_pins>; >> status = "okay"; >> }; >> >> @@ -1168,3 +1205,20 @@ usbdp_phy0_dp_altmode_mux: endpoint@1 { >> }; >> }; >> }; >> + >> +&vop_mmu { >> + status = "okay"; >> +}; >> + >> +&vop { >> + assigned-clocks = <&cru DCLK_VOP2_SRC>; >> + assigned-clock-parents = <&cru PLL_V0PLL>; >> + status = "okay"; >> +}; >> + >> +&vp2 { >> + vp2_out_edp0: endpoint@ROCKCHIP_VOP2_EP_EDP0 { >> + reg = ; >> + remote-endpoint = <&edp0_in_vp2>; >> + }; >> +}; >> -- >> 2.34.1 >> > Best regards, Damon