linux-kernel - Re: [PATCH v3 1/1] arm64: dts: qcom: Add coresight nodes for x1e80100

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <Zt+okXSkE63O/PYk@jiegan-gv.ap.qualcomm.com>
Date: Tue, 10 Sep 2024 10:01:53 +0800
From: JieGan <quic_jiegan@...cinc.com>
To: Konrad Dybcio <konradybcio@...nel.org>
CC: Bjorn Andersson <andersson@...nel.org>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>, <linux-arm-msm@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        Tingwei Zhang <quic_tingweiz@...cinc.com>,
        Jinlong Mao <quic_jinlmao@...cinc.com>,
        Tao Zhang <quic_taozha@...cinc.com>,
        Yuanfang Zhang <quic_yuanfang@...cinc.com>,
        Song Chai
	<quic_songchai@...cinc.com>,
        Yushan Li <quic_yushli@...cinc.com>
Subject: Re: [PATCH v3 1/1] arm64: dts: qcom: Add coresight nodes for x1e80100

On Mon, Sep 09, 2024 at 10:05:08PM +0200, Konrad Dybcio wrote:
> On 5.09.2024 12:38 PM, Jie Gan wrote:
> > Add following coresight components for x1e80100 platform.
> > It includes CTI, dummy sink, dynamic Funnel, Replicator, STM,
> > TPDM, TPDA and TMC ETF.
> > 
> > Signed-off-by: Jie Gan <quic_jiegan@...cinc.com>
> > Tested-by: Yushan Li <quic_yushli@...cinc.com>
> > ---
> >  arch/arm64/boot/dts/qcom/x1e80100.dtsi | 1516 ++++++++++++++++++++++++
> >  1 file changed, 1516 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/qcom/x1e80100.dtsi b/arch/arm64/boot/dts/qcom/x1e80100.dtsi
> > index 74b694e74705..9d6f3098e144 100644
> > --- a/arch/arm64/boot/dts/qcom/x1e80100.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/x1e80100.dtsi
> > @@ -305,6 +305,19 @@ CLUSTER_CL5: cluster-sleep-1 {
> >  		};
> >  	};
> >  
> > +	dummy-sink {
> > +		compatible = "arm,coresight-dummy-sink";
> > +
> > +		in-ports {
> > +			port {
> > +				eud_in: endpoint {
> > +					remote-endpoint =
> > +					<&swao_rep_out1>;
> 
> Don't be scared to keep the lines 100-long, easier to read that way
>
sure, will fix in next version.
 
> [...]
> 
> > +		tpda@...04000 {
> > +			compatible = "qcom,coresight-tpda", "arm,primecell";
> > +			reg = <0x0 0x10004000 0x0 0x1000>;
> > +
> > +			clocks = <&aoss_qmp>;
> > +			clock-names = "apb_pclk";
> > +
> > +			out-ports {
> > +				port {
> > +					qdss_tpda_out: endpoint {
> > +						remote-endpoint =
> > +						<&funnel0_in6>;
> > +					};
> > +				};
> > +			};
> > +
> > +			in-ports {
> 
> 'i' < 'o', please sort things alphabetically if there's no other sorting key
> as per Documentation/devicetree/bindings/dts-coding-style.rst
> 
Will sort with alphabet order for "in-ports" and "out-ports" property.

> [...]
> 
> > +		tpda@...2b000 {
> > +			compatible = "qcom,coresight-tpda", "arm,primecell";
> > +			reg = <0x0 0x10c2b000 0x0 0x1000>;
> > +
> > +			clocks = <&aoss_qmp>;
> > +			clock-names = "apb_pclk";
> > +
> > +			out-ports {
> > +				port {
> > +					dlct1_tpda_out: endpoint {
> > +						remote-endpoint =
> > +						<&dlct1_funnel_in0>;
> > +					};
> > +				};
> > +			};
> > +
> > +			in-ports {
> > +				#address-cells = <1>;
> > +				#size-cells = <0>;
> > +
> > +				port@4 {
> > +					reg = <4>;
> > +					dlct1_tpda_in4: endpoint {
> 
> Please keep a new line between the last property (reg here) and the
> following subnode
> 
Will fix in next version.

> I was able to confirm that this patch doesn't break booting on the
> Surface laptop anymore.
> 
> Konrad

Thanks for testing!

Thanks,
Jie