From patchwork Thu Jan 23 16:37:10 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Geert Uytterhoeven X-Patchwork-Id: 13948446 X-Patchwork-Delegate: geert@linux-m68k.org Received: from albert.telenet-ops.be (albert.telenet-ops.be [195.130.137.90]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2854820F96D for ; Thu, 23 Jan 2025 16:37:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=195.130.137.90 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737650244; cv=none; b=cm+rgiDlrwPMkoXrAPboQ6axkMB0v/cdC4rMn/QKPb6oDHxTjze/6An6H/ZxJEfz+NqxjRVaGc7UKsFcRsN/hZ3iP0IbQihkeVBXsQCjE2aBiXs33w3PrlV1n4up/6YFqPxJYKohBMkQieF2cY97uSRgtbrwKsGKHE2KorI/MJ8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737650244; c=relaxed/simple; bh=oovoM9/NfpO+tA/KvHQz6ixKY4kGN3JA9GPNgic5Chw=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=UXiYBd7polDiKIj9czj5s98OFEhzJBjr1Q2q03YIkFpGOmdqv1h1kfZKGxJO24RAWF52/JSBAbd5/H4bMDUMR0F0uHVznO7GvgYjjZ4iGfL31chd9+HuuAKfHXskoscWPQb8dRAQduWOM01ZpixdgXzRdp9fqV9Amx0pAoC0qS0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=glider.be; spf=none smtp.mailfrom=linux-m68k.org; arc=none smtp.client-ip=195.130.137.90 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=glider.be Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux-m68k.org Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed80:4563:f165:f885:ebc1]) by albert.telenet-ops.be with cmsmtp id 4gdB2E00F5PrELR06gdBEk; Thu, 23 Jan 2025 17:37:13 +0100 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtp (Exim 4.97) (envelope-from ) id 1tb0CU-0000000EEZY-3aR7; Thu, 23 Jan 2025 17:37:11 +0100 Received: from geert by rox.of.borg with local (Exim 4.97) (envelope-from ) id 1tb0Cd-00000000mL3-24G3; Thu, 23 Jan 2025 17:37:11 +0100 From: Geert Uytterhoeven To: Magnus Damm , Marek Vasut , Yoshihiro Shimoda , Kuninori Morimoto , =?utf-8?q?Niklas_S?= =?utf-8?q?=C3=B6derlund?= Cc: linux-renesas-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, Geert Uytterhoeven Subject: [PATCH] arm64: dts: renesas: r8a779f0: Disable rswitch ports by default Date: Thu, 23 Jan 2025 17:37:10 +0100 Message-ID: X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-renesas-soc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 The Renesas Ethernet Switch has three independent ports. Each port can act as a separate interface, and can be enabled or disabled independently. Currently all ports are enabled by default, hence board DTS files that enable the switch must disable all unused ports explicitly. Disable all ports by default, and explicitly enable ports that are used, next to their configuration. Signed-off-by: Geert Uytterhoeven Reviewed-by: Niklas Söderlund Reviewed-by: Marek Vasut --- This is similar to how Ethernet switch ports are handled on RZ/N1D, and CAN-FD channels on R-Car Gen3/4. If accepted, I'm wondering if I should move it forward, before "[PATCH 3/5] arm64: dts: renesas: s4sk: Access rswitch ports via phandles"[1], which would become slightly simpler. [1] https://lore.kernel.org/all/20250118111344.361617-3-marek.vasut+renesas@mailbox.org --- arch/arm64/boot/dts/renesas/r8a779f0-spider-ethernet.dtsi | 3 +++ arch/arm64/boot/dts/renesas/r8a779f0.dtsi | 3 +++ arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts | 6 ++---- 3 files changed, 8 insertions(+), 4 deletions(-) diff --git a/arch/arm64/boot/dts/renesas/r8a779f0-spider-ethernet.dtsi b/arch/arm64/boot/dts/renesas/r8a779f0-spider-ethernet.dtsi index 892729786d00b36e..ad2b0398d35431a1 100644 --- a/arch/arm64/boot/dts/renesas/r8a779f0-spider-ethernet.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a779f0-spider-ethernet.dtsi @@ -57,6 +57,7 @@ &rswitch_port0 { phy-handle = <&u101>; phy-mode = "sgmii"; phys = <ð_serdes 0>; + status = "okay"; mdio { #address-cells = <1>; @@ -75,6 +76,7 @@ &rswitch_port1 { phy-handle = <&u201>; phy-mode = "sgmii"; phys = <ð_serdes 1>; + status = "okay"; mdio { #address-cells = <1>; @@ -93,6 +95,7 @@ &rswitch_port2 { phy-handle = <&u301>; phy-mode = "sgmii"; phys = <ð_serdes 2>; + status = "okay"; mdio { #address-cells = <1>; diff --git a/arch/arm64/boot/dts/renesas/r8a779f0.dtsi b/arch/arm64/boot/dts/renesas/r8a779f0.dtsi index 840ba73c03f1b6f4..06be9f7720d64179 100644 --- a/arch/arm64/boot/dts/renesas/r8a779f0.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a779f0.dtsi @@ -989,14 +989,17 @@ ethernet-ports { rswitch_port0: port@0 { reg = <0>; phys = <ð_serdes 0>; + status = "disabled"; }; rswitch_port1: port@1 { reg = <1>; phys = <ð_serdes 1>; + status = "disabled"; }; rswitch_port2: port@2 { reg = <2>; phys = <ð_serdes 2>; + status = "disabled"; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts b/arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts index 24ecdf6e8476a945..df652e7b85c95ebe 100644 --- a/arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts +++ b/arch/arm64/boot/dts/renesas/r8a779f4-s4sk.dts @@ -187,6 +187,7 @@ &rswitch_port0 { phy-handle = <&ic99>; phy-mode = "sgmii"; phys = <ð_serdes 0>; + status = "okay"; mdio { #address-cells = <1>; @@ -205,6 +206,7 @@ &rswitch_port1 { phy-handle = <&ic102>; phy-mode = "sgmii"; phys = <ð_serdes 1>; + status = "okay"; mdio { #address-cells = <1>; @@ -218,10 +220,6 @@ ic102: ethernet-phy@2 { }; }; -&rswitch_port2 { - status = "disabled"; -}; - &rwdt { timeout-sec = <60>; status = "okay";