Received: by 2002:ab2:7a55:0:b0:1f4:4a7d:290d with SMTP id u21csp592366lqp; Fri, 5 Apr 2024 03:09:23 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCUxJQTgVHn/7rFmMPjli5tKNUIWHtkwNZTS44ziisv9uu5BFrBovRfYWcFe/4/T7OK0NBPlrRRDwY7+quB95NlR5zK/aOPKxForeFIBjA== X-Google-Smtp-Source: AGHT+IHKaYzm48AFbsPve0vXKa8ReqTwqNxVVGp53Q72GNW0SqBPnsYC5ZzWBcg4Cz57d/pM4W3l X-Received: by 2002:a25:db46:0:b0:dcd:a9ad:7d67 with SMTP id g67-20020a25db46000000b00dcda9ad7d67mr941503ybf.8.1712311762726; Fri, 05 Apr 2024 03:09:22 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1712311762; cv=pass; d=google.com; s=arc-20160816; b=Az4jFnOnVXrXKTZmxfdU9YkFQvYOR6E39JZ3TZvz6pOM3vjrZc1WbYUIwVD9ZHxs92 TrgbLyoF3HQnSNRE/naJIGhgpJkn+PvYL4u8npAkR59dBKCI5Eb9oBfB+3wYpsrG3f7X 6tel8fkoUQub78wv9wgPJ9zDr48gw5/pP/8DmKsRg1rJleqzHcpYHkpaRihrxLvsG0HA BqG6IsngAVgmAxncxxzGIcVJff+ctcIrd5qzgxd2s1xep71q8E8t6UF8blD0qAp/v87n zFN8RYyrrzl9B8TxCO26GMvfLVUqzZZlQvdsXqbO74kr2czXW41WvK7V6Z+9+nOoJPXc RYXw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date; bh=k2kvQRXljepvFpMUm2drHHSYcqh7pu5arfznjUtb1Q8=; fh=DM9C/5eXKzw1iYTAj9l/UQ/jYWFNsencHJEC1HXdzzo=; b=T2RW0koTgWCz/KgTCkmJOQKrJq444BgHnvNEIj75e3LPWUO5JeB+l5A5dNkB3UIf/z BPmM6Sp/Xc7y+ung6ou+vYFamLR/UQUZZsfNG0MNUJdyK1Lj6NY2CwTkrLt9liGHgmQo EOU81sB7JXJ6W78I3HG6cXkhkD7ffLleUG4i+RJHma2IC6AZO6/CTD75SXEbLmtDW+0W pOA1nr29Y+wbsWyDragQJHPqNYTs2H9o8S4Py+LcLXY9CTg/aJn/u/TlPhFcV+GB1hta gCxEhucp23oT2FopROvgxab/7IGcXwvX8iAE/Aoc+Eig55/LLdee2ofryGe6seBHzZOD cMYA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=pengutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-132794-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-132794-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id g3-20020ac87f43000000b00432c71df373si1442859qtk.448.2024.04.05.03.09.22 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Apr 2024 03:09:22 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-132794-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=pengutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-132794-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-132794-linux.lists.archive=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 3F8A21C2037D for ; Fri, 5 Apr 2024 10:09:22 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5720D161907; Fri, 5 Apr 2024 10:09:19 +0000 (UTC) Received: from metis.whiteo.stw.pengutronix.de (metis.whiteo.stw.pengutronix.de [185.203.201.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6F50D16132A for ; Fri, 5 Apr 2024 10:09:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.203.201.7 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712311758; cv=none; b=nkkOsycVUjjpERi+7b9YFGI9K4XjNEnx9JdG/CyzTRJS15DVAMeMFUnZ77lbgegxnfbDxew2dAzgwz/r2WHJXzt4ygiPAiiGg6fZgkvzPK6wikBqAc+I5bjqW2FLDTYyYW93ldhLbt57dSQSkvC2TAmCcCI+BG9tXYDdavBl/+U= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712311758; c=relaxed/simple; bh=3z1V1yFkK7QVYQRREAv0f7Hj3YGHVm1xuW1J6axhZSM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=MAk6DHYPJAmGjE+VOkoe8Xpe5Kd1C2dsMAqJKf6PfPuFax+BcCIfcXGpD+zsEu5ZAYch/cUREE2RlUKJjwD5EZKzGYj0shZbH6J8FzC65cNehf1+S7GT/Ls3BSed727NEMELGNlKN/P1ixH24nwjqfAag/W512XVPqWmYaq6wDw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de; spf=pass smtp.mailfrom=pengutronix.de; arc=none smtp.client-ip=185.203.201.7 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pengutronix.de Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1rsgVO-0003n8-Ol; Fri, 05 Apr 2024 12:09:06 +0200 Received: from [2a0a:edc0:2:b01:1d::c5] (helo=pty.whiteo.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rsgVN-00AY48-7T; Fri, 05 Apr 2024 12:09:05 +0200 Received: from sha by pty.whiteo.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1rsgVN-00Fea4-0P; Fri, 05 Apr 2024 12:09:05 +0200 Date: Fri, 5 Apr 2024 12:09:05 +0200 From: Sascha Hauer To: Andrew Lunn Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, David Jander Subject: Re: [PATCH 2/2] arm64: dts: rockchip: add Protonic MECSBC device-tree Message-ID: References: <20240404-protonic-mecsbc-v1-0-ad5b42ade6c6@pengutronix.de> <20240404-protonic-mecsbc-v1-2-ad5b42ade6c6@pengutronix.de> <9d325b4e-031c-4f6c-9788-fa5a68470efa@lunn.ch> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9d325b4e-031c-4f6c-9788-fa5a68470efa@lunn.ch> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-SA-Exim-Connect-IP: 2a0a:edc0:0:c01:1d::a2 X-SA-Exim-Mail-From: sha@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org On Thu, Apr 04, 2024 at 05:10:41PM +0200, Andrew Lunn wrote: > > +&gmac1 { > > + assigned-clocks = <&cru SCLK_GMAC1_RX_TX>, <&cru SCLK_GMAC1>; > > + assigned-clock-parents = <&cru SCLK_GMAC1_RGMII_SPEED>, <&cru CLK_MAC1_2TOP>; > > + phy-handle = <&rgmii_phy1>; > > + phy-mode = "rgmii"; > > + clock_in_out = "output"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&gmac1m1_miim > > + &gmac1m1_tx_bus2 > > + &gmac1m1_rx_bus2 > > + &gmac1m1_rgmii_clk > > + &gmac1m1_clkinout > > + &gmac1m1_rgmii_bus>; > > + status = "okay"; > > + tx_delay = <0x30>; > > + rx_delay = <0x10>; > > +}; > > There was a discussion about phy-mode = "rgmii"; and these > tx/rx_delays last month. Please could you go read that discussion and > them make use of rgmii-id, and change the delays. Ok, I'll switch to rgmii-id. > > Also, where did you copy this from? If possible, it would be good to > fix the example everybody copies into new DT blobs. These are the default values used in over a dozen boards and a also given in the example in Documentation/devicetree/bindings/net/rockchip-dwmac.yaml. These are also the default values the driver uses when tx_delay and rx_delay are not given in the device tree. I can prepare a patch to fix the example. Do you have a pointer why setting the delays in the phy is preferred over setting them in the network driver? In the end this requires us to have the correct phy driver whereas setting them in the network driver would just work for any phy driver? Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |