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 X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4E1F5C432BE for ; Mon, 30 Aug 2021 20:49:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2735860ED6 for ; Mon, 30 Aug 2021 20:49:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235949AbhH3Uuf (ORCPT ); Mon, 30 Aug 2021 16:50:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52728 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229708AbhH3Uue (ORCPT ); Mon, 30 Aug 2021 16:50:34 -0400 Received: from mail-ej1-x636.google.com (mail-ej1-x636.google.com [IPv6:2a00:1450:4864:20::636]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A42BDC061575 for ; Mon, 30 Aug 2021 13:49:40 -0700 (PDT) Received: by mail-ej1-x636.google.com with SMTP id u3so33819384ejz.1 for ; Mon, 30 Aug 2021 13:49:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=QwGrx3DZXcDFxxZinjWxnatOl75Xen4FtkBd/Z3DLhI=; b=Sl9vplX5DLdVbrtFEFZTDDYKI71oUji9/VLjm7L93tkMzl54Dx7zV8IHdaz7/uHsrN /d/7Pkm0zAexE22PnCkp1aoy24E5IojeuEFTPSBbJTi2V3NrC53iUIN4f0fwUc2mJDU2 rD+IO2iRyOiETDEuhuZ8G1AQZhUP0UfO/ZcS8+KDZ5uZNJV/7t6j7AhzZXQCeVCgJ4Jo RkH2C/8vfbu6CU7ZM77JbnWa2CCzTZHrMR8Rj4po1uUYMi2NDfi9ryo4Tj2xBdcwzqWM tigAIC3LAL2KkcRmz5NBmPGticq33ya88WtHaSPBs5F1RqjTYaZU36mSNu9usE4OZfqI 012g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=QwGrx3DZXcDFxxZinjWxnatOl75Xen4FtkBd/Z3DLhI=; b=MI4+hK7XY1ki+qEZ3Oi6PqdHrXOvaeZthHcqUkFGunSKW6qHkNKiHwsOtGGKDCeRAm adGJaHF2IXFKcDQ2ZcqoxGAs4A+SC+PNAJ4FRn9WGGUp++jMYZAXt5M9dPdJDbNSFBgO CrQ9mYp3iGeQ6GBKIe7q6EG+zH7wlGHIhX9Gqr/8WfHX7JmHbclvaGvQVZDoAUi9Wuu7 7p4KNtLc/m3caBkWDxwqM4p57H452rDAHjBge+u80JKKakKJcSyTQ/OyirGmpkGcOgHv gIjqGmHmRE2RtXjeo7k3pYjz5PiD1iEi3sSuoBwpjK/MSM3MKTycKPREqa5gk6OxIchs Bgsg== X-Gm-Message-State: AOAM532HMAMrKI8WQCN9veDsYvP7Mj4+JG7g1su5rq1b1HalX64CXrs4 V5DwYHd8rqpVCN0LSovOGZw= X-Google-Smtp-Source: ABdhPJyH022G1mTKT1wDyArDNInyC7uX4awGVpVzyYmdox3yySaa0R+0EfEbw3xseEJ2hRSFG9TGEw== X-Received: by 2002:a17:907:7848:: with SMTP id lb8mr27593494ejc.494.1630356579251; Mon, 30 Aug 2021 13:49:39 -0700 (PDT) Received: from jernej-laptop.localnet (cpe-86-58-29-253.static.triera.net. [86.58.29.253]) by smtp.gmail.com with ESMTPSA id s23sm8345620eds.12.2021.08.30.13.49.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Aug 2021 13:49:38 -0700 (PDT) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Maxime Ripard , Chen-Yu Tsai , =?utf-8?B?Q2zDqW1lbnQgQsWTc2No?= Cc: Willy Liu , Rob Herring , "David S. Miller" , Andrew Lunn , linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, =?utf-8?B?Q2zDqW1lbnQgQsWTc2No?= Subject: Re: [PATCH] arm64: dts: sun50i: h5: NanoPI Neo 2: phy-mode rgmii-id Date: Mon, 30 Aug 2021 22:49:37 +0200 Message-ID: <116454729.UZi3dMzWh7@jernej-laptop> In-Reply-To: <20210830151645.18018-1-u@pkh.me> References: <20210830151645.18018-1-u@pkh.me> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi! Dne ponedeljek, 30. avgust 2021 ob 17:16:45 CEST je Cl=C3=A9ment B=C5=93sch= napisal(a): > Since commit bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay > config") network is broken on the NanoPi Neo 2. >=20 > This patch changes the phy-mode to use internal delays both for RX and > TX as has been done for other boards affected by the same commit. >=20 > Fixes: bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay config") This commit fixes DT issue, so "fixes" tag should be: =46ixes: 44a94c7ef989 ("arm64: dts: allwinner: H5: Restore EMAC changes") Here, a node with wrong phy-mode property was added to NanoPi Neo 2 board D= T. =20 Other than that, this patch is fine and once fixes tag is fixed, you can ad= d: Reviewed-by: Jernej Skrabec =46or next version, you should: =2D change fixed tag =2D add my review-by tag right above your signed-off-by tag =2D mark patch as v2 (add "-v2" parameter to git format-patch) =2D describe change right under "---" line Note, if you borked something when sending, you should mark patch or patch= =20 series as "RESEND", so recipients don't look for changes in two subsequent = e- mails (--subject-prefix=3D"RESEND PATCH"). Thanks for the fix! Best regards, Jernej > Signed-off-by: Cl=C3=A9ment B=C5=93sch > --- > arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >=20 > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts index > 02f8e72f0cad..05486cccee1c 100644 > --- a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > +++ b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > @@ -75,7 +75,7 @@ &emac { > pinctrl-0 =3D <&emac_rgmii_pins>; > phy-supply =3D <®_gmac_3v3>; > phy-handle =3D <&ext_rgmii_phy>; > - phy-mode =3D "rgmii"; > + phy-mode =3D "rgmii-id"; > status =3D "okay"; > };