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 3A4D1ECAAD3 for ; Tue, 6 Sep 2022 00:24:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: 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=IHJfCo6YsQIA5K0G+YcEFjSec3606YL7h3h3aa2oVic=; b=h+2rat+7093/m+ 9pvv4y/oesf6Iqpeu2KSL5A+nX3iZw+JjUhXKUJwQz1db4OooyWpCscsiO/FvUWN/bDzxGS6V1CYt iYy+905/unKx8o/y/HbgRtkogp93woXy/bjB7PrQkvFhpNTr6EC+slEbsKtKk5EmJRffqzrJlawbZ rzrVFPsFEmXuA8EKuu0e7Q64dwrMYRdakxHSnDCd+gUORbvtA3q9mxKQVTW4GTQ7pfsa8DnQX0vKI 2cEKU8U8ffHBPNbx9UEPzUeEDse95kY8q+WqlipGFFODIys68UU00F8PEJEYEHXDw4fig9o3RzbDO LyrahGhx9nAYer0oalxA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVMNR-000iMo-VI; Tue, 06 Sep 2022 00:23:42 +0000 Received: from vps0.lunn.ch ([185.16.172.187]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVMNM-000iJi-Ut for linux-arm-kernel@lists.infradead.org; Tue, 06 Sep 2022 00:23:38 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=d9OXgRNebu5UPVkhM/ybQh6k0eihy+EGudxgU/QSYdI=; b=LaY2GSFeaQvgTZ/4J74fhAgY23 krIGialM91/ySLoO1qK0qWDfDzlsY7PcVXmFvDXFiY2ZEvRtb2f1ZIrjwRO/ySkqdNsQNrhi29efN dEaJi6stegK0+Xcaa3kkU1f+C0KfqyIrcNipRMXfv3Gx+AfufaKCibiF9Rvuw1FP6rek=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1oVMMc-00FhWk-LL; Tue, 06 Sep 2022 02:22:50 +0200 Date: Tue, 6 Sep 2022 02:22:50 +0200 From: Andrew Lunn To: rentao.bupt@gmail.com Cc: "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Heyi Guo , Dylan Hung , Guangbin Huang , Liang He , Hao Chen , Rob Herring , Krzysztof Kozlowski , Joel Stanley , Andrew Jeffery , Tao Ren , netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next 2/2] ARM: dts: aspeed: elbert: Enable mac3 controller Message-ID: References: <20220905235634.20957-1-rentao.bupt@gmail.com> <20220905235634.20957-3-rentao.bupt@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220905235634.20957-3-rentao.bupt@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220905_172337_030495_98C0E4B2 X-CRM114-Status: GOOD ( 18.03 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Sep 05, 2022 at 04:56:34PM -0700, rentao.bupt@gmail.com wrote: > From: Tao Ren > > Enable mac3 controller in Elbert dts: Elbert MAC3 is connected to the > onboard switch directly (fixed link). What is the switch? Could you also add a DT node for it? > > Signed-off-by: Tao Ren > --- > arch/arm/boot/dts/aspeed-bmc-facebook-elbert.dts | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-elbert.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-elbert.dts > index 27b43fe099f1..52cb617783ac 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-facebook-elbert.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-elbert.dts > @@ -183,3 +183,14 @@ imux31: i2c@7 { > &i2c11 { > status = "okay"; > }; > + > +&mac3 { > + status = "okay"; > + phy-mode = "rgmii"; 'rgmii' is suspicious, though not necessarily wrong. This value is normally passed to the PHY, so the PHY inserts the RGMII delay. You however don't have a PHY. So i assume the switch is inserting the delay? Again, being able to see the DT properties for the switch would be useful. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel