Received-SPF: pass (google.com: domain of linux-kernel+bounces-45143-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <e2d254cf-090e-403a-a445-6cd4f6e1a184@ti.com>
Date: Tue, 30 Jan 2024 12:52:52 -0600
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 02/11] arm64: dts: ti: k3-am65: Convert serdes_mux node
 into reg-mux
Content-Language: en-US
To: Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>,
        Tero
 Kristo <kristo@kernel.org>, Rob Herring <robh+dt@kernel.org>,
        Krzysztof
 Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley
	<conor+dt@kernel.org>
CC: <linux-arm-kernel@lists.infradead.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
References: <20240124184722.150615-1-afd@ti.com>
 <20240124184722.150615-2-afd@ti.com>
From: Andrew Davis <afd@ti.com>
In-Reply-To: <20240124184722.150615-2-afd@ti.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit

On 1/24/24 12:47 PM, Andrew Davis wrote:
> This removes a dependency on the parent node being a syscon node.
> Convert from mmio-mux to reg-mux adjusting node name and properties
> as needed.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---

This one patch is not needed, I'm going to fix this for AM65 in
a slightly different way. The rest of the series is still fine.

Andrew

>   arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 9 +++++----
>   1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> index fcea544656360..a18edd34283c7 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> @@ -492,11 +492,12 @@ serdes1_clk: clock@4090 {
>   			reg = <0x00004090 0x4>;
>   		};
>   
> -		serdes_mux: mux-controller {
> -			compatible = "mmio-mux";
> +		serdes_mux: mux-controller@4080 {
> +			compatible = "reg-mux";
> +			reg = <0x4080 0x14>;
>   			#mux-control-cells = <1>;
> -			mux-reg-masks = <0x4080 0x3>, /* SERDES0 lane select */
> -					<0x4090 0x3>; /* SERDES1 lane select */
> +			mux-reg-masks = <0x0 0x3>, /* SERDES0 lane select */
> +					<0x10 0x3>; /* SERDES1 lane select */
>   		};
>   
>   		dss_oldi_io_ctrl: dss-oldi-io-ctrl@41e0 {