From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5DC24C369C2 for ; Fri, 25 Apr 2025 17:33:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=C4pfyAQIOAg1ZR9eaHE8W8n+mwoN6/LuLVOaQtXOgLo=; b=X0+Xj2uOBcpUeJn00q9UEQob02 cnM9/Wf5sWCjFrg6o32JRLHOcQS8YhNysgkieUUhuXgteil+5KCsedB+UUUn+qd5vJhwMtrYEOH1B rSQghvRtVhuZZ2Bqv24gHLvygF9yleNLW7uHdGM36/TYcKcwZq/uRsqdws+P0/szjgrG66zy1SS+i pdUxk2DOAYd5o481ejefmGn8DsVAyVfyX91U1NebG1c7PJX9PmUTpwsA76gpfxXACFUXWCmK4zkTN FAZYxSGROIbgu7d8pwzrD2VQ4bK6Z5ILyCN+0tHAHq90xAZtl28WkSCZVYu7X92yc4I9SoOTQc7I5 XxBrvW8Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1u8Mvq-00000000P01-0Xvp; Fri, 25 Apr 2025 17:33:46 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1u8KcZ-0000000HYxW-056f for linux-arm-kernel@lists.infradead.org; Fri, 25 Apr 2025 15:05:44 +0000 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 9847D106F; Fri, 25 Apr 2025 08:05:36 -0700 (PDT) Received: from donnerap.manchester.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 1867B3F59E; Fri, 25 Apr 2025 08:05:38 -0700 (PDT) Date: Fri, 25 Apr 2025 16:05:35 +0100 From: Andre Przywara To: Chen-Yu Tsai , Linus Walleij Cc: Yixun Lan , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jernej Skrabec , Samuel Holland , Maxime Ripard , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Corentin Labbe , , , , , Subject: Re: [PATCH v2 3/5] arm64: dts: allwinner: a523: Add EMAC0 ethernet MAC Message-ID: <20250425160535.5a18adbb@donnerap.manchester.arm.com> In-Reply-To: References: <20250424-01-sun55i-emac0-v2-0-833f04d23e1d@gentoo.org> <20250424-01-sun55i-emac0-v2-3-833f04d23e1d@gentoo.org> <20250425104128.14f953f3@donnerap.manchester.arm.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=UTF-8 Content-Transfer-Encoding: quoted-printable X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250425_080543_151108_6164C4AB X-CRM114-Status: GOOD ( 27.33 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 25 Apr 2025 22:35:59 +0800 Chen-Yu Tsai wrote: adding LinusW for a more generic pinctrl question ... > On Fri, Apr 25, 2025 at 5:41=E2=80=AFPM Andre Przywara wrote: > > > > On Fri, 25 Apr 2025 13:26:25 +0800 > > Chen-Yu Tsai wrote: > > > > Hi Chen-Yu, > > =20 > > > On Thu, Apr 24, 2025 at 6:09=E2=80=AFPM Yixun Lan w= rote: =20 > > > > > > > > Add EMAC0 ethernet MAC support which found on A523 variant SoCs, > > > > including the A527/T527 chips. MAC0 is compatible to the A64 chip w= hich > > > > requires an external PHY. This patch only add RGMII pins for now. > > > > > > > > Signed-off-by: Yixun Lan > > > > --- > > > > arch/arm64/boot/dts/allwinner/sun55i-a523.dtsi | 40 ++++++++++++++= ++++++++++++ > > > > 1 file changed, 40 insertions(+) > > > > > > > > diff --git a/arch/arm64/boot/dts/allwinner/sun55i-a523.dtsi b/arch/= arm64/boot/dts/allwinner/sun55i-a523.dtsi > > > > index ee485899ba0af69f32727a53de20051a2e31be1d..c9a9b9dd479af05ba22= fe9d783e32f6d61a74ef7 100644 > > > > --- a/arch/arm64/boot/dts/allwinner/sun55i-a523.dtsi > > > > +++ b/arch/arm64/boot/dts/allwinner/sun55i-a523.dtsi > > > > @@ -126,6 +126,15 @@ pio: pinctrl@2000000 { > > > > interrupt-controller; > > > > #interrupt-cells =3D <3>; > > > > > > > > + rgmii0_pins: rgmii0-pins { > > > > + pins =3D "PH0", "PH1", "PH2", "PH3"= , "PH4", > > > > + "PH5", "PH6", "PH7", "PH9", = "PH10", > > > > + "PH14", "PH15", "PH16", "PH1= 7", "PH18"; > > > > + allwinner,pinmux =3D <5>; > > > > + function =3D "emac0"; > > > > + drive-strength =3D <40>; =20 > > > > > > We should probably add > > > > > > bias-disable; > > > > > > to explicitly turn off pull-up and pull-down. =20 > > > > Should we? I don't see this anywhere else for sunxi, probably because i= t is > > the (reset) default (0b00). > > I wonder if we have a hidden assumption about this? As in: if no bias is > > specified, we assume bias-disable? Then we should maybe enforce this is= in > > the driver? =20 >=20 > There isn't any assumption, as in we were fine with either the reset > default or whatever the bootloader left it in. However in projects at > work I learned that it's better to have explicit settings despite > working defaults. I totally agree, but my point was that this applies basically to every pinctrl user. I usually think of the bias settings as "do we need pull-ups or pull-downs", and if nothing is specified, I somewhat assume bias-disable. So I am fine with this being added here, but was wondering if we should look at a more generic solution. Linus: is bias-disable assumed to be the default, that pinctrl drivers should set in absence of explicit properties? Or is this "whatever is in the registers at boot" the default we have to live with? Cheers, Andre