All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: Damon Ding <damon.ding@rock-chips.com>
Cc: 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,
	Damon Ding <damon.ding@rock-chips.com>
Subject: Re: [PATCH v2 11/11] arm64: dts: rockchip: Add nodes related to eDP1 for RK3588
Date: Wed, 18 Dec 2024 13:45:54 +0100	[thread overview]
Message-ID: <2173171.OBFZWjSADL@diego> (raw)
In-Reply-To: <20241216031225.3746-12-damon.ding@rock-chips.com>

Hi Damon,

Am Montag, 16. Dezember 2024, 04:12:25 CET schrieb Damon Ding:
> The related nodes are hdptxphy1_grf, hdptxphy1 and edp1. And the
> aliases edp0 and edp1 are added to separate two independent eDP
> devices.
> 
> Signed-off-by: Damon Ding <damon.ding@rock-chips.com>
> ---
>  .../arm64/boot/dts/rockchip/rk3588-extra.dtsi | 55 +++++++++++++++++++
>  1 file changed, 55 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi b/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> index 0ce0934ec6b7..17cc0b619744 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> @@ -7,6 +7,11 @@
>  #include "rk3588-extra-pinctrl.dtsi"
>  
>  / {
> +	aliases {
> +		edp0 = &edp0;
> +		edp1 = &edp1;
> +	};
> +

drop aliases please


> @@ -67,6 +72,11 @@ u2phy1_otg: otg-port {
>  		};
>  	};
>  
> +	hdptxphy1_grf: syscon@fd5e4000 {
> +		compatible = "rockchip,rk3588-hdptxphy-grf", "syscon";
> +		reg = <0x0 0xfd5e4000 0x0 0x100>;
> +	};
> +
>  	i2s8_8ch: i2s@fddc8000 {
>  		compatible = "rockchip,rk3588-i2s-tdm";
>  		reg = <0x0 0xfddc8000 0x0 0x1000>;

> @@ -395,6 +434,22 @@ sata-port@0 {
>  		};
>  	};
>  
> +	hdptxphy1: phy@fed70000 {
> +		compatible = "rockchip,rk3588-hdptx-phy";
> +		reg = <0x0 0xfed70000 0x0 0x2000>;
> +		clocks = <&cru CLK_USB2PHY_HDPTXRXPHY_REF>, <&cru PCLK_HDPTX1>;
> +		clock-names = "ref", "apb";
> +		#phy-cells = <0>;
> +		resets = <&cru SRST_HDPTX1>, <&cru SRST_P_HDPTX1>,
> +			 <&cru SRST_HDPTX1_INIT>, <&cru SRST_HDPTX1_CMN>,
> +			 <&cru SRST_HDPTX1_LANE>, <&cru SRST_HDPTX1_ROPLL>,
> +			 <&cru SRST_HDPTX1_LCPLL>;
> +		reset-names = "phy", "apb", "init", "cmn", "lane", "ropll",
> +			      "lcpll";
> +		rockchip,grf = <&hdptxphy1_grf>;
> +		status = "disabled";
> +	};

the hdptxphy nodes should be their own patch, also because most likely
Cristian's patch for hdmi1 [0] will be slightly faster.


[0] https://lore.kernel.org/r/20241211-rk3588-hdmi1-v2-2-02cdca22ff68@collabora.com





WARNING: multiple messages have this Message-ID (diff)
From: "Heiko Stübner" <heiko@sntech.de>
To: Damon Ding <damon.ding@rock-chips.com>
Cc: 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,
	Damon Ding <damon.ding@rock-chips.com>
Subject: Re: [PATCH v2 11/11] arm64: dts: rockchip: Add nodes related to eDP1 for RK3588
Date: Wed, 18 Dec 2024 13:45:54 +0100	[thread overview]
Message-ID: <2173171.OBFZWjSADL@diego> (raw)
In-Reply-To: <20241216031225.3746-12-damon.ding@rock-chips.com>

Hi Damon,

Am Montag, 16. Dezember 2024, 04:12:25 CET schrieb Damon Ding:
> The related nodes are hdptxphy1_grf, hdptxphy1 and edp1. And the
> aliases edp0 and edp1 are added to separate two independent eDP
> devices.
> 
> Signed-off-by: Damon Ding <damon.ding@rock-chips.com>
> ---
>  .../arm64/boot/dts/rockchip/rk3588-extra.dtsi | 55 +++++++++++++++++++
>  1 file changed, 55 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi b/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> index 0ce0934ec6b7..17cc0b619744 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> @@ -7,6 +7,11 @@
>  #include "rk3588-extra-pinctrl.dtsi"
>  
>  / {
> +	aliases {
> +		edp0 = &edp0;
> +		edp1 = &edp1;
> +	};
> +

drop aliases please


> @@ -67,6 +72,11 @@ u2phy1_otg: otg-port {
>  		};
>  	};
>  
> +	hdptxphy1_grf: syscon@fd5e4000 {
> +		compatible = "rockchip,rk3588-hdptxphy-grf", "syscon";
> +		reg = <0x0 0xfd5e4000 0x0 0x100>;
> +	};
> +
>  	i2s8_8ch: i2s@fddc8000 {
>  		compatible = "rockchip,rk3588-i2s-tdm";
>  		reg = <0x0 0xfddc8000 0x0 0x1000>;

> @@ -395,6 +434,22 @@ sata-port@0 {
>  		};
>  	};
>  
> +	hdptxphy1: phy@fed70000 {
> +		compatible = "rockchip,rk3588-hdptx-phy";
> +		reg = <0x0 0xfed70000 0x0 0x2000>;
> +		clocks = <&cru CLK_USB2PHY_HDPTXRXPHY_REF>, <&cru PCLK_HDPTX1>;
> +		clock-names = "ref", "apb";
> +		#phy-cells = <0>;
> +		resets = <&cru SRST_HDPTX1>, <&cru SRST_P_HDPTX1>,
> +			 <&cru SRST_HDPTX1_INIT>, <&cru SRST_HDPTX1_CMN>,
> +			 <&cru SRST_HDPTX1_LANE>, <&cru SRST_HDPTX1_ROPLL>,
> +			 <&cru SRST_HDPTX1_LCPLL>;
> +		reset-names = "phy", "apb", "init", "cmn", "lane", "ropll",
> +			      "lcpll";
> +		rockchip,grf = <&hdptxphy1_grf>;
> +		status = "disabled";
> +	};

the hdptxphy nodes should be their own patch, also because most likely
Cristian's patch for hdmi1 [0] will be slightly faster.


[0] https://lore.kernel.org/r/20241211-rk3588-hdmi1-v2-2-02cdca22ff68@collabora.com




-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

WARNING: multiple messages have this Message-ID (diff)
From: "Heiko Stübner" <heiko@sntech.de>
To: Damon Ding <damon.ding@rock-chips.com>
Cc: robh@kernel.org, conor+dt@kernel.org, algea.cao@rock-chips.com,
	rfoss@kernel.org, devicetree@vger.kernel.org,
	linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org,
	sebastian.reichel@collabora.com, dri-devel@lists.freedesktop.org,
	hjc@rock-chips.com, kever.yang@rock-chips.com,
	linux-rockchip@lists.infradead.org, vkoul@kernel.org,
	Damon Ding <damon.ding@rock-chips.com>,
	andy.yan@rock-chips.com, krzk+dt@kernel.org,
	linux-arm-kernel@lists.infradead.org, l.stach@pengutronix.de
Subject: Re: [PATCH v2 11/11] arm64: dts: rockchip: Add nodes related to eDP1 for RK3588
Date: Wed, 18 Dec 2024 13:45:54 +0100	[thread overview]
Message-ID: <2173171.OBFZWjSADL@diego> (raw)
In-Reply-To: <20241216031225.3746-12-damon.ding@rock-chips.com>

Hi Damon,

Am Montag, 16. Dezember 2024, 04:12:25 CET schrieb Damon Ding:
> The related nodes are hdptxphy1_grf, hdptxphy1 and edp1. And the
> aliases edp0 and edp1 are added to separate two independent eDP
> devices.
> 
> Signed-off-by: Damon Ding <damon.ding@rock-chips.com>
> ---
>  .../arm64/boot/dts/rockchip/rk3588-extra.dtsi | 55 +++++++++++++++++++
>  1 file changed, 55 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi b/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> index 0ce0934ec6b7..17cc0b619744 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3588-extra.dtsi
> @@ -7,6 +7,11 @@
>  #include "rk3588-extra-pinctrl.dtsi"
>  
>  / {
> +	aliases {
> +		edp0 = &edp0;
> +		edp1 = &edp1;
> +	};
> +

drop aliases please


> @@ -67,6 +72,11 @@ u2phy1_otg: otg-port {
>  		};
>  	};
>  
> +	hdptxphy1_grf: syscon@fd5e4000 {
> +		compatible = "rockchip,rk3588-hdptxphy-grf", "syscon";
> +		reg = <0x0 0xfd5e4000 0x0 0x100>;
> +	};
> +
>  	i2s8_8ch: i2s@fddc8000 {
>  		compatible = "rockchip,rk3588-i2s-tdm";
>  		reg = <0x0 0xfddc8000 0x0 0x1000>;

> @@ -395,6 +434,22 @@ sata-port@0 {
>  		};
>  	};
>  
> +	hdptxphy1: phy@fed70000 {
> +		compatible = "rockchip,rk3588-hdptx-phy";
> +		reg = <0x0 0xfed70000 0x0 0x2000>;
> +		clocks = <&cru CLK_USB2PHY_HDPTXRXPHY_REF>, <&cru PCLK_HDPTX1>;
> +		clock-names = "ref", "apb";
> +		#phy-cells = <0>;
> +		resets = <&cru SRST_HDPTX1>, <&cru SRST_P_HDPTX1>,
> +			 <&cru SRST_HDPTX1_INIT>, <&cru SRST_HDPTX1_CMN>,
> +			 <&cru SRST_HDPTX1_LANE>, <&cru SRST_HDPTX1_ROPLL>,
> +			 <&cru SRST_HDPTX1_LCPLL>;
> +		reset-names = "phy", "apb", "init", "cmn", "lane", "ropll",
> +			      "lcpll";
> +		rockchip,grf = <&hdptxphy1_grf>;
> +		status = "disabled";
> +	};

the hdptxphy nodes should be their own patch, also because most likely
Cristian's patch for hdmi1 [0] will be slightly faster.


[0] https://lore.kernel.org/r/20241211-rk3588-hdmi1-v2-2-02cdca22ff68@collabora.com




_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

  reply	other threads:[~2024-12-18 12:47 UTC|newest]

Thread overview: 78+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-16  3:12 [PATCH v2 00/11] Add eDP support for RK3588 Damon Ding
2024-12-16  3:12 ` Damon Ding
2024-12-16  3:12 ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 01/11] drm/rockchip: analogix_dp: Use formalized struct definition for grf field Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 02/11] dt-bindings: display: rockchip: analogix-dp: Add support for RK3588 Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 03/11] drm/rockchip: analogix_dp: " Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  8:57   ` Dmitry Baryshkov
2024-12-16  8:57     ` Dmitry Baryshkov
2024-12-16  8:57     ` Dmitry Baryshkov
2024-12-16  9:11     ` Heiko Stübner
2024-12-16  9:11       ` Heiko Stübner
2024-12-16  9:11       ` Heiko Stübner
2024-12-16 10:04       ` Dmitry Baryshkov
2024-12-16 10:04         ` Dmitry Baryshkov
2024-12-16 10:04         ` Dmitry Baryshkov
2024-12-16 10:09       ` Damon Ding
2024-12-16 10:09         ` Damon Ding
2024-12-16 10:09         ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 04/11] phy: phy-rockchip-samsung-hdptx: Add support for eDP mode Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  9:00   ` Dmitry Baryshkov
2024-12-16  9:00     ` Dmitry Baryshkov
2024-12-16  9:00     ` Dmitry Baryshkov
2024-12-16 10:06     ` Damon Ding
2024-12-16 10:06       ` Damon Ding
2024-12-16 10:06       ` Damon Ding
2024-12-18 12:35   ` Heiko Stübner
2024-12-18 12:35     ` Heiko Stübner
2024-12-18 12:35     ` Heiko Stübner
2024-12-19  1:35     ` Damon Ding
2024-12-19  1:35       ` Damon Ding
2024-12-19  1:35       ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 05/11] drm/bridge: analogix_dp: add support for RK3588 Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 06/11] drm/bridge: analogix_dp: Add support for phy configuration Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 07/11] dt-bindings: display: rockchip: Fix label name of hdptxphy for RK3588 HDMI TX Controller Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 08/11] arm64: dts: rockchip: Fix label name of hdptxphy for RK3588 Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 09/11] arm64: dts: rockchip: Add eDP0 node " Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 10/11] arm64: dts: rockchip: Enable eDP0 display on RK3588S EVB1 board Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  9:06   ` Dmitry Baryshkov
2024-12-16  9:06     ` Dmitry Baryshkov
2024-12-16  9:06     ` Dmitry Baryshkov
2024-12-16 10:15     ` Damon Ding
2024-12-16 10:15       ` Damon Ding
2024-12-16 10:15       ` Damon Ding
2024-12-16  3:12 ` [PATCH v2 11/11] arm64: dts: rockchip: Add nodes related to eDP1 for RK3588 Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-16  3:12   ` Damon Ding
2024-12-18 12:45   ` Heiko Stübner [this message]
2024-12-18 12:45     ` Heiko Stübner
2024-12-18 12:45     ` Heiko Stübner
2024-12-19  1:27     ` Damon Ding
2024-12-19  1:27       ` Damon Ding
2024-12-19  1:27       ` Damon Ding
2024-12-16 16:29 ` [PATCH v2 00/11] Add eDP support " Rob Herring (Arm)
2024-12-16 16:29   ` Rob Herring (Arm)
2024-12-16 16:29   ` Rob Herring (Arm)
2024-12-17  2:32   ` Damon Ding
2024-12-17  2:32     ` Damon Ding
2024-12-17  2:32     ` Damon Ding

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2173171.OBFZWjSADL@diego \
    --to=heiko@sntech.de \
    --cc=algea.cao@rock-chips.com \
    --cc=andy.yan@rock-chips.com \
    --cc=conor+dt@kernel.org \
    --cc=cristian.ciocaltea@collabora.com \
    --cc=damon.ding@rock-chips.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hjc@rock-chips.com \
    --cc=kever.yang@rock-chips.com \
    --cc=krzk+dt@kernel.org \
    --cc=l.stach@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-phy@lists.infradead.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=rfoss@kernel.org \
    --cc=robh@kernel.org \
    --cc=sebastian.reichel@collabora.com \
    --cc=vkoul@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.