Received: by 2002:a6b:500f:0:0:0:0:0 with SMTP id e15csp4915188iob; Mon, 9 May 2022 04:47:55 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxMXRPZSu3W1wyRHioMKaBJrzI35jzQUyxtKhOsvdv7l2JfXwlQTG1OdtyfT1VJlxM3Kgnb X-Received: by 2002:a65:6e82:0:b0:381:71c9:9856 with SMTP id bm2-20020a656e82000000b0038171c99856mr13087145pgb.316.1652096875391; Mon, 09 May 2022 04:47:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1652096875; cv=none; d=google.com; s=arc-20160816; b=Hp5vvMZSz7IBCxKUl976iOqmtxCYNkcyD/9oNuFc8wg4rgSvMLj+UvoLmfM+riGtqs lE53+YOyhrzdWPQ69JbFaD95RCKjRTW1Q6UalMVV5qfM83MNMTY7bYfqb0mXvZXIau5D 9zusfrNd4IXvi4bFodsdViYxI/TrhWc69PtWz2QPegczk4BR+guNJxpi69SbMCI4NUu+ ns9FvDcBAsDaSKqqnkDXfkEzDNaOHEPAJ3ReRSvNOISXlJAdrbgqX+CmuZn5XdIKoJ3f M9LoYjoQjISQ3V3sGZrnA3ewInOy+iTdRmV2liVLBalFw4lPmq+Zh6lJn4gS2uS6gdc9 ZiSg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :organization:references:in-reply-to:message-id:subject:cc:to:from :date; bh=MavPZChZfYV5o2aLsm9sk8kfM6Lz+RDSe2zgm4KPv54=; b=EH+eD82JS5w14GPmZKg/jS362WRDUOWebjSWVNxnmuMzJqoCQ6fZJ1wXCRT4XW6Lfm /RTM6yPpXwMgln5TODXYI8ZiqbUEZzASlMd3n8o1yhIbyheId/CccyGUJAt+ESGyrj3p MEP97bIug7EYuu8zNEKJ7ywfpoE3Lyv1Gd5cwrxMzBj9DgWeYshNRYJ/+zpqnFVMuTYi cOXM8iH11gGhR+vKeymd/VFik607mEssvQ9GGMmDQXEKq3M+Zol6h/m1IXURDRu1K1If RMhJvZQwmWVBIlsNdWTPM7Ll4/+lhFMrjg4qEg/CSNHf4pv+2prS4h+cwlU0f8lYNwF/ tr9Q== ARC-Authentication-Results: i=1; mx.google.com; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [23.128.96.19]) by mx.google.com with ESMTPS id t191-20020a6381c8000000b003ab1f09b881si13787683pgd.514.2022.05.09.04.47.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 09 May 2022 04:47:55 -0700 (PDT) Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19; Authentication-Results: mx.google.com; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id A6A4E2229FC; Mon, 9 May 2022 03:56:01 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229885AbiEIK7r (ORCPT + 99 others); Mon, 9 May 2022 06:59:47 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35496 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230230AbiEIK7i (ORCPT ); Mon, 9 May 2022 06:59:38 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id CAE221FC2EA; Mon, 9 May 2022 03:55:39 -0700 (PDT) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 23B421480; Mon, 9 May 2022 03:55:39 -0700 (PDT) Received: from donnerap.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 35BCB3F66F; Mon, 9 May 2022 03:55:36 -0700 (PDT) Date: Mon, 9 May 2022 11:55:33 +0100 From: Andre Przywara To: Corentin Labbe Cc: alexandre.torgue@foss.st.com, andrew@lunn.ch, broonie@kernel.org, calvin.johnson@oss.nxp.com, davem@davemloft.net, edumazet@google.com, hkallweit1@gmail.com, jernej.skrabec@gmail.com, joabreu@synopsys.com, krzysztof.kozlowski+dt@linaro.org, kuba@kernel.org, lgirdwood@gmail.com, linux@armlinux.org.uk, pabeni@redhat.com, peppe.cavallaro@st.com, robh+dt@kernel.org, samuel@sholland.org, wens@csie.org, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@lists.linux.dev Subject: Re: [PATCH 4/6] ARM: dts: sunxi: move phy regulator in PHY node Message-ID: <20220509115533.1493db30@donnerap.cambridge.arm.com> In-Reply-To: <20220509074857.195302-5-clabbe@baylibre.com> References: <20220509074857.195302-1-clabbe@baylibre.com> <20220509074857.195302-5-clabbe@baylibre.com> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RDNS_NONE, SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 9 May 2022 07:48:55 +0000 Corentin Labbe wrote: Hi! > Now that PHY core can handle regulators, move regulator handle in PHY > node. Other than this is somewhat more "correct", is it really needed for those boards? Because it breaks compatibility with older kernels, so when we update the DTs in U-Boot, we run into problems (again). IIUC this series is about the OPi3 & friends, which didn't work with older kernels anyway, so can we just skip this patch (and 5/6), to just enable the boards that didn't work before? Cheers, Andre > > Signed-off-by: Corentin Labbe > --- > arch/arm/boot/dts/sun8i-a83t-bananapi-m3.dts | 2 +- > arch/arm/boot/dts/sun8i-a83t-cubietruck-plus.dts | 2 +- > arch/arm/boot/dts/sun8i-h3-nanopi-m1-plus.dts | 2 +- > arch/arm/boot/dts/sun8i-h3-nanopi-r1.dts | 2 +- > arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts | 2 +- > arch/arm/boot/dts/sun8i-h3-orangepi-plus2e.dts | 2 +- > arch/arm/boot/dts/sun8i-h3-zeropi.dts | 2 +- > arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts | 2 +- > arch/arm/boot/dts/sun8i-r40-oka40i-c.dts | 2 +- > arch/arm/boot/dts/sun8i-v40-bananapi-m2-berry.dts | 2 +- > arch/arm/boot/dts/sun9i-a80-cubieboard4.dts | 2 +- > arch/arm/boot/dts/sun9i-a80-optimus.dts | 2 +- > arch/arm/boot/dts/sunxi-bananapi-m2-plus.dtsi | 2 +- > 13 files changed, 13 insertions(+), 13 deletions(-) > > diff --git a/arch/arm/boot/dts/sun8i-a83t-bananapi-m3.dts b/arch/arm/boot/dts/sun8i-a83t-bananapi-m3.dts > index 5a7e1bd5f825..b450be0a45ed 100644 > --- a/arch/arm/boot/dts/sun8i-a83t-bananapi-m3.dts > +++ b/arch/arm/boot/dts/sun8i-a83t-bananapi-m3.dts > @@ -129,7 +129,6 @@ &ehci0 { > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_sw>; > phy-handle = <&rgmii_phy>; > phy-mode = "rgmii-id"; > allwinner,rx-delay-ps = <700>; > @@ -151,6 +150,7 @@ &mdio { > rgmii_phy: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <1>; > + phy-supply = <®_sw>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun8i-a83t-cubietruck-plus.dts b/arch/arm/boot/dts/sun8i-a83t-cubietruck-plus.dts > index 870993393fc2..fe70b350cdbb 100644 > --- a/arch/arm/boot/dts/sun8i-a83t-cubietruck-plus.dts > +++ b/arch/arm/boot/dts/sun8i-a83t-cubietruck-plus.dts > @@ -181,7 +181,6 @@ &ehci1 { > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_dldo4>; > phy-handle = <&rgmii_phy>; > phy-mode = "rgmii-id"; > status = "okay"; > @@ -201,6 +200,7 @@ &mdio { > rgmii_phy: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <1>; > + phy-supply = <®_dldo4>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun8i-h3-nanopi-m1-plus.dts b/arch/arm/boot/dts/sun8i-h3-nanopi-m1-plus.dts > index a2f2ef2b0092..c393612f44c6 100644 > --- a/arch/arm/boot/dts/sun8i-h3-nanopi-m1-plus.dts > +++ b/arch/arm/boot/dts/sun8i-h3-nanopi-m1-plus.dts > @@ -103,7 +103,6 @@ &ehci2 { > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > phy-mode = "rgmii"; > > @@ -114,6 +113,7 @@ &external_mdio { > ext_rgmii_phy: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <7>; > + phy-supply = <®_gmac_3v3>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun8i-h3-nanopi-r1.dts b/arch/arm/boot/dts/sun8i-h3-nanopi-r1.dts > index 26e2e6172e0d..70bde396856b 100644 > --- a/arch/arm/boot/dts/sun8i-h3-nanopi-r1.dts > +++ b/arch/arm/boot/dts/sun8i-h3-nanopi-r1.dts > @@ -80,7 +80,6 @@ &ehci2 { > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > phy-mode = "rgmii-id"; > status = "okay"; > @@ -90,6 +89,7 @@ &external_mdio { > ext_rgmii_phy: ethernet-phy@7 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <7>; > + phy-supply = <®_gmac_3v3>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts > index d05fa679dcd3..c6dcf1af3298 100644 > --- a/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts > +++ b/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts > @@ -83,7 +83,6 @@ &ehci3 { > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > phy-mode = "rgmii-id"; > > @@ -94,6 +93,7 @@ &external_mdio { > ext_rgmii_phy: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <0>; > + phy-supply = <®_gmac_3v3>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-plus2e.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-plus2e.dts > index b6ca45d18e51..61eb8c003186 100644 > --- a/arch/arm/boot/dts/sun8i-h3-orangepi-plus2e.dts > +++ b/arch/arm/boot/dts/sun8i-h3-orangepi-plus2e.dts > @@ -65,7 +65,6 @@ reg_gmac_3v3: gmac-3v3 { > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > phy-mode = "rgmii-id"; > status = "okay"; > @@ -75,5 +74,6 @@ &external_mdio { > ext_rgmii_phy: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <1>; > + phy-supply = <®_gmac_3v3>; > }; > }; > diff --git a/arch/arm/boot/dts/sun8i-h3-zeropi.dts b/arch/arm/boot/dts/sun8i-h3-zeropi.dts > index 7d3e7323b661..54174ef18823 100644 > --- a/arch/arm/boot/dts/sun8i-h3-zeropi.dts > +++ b/arch/arm/boot/dts/sun8i-h3-zeropi.dts > @@ -65,13 +65,13 @@ &external_mdio { > ext_rgmii_phy: ethernet-phy@7 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <7>; > + phy-supply = <®_gmac_3v3>; > }; > }; > > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > phy-mode = "rgmii-id"; > > diff --git a/arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts b/arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts > index a6a1087a0c9b..b1f269bbd479 100644 > --- a/arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts > +++ b/arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts > @@ -130,7 +130,6 @@ &gmac { > pinctrl-0 = <&gmac_rgmii_pins>; > phy-handle = <&phy1>; > phy-mode = "rgmii-id"; > - phy-supply = <®_dc1sw>; > status = "okay"; > }; > > @@ -138,6 +137,7 @@ &gmac_mdio { > phy1: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <1>; > + phy-supply = <®_dc1sw>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun8i-r40-oka40i-c.dts b/arch/arm/boot/dts/sun8i-r40-oka40i-c.dts > index 0bd1336206b8..c43476b426df 100644 > --- a/arch/arm/boot/dts/sun8i-r40-oka40i-c.dts > +++ b/arch/arm/boot/dts/sun8i-r40-oka40i-c.dts > @@ -93,7 +93,6 @@ &gmac { > pinctrl-0 = <&gmac_rgmii_pins>; > phy-handle = <&phy1>; > phy-mode = "rgmii-id"; > - phy-supply = <®_dcdc1>; > status = "okay"; > }; > > @@ -101,6 +100,7 @@ &gmac_mdio { > phy1: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <1>; > + phy-supply = <®_dcdc1>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun8i-v40-bananapi-m2-berry.dts b/arch/arm/boot/dts/sun8i-v40-bananapi-m2-berry.dts > index 47954551f573..050a649d7bda 100644 > --- a/arch/arm/boot/dts/sun8i-v40-bananapi-m2-berry.dts > +++ b/arch/arm/boot/dts/sun8i-v40-bananapi-m2-berry.dts > @@ -121,7 +121,6 @@ &gmac { > pinctrl-0 = <&gmac_rgmii_pins>; > phy-handle = <&phy1>; > phy-mode = "rgmii-id"; > - phy-supply = <®_dc1sw>; > status = "okay"; > }; > > @@ -129,6 +128,7 @@ &gmac_mdio { > phy1: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <1>; > + phy-supply = <®_dc1sw>; > }; > }; > > diff --git a/arch/arm/boot/dts/sun9i-a80-cubieboard4.dts b/arch/arm/boot/dts/sun9i-a80-cubieboard4.dts > index c8ca8cb7f5c9..ab9bf4bf7343 100644 > --- a/arch/arm/boot/dts/sun9i-a80-cubieboard4.dts > +++ b/arch/arm/boot/dts/sun9i-a80-cubieboard4.dts > @@ -130,7 +130,6 @@ &gmac { > pinctrl-0 = <&gmac_rgmii_pins>; > phy-handle = <&phy1>; > phy-mode = "rgmii-id"; > - phy-supply = <®_cldo1>; > status = "okay"; > }; > > @@ -142,6 +141,7 @@ &i2c3 { > > &mdio { > phy1: ethernet-phy@1 { > + phy-supply = <®_cldo1>; > reg = <1>; > }; > }; > diff --git a/arch/arm/boot/dts/sun9i-a80-optimus.dts b/arch/arm/boot/dts/sun9i-a80-optimus.dts > index 5c3580d712e4..48219b8049b1 100644 > --- a/arch/arm/boot/dts/sun9i-a80-optimus.dts > +++ b/arch/arm/boot/dts/sun9i-a80-optimus.dts > @@ -125,13 +125,13 @@ &gmac { > pinctrl-0 = <&gmac_rgmii_pins>; > phy-handle = <&phy1>; > phy-mode = "rgmii-id"; > - phy-supply = <®_cldo1>; > status = "okay"; > }; > > &mdio { > phy1: ethernet-phy@1 { > reg = <1>; > + phy-supply = <®_cldo1>; > }; > }; > > diff --git a/arch/arm/boot/dts/sunxi-bananapi-m2-plus.dtsi b/arch/arm/boot/dts/sunxi-bananapi-m2-plus.dtsi > index d03f5853ef7b..65f0a3c2af3f 100644 > --- a/arch/arm/boot/dts/sunxi-bananapi-m2-plus.dtsi > +++ b/arch/arm/boot/dts/sunxi-bananapi-m2-plus.dtsi > @@ -125,7 +125,6 @@ &ehci2 { > &emac { > pinctrl-names = "default"; > pinctrl-0 = <&emac_rgmii_pins>; > - phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > phy-mode = "rgmii-id"; > > @@ -136,6 +135,7 @@ &external_mdio { > ext_rgmii_phy: ethernet-phy@1 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <1>; > + phy-supply = <®_gmac_3v3>; > }; > }; >