From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-dl1-f44.google.com (mail-dl1-f44.google.com [74.125.82.44]) (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 4D9262F290A for ; Mon, 20 Apr 2026 06:58:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776668331; cv=none; b=OReFZM/b/kvIWefjUaWCWuQy7/qTECydzollEvd/1nc6noHHII1nqlxUTglOSgGAifWRk/xInFWjywtr9jmQwxtnH9p1YhhyZxL77Z6th2KxjP+urfDsdoyMrsT9Z8Wmc6eYI6fVkNZYjB0SpOPoHA4XqEcqZFLKVpw8fTEofSQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776668331; c=relaxed/simple; bh=4jXkQwGliVjo5HQ8xpKkQLE97BY4mpvsOew8INsIQk0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=jL9aEVJgYFN3GiQEOB0hIibM8YLHdTsaLa4K30JcW46h5/4cZ7Zx5Ex2rn0d9Z6XqeWVtKPMhCXCkTPydw01MvuVTUEXKjRZMqvx51R43GBWTvcKicpB6i6PzRchZaAVy95u2DCXL0eqvCGW2Dw1VIXZsdJ1uWUX8Hu1Ei7cJ1g= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=XYxp7XX9; arc=none smtp.client-ip=74.125.82.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="XYxp7XX9" Received: by mail-dl1-f44.google.com with SMTP id a92af1059eb24-12713e56abdso1508300c88.1 for ; Sun, 19 Apr 2026 23:58:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1776668329; x=1777273129; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to:subject :user-agent:mime-version:date:message-id:from:to:cc:subject:date :message-id:reply-to; bh=T3r02hhZx3Vxqyj1q31IQgqGPj+UcnZ1kBemjU+jdaU=; b=XYxp7XX9CUfsbbSfbeDSXVNWeWn///m+EdCkx8eVmJt4ddO2FYnbsxLXlxK/b0vfv8 Y/Pq17fvUM3XhrTIbRNJAS53b9Gpa9F0h4IRqPC5LXcHFIehi0F1nPUj1WWZUSBtkAqY w7nphHcv0sAbXTRtT2owBbnMbX0OQTNMKK6yn+W6JD3CWXtGDL8Pq67Pcl6+a4o7vONq 0nrtuLQahOxaDOf9ZNFdDiO+a/UJ3XWk2gYgqGIL7aMQ/4YGA1T3Et/8O/ILizRaEhAO enUtDYcfhQVbz5ftRSTEu7KsN7+v6A+M7cV0XKKJV3S0JzqShMyWH6MKhD+XtaSgtkMu nHJg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776668329; x=1777273129; h=content-transfer-encoding:in-reply-to:from:references:cc:to:subject :user-agent:mime-version:date:message-id:x-gm-gg:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=T3r02hhZx3Vxqyj1q31IQgqGPj+UcnZ1kBemjU+jdaU=; b=OxHSxmMi/zNI0o+kCnrIdz4cQeT+sKHAIqPe8coUV3JMGxgnaIhI6CRdof0bi6KbaB lAovwF17PanSns7p11r3jKzyzhr8DTd3RROA9HNgxfaWoEVK0fn78hdBsujBPZ1pln2Q FIevYMbuK3nHDjq3/lpkQTtlx3ZQPAd2YI0hO9K5zIfJx9wHFadpsmrIC1DyquTvkSf/ GcqDTrk2if7ZMO+CbOUcMd3cuFTnHbN4xLL2XVQDWes5KBe1kWJTHqfIhw6yRT6GLMkk 2Rsd2Azp9Jg/eY4OrAYlxNe9knwtvIzWOu/eGeR66ZZ+/nBOt4iWrNM28f2Xo/YlPX/t unUg== X-Forwarded-Encrypted: i=1; AFNElJ8Q4IMyJ26iECDXcgMCKL6Uqd92c5imps8GFAbdlHIn3KjfF9O/kXnllW607DRekQcnmJvNrje174jA3PI=@vger.kernel.org X-Gm-Message-State: AOJu0YyjjGNndMjzF6GBAFnLSI5tyAcZPYOzunHreeRP8psrbh8vzepB ABgsNd8N5P4ohLLVukhPb9sgb1gJsjIY01HHIXNnH/ZyrielMwi90qra X-Gm-Gg: AeBDieu2dHjSqntWlMb1+MRmAjETIelgbr4NVwgCllZ/YFh/EDCsUhylL1pA3dYLEyU irAS893lyMr2zV0VWWUUOySljZS/3DhxkskP8/D/RQS5IIUCTYiA+xWt8UBuMfsNIeHJCqBMaBq vayfpm+NhvuF3YA45wNWnjBbkrLSCzAyHaCyk5KapUJNHwMmxMp6sok2ldv5TmxscJhFpscKYtp bcIpWbqWaZF2vY9pQhfMASfKygrkIgeVB0RAzmgw4ENUwl0eI+8+0EXO8faQpRTXqmUZxYZQlOc skoE8xAsUeD1cK49Sh2dKLtt1yd4xCZabTz2x7TKSb8+q5rcXTWuhiCLxKwvyRrb8cSpOH4lGWq evLa3WGSuiPDv9jqjLwN336xxSnXjGJ+UA9rcBwF6TkxfUvfMtoqUqpgBZeSNROwrzulUSK/l0I hhQbxiV6yI X-Received: by 2002:a05:7022:4191:b0:12c:427:dacf with SMTP id a92af1059eb24-12c73b05ceamr5070810c88.10.1776668329484; Sun, 19 Apr 2026 23:58:49 -0700 (PDT) Received: from CNSZTL-PC.lan ([2607:f130:0:13b::3a]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-12c74a20b9csm13380651c88.12.2026.04.19.23.58.46 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 19 Apr 2026 23:58:49 -0700 (PDT) Message-ID: <2d2b1e17-388f-431a-be86-a0f26b5be6cf@gmail.com> Date: Mon, 20 Apr 2026 14:58:44 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/2] arm64: dts: rockchip: Replace deprecated snps,* props for NanoPi R5S To: Diederik de Haas , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner Cc: Arnd Bergmann , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Quentin Schulz , Jonas Karlman References: <20260401131551.734456-1-diederik@cknow-tech.com> <20260401131551.734456-3-diederik@cknow-tech.com> From: Tianling Shen In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 2026/4/15 22:23, Diederik de Haas wrote: > On Wed Apr 1, 2026 at 3:11 PM CEST, Diederik de Haas wrote: >> The various snps,reset-* properties are deprecated, so convert them into >> their replacements. >> >> Signed-off-by: Diederik de Haas >> --- >> arch/arm64/boot/dts/rockchip/rk3568-nanopi-r5s.dts | 7 +++---- >> 1 file changed, 3 insertions(+), 4 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3568-nanopi-r5s.dts b/arch/arm64/boot/dts/rockchip/rk3568-nanopi-r5s.dts >> index 90ce6f0e1dcf..92d044ec696b 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3568-nanopi-r5s.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3568-nanopi-r5s.dts >> @@ -85,10 +85,6 @@ &gmac0_tx_bus2 >> &gmac0_rx_bus2 >> &gmac0_rgmii_clk >> &gmac0_rgmii_bus>; >> - snps,reset-gpio = <&gpio0 RK_PC5 GPIO_ACTIVE_LOW>; >> - snps,reset-active-low; >> - /* Reset time is 15ms, 50ms for rtl8211f */ >> - snps,reset-delays-us = <0 15000 50000>; >> tx_delay = <0x3c>; >> rx_delay = <0x2f>; >> status = "okay"; >> @@ -100,6 +96,9 @@ rgmii_phy0: ethernet-phy@1 { >> reg = <1>; >> pinctrl-0 = <&gmac0_rstn_gpio0_c5_pin>; >> pinctrl-names = "default"; >> + reset-assert-us = <15000>; >> + reset-deassert-us = <50000>; >> + reset-gpios = <&gpio0 RK_PC5 GPIO_ACTIVE_LOW>; >> }; >> }; >> > > Please disregard/drop this patch. > > I was recently made aware of 'sashiko.dev' and checked whether it had > also checked my patch, which it did: > https://sashiko.dev/#/patchset/20260401131551.734456-1-diederik%40cknow-tech.com > > And it turns out that the concern raised is valid (thanks Quentin!), so > this patch could introduce a regression. > So it looks like staying with the deprecated properties is actually > better (in this case?). Well actually we more or less rely on U-Boot to reset the PHY first now. Many rockchip boards in tree require a reset before the PHY can be recognized, but we just use the generic "ethernet-phy-ieee802.3-c22" compatible. Another option is to move the reset props to mdio node instead of PHY node, though. Thanks, Tianling. > > Cheers, > Diederik