linux-kernel - Re: [PATCH v5 1/4] arm64: dts: lx2160a: describe the SerDes block #2

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <2f1571a6-014b-4b70-9135-306cf9014238@solid-run.com>
Date:   Sun, 17 Sep 2023 16:33:54 +0200
From:   Josua Mayer <josua@...id-run.com>
To:     devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org
Cc:     Shawn Guo <shawnguo@...nel.org>, Li Yang <leoyang.li@....com>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>
Subject: Re: [PATCH v5 1/4] arm64: dts: lx2160a: describe the SerDes block #2

Hello everyone,

This is one of the patches in my series that hasn't received any 
feedback yet ...
(apart from Krzysztof correctly pointing out commit a description 
formatting mistake).

Please note that I intend on changing the new serdes_2 node to add 
status disabled by default.
Since the lynx-28g driver currently only supports network protocols,
enabling it by default could interfere with configurations using those 
SerDes's as PCI.

Am 18.08.23 um 13:10 schrieb Josua Mayer:
> Add description for the LX2160A second SerDes block.
> It is functionally identical to the first one already added in
> commit 3cbe93a1f540 ("arch: arm64: dts: lx2160a: describe the SerDes
> block #1").
>
> Signed-off-by: Josua Mayer <josua@...id-run.com>
> ---
> V4 -> V5: expand commit hash reference in commit message
>
>   arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> index ea6a94b57aeb..2e68c05181dd 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> @@ -626,6 +626,12 @@ serdes_1: phy@...0000 {
>   			#phy-cells = <1>;
>   		};
>   
> +		serdes_2: phy@...0000 {
> +			compatible = "fsl,lynx-28g";
> +			reg = <0x0 0x1eb0000 0x0 0x1e30>;
> +			#phy-cells = <1>;
> +		};
> +
>   		crypto: crypto@...0000 {
>   			compatible = "fsl,sec-v5.0", "fsl,sec-v4.0";
>   			fsl,sec-era = <10>;
- Josua Mayer