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 4BF8BC2BD09 for ; Wed, 3 Jul 2024 09:36:39 +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:In-Reply-To: Content-Transfer-Encoding:Content-Type: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=T3YRRV0u13l2lvIEvP9DrDF1Hy6ZLNQ46QLq5OnUiKA=; b=fLsFGfDLq4DYibQtm6AuLU4cR6 e3pbT+5AiITxa/GeKtEgmrhNH/zw+UErh5DfhkYrY2bHdaVhU5IJxO45yN6zECRlz6w8iaeCXGMMM /l+TYdOkku+hfeiDhOupvOcKsiq/68cEnbbdizLLXUBrKWATuF0rvGjuj4tIdO8nMZZFHClDIuswt Ipa+BChljphObLAfldt2L58zsYLtn5tzSAHOo5UeZkO9Yx/8mrhZZXCkhyyVvJQ/wTJV1rHquaw1I slgPzUcVYSgrlrq2djdTRdlwVNMWR6rAILp8jZPgyBKLebGsJUcDVht3abLg7LcjWXYeK2KRFDibT Rl/wtfVQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOwPa-00000009a1n-0gql; Wed, 03 Jul 2024 09:36:26 +0000 Received: from fhigh2-smtp.messagingengine.com ([103.168.172.153]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOwPO-00000009ZzA-1uOP for linux-arm-kernel@lists.infradead.org; Wed, 03 Jul 2024 09:36:16 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailfhigh.nyi.internal (Postfix) with ESMTP id CC35E1140210; Wed, 3 Jul 2024 05:36:10 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute6.internal (MEProxy); Wed, 03 Jul 2024 05:36:10 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ragnatech.se; h= cc:cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm1; t=1719999370; x=1720085770; bh=T3YRRV0u13l2lvIEvP9DrDF1Hy6ZLNQ46QLq5OnUiKA=; b= s40Vr57dSDcQ4UZlMxgcu5W6mbq3hFdCgkjxv+WxKVXIjlaBFf71gXXaEUWcjR5T Jmx8lbeBosbXvuY3ANiEM5i4GJPa8i3YNnaY8DUUr+S4aLAveH8hiJxGh4AmGrsz J89J0K7innS9OXbri6Nt/4uh/l87rXmxQTCmq09YP7twG+rCZ29X3Dn89aEWjrB8 UrJyrWJCyoO/Imn0qjaekRVuEhIVtf336FzwGNVNBwUT8JsImgeV7LA38ZiF4CCv elAQn7BThYQoiy3TfgbR/YIw00jWMNgtuAxM6q3CrmUJMNoIVyN3DWN/W+QvwLmJ 36tRFZ7fWGQf9V45IoB9OA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t=1719999370; x= 1720085770; bh=T3YRRV0u13l2lvIEvP9DrDF1Hy6ZLNQ46QLq5OnUiKA=; b=Z LE2TK+ev5ZD1UuLhf3h7OtoCOoVvRrIMIE5mIpTqMQd9pBvjXv5kwNqVdMkNK3na /dU39U+l9m9weypHDPchVlc1PNINhqVhJTx+CyXUKfm9V9K2okyHNl/eZy0RswGn wdZA3jRL/h+ygqQydJpPCdJixw0SU7Vj7crLzz0L2RAB8kAVHTF2sm7SzqCU5Buw aGORBaOY6JvAZ5ZmLxm35BikaqqxIHry42LTbW8Rw4m5buPKl9PcmDmXSychLa7P huXkKXsaMDscRaDgF/MZs5NlknkF3GeGjiOg60o+HNQhxK3zwWShn1WAiSCjat2T y2Pm999emHvQAQCFRH95Q== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeeftddrudejgddukecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvvefukfhfgggtugfgjgesthekredttddtjeenucfhrhhomheppfhikhhl rghsucfunpguvghrlhhunhguuceonhhikhhlrghsrdhsohguvghrlhhunhguodhrvghnvg hsrghssehrrghgnhgrthgvtghhrdhsvgeqnecuggftrfgrthhtvghrnhepfefhleelhfff jefgfedugfegjeelhfevheeikefhueelgfdtfeeuhefftddvleeinecuvehluhhsthgvrh fuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepnhhikhhlrghsrdhsohguvghr lhhunhguodhrvghnvghsrghssehrrghgnhgrthgvtghhrdhsvg X-ME-Proxy: Feedback-ID: i80c9496c:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 3 Jul 2024 05:36:09 -0400 (EDT) Date: Wed, 3 Jul 2024 11:36:01 +0200 From: Niklas =?utf-8?Q?S=C3=B6derlund?= To: Geert Uytterhoeven Cc: Marek Vasut , linux-arm-kernel@lists.infradead.org, andrew@lunn.ch, kernel@dh-electronics.com, kernel test robot , Conor Dooley , Khuong Dinh , Krzysztof Kozlowski , Magnus Damm , Rob Herring , devicetree@vger.kernel.org, linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH 2/2] arm64: dts: renesas: Drop ethernet-phy-ieee802.3-c22 from PHY compatible string on all RZ boards Message-ID: <20240703093601.GA6493@ragnatech.se> References: <20240630034649.173229-1-marex@denx.de> <20240630034649.173229-2-marex@denx.de> <9f1ae430-4cc4-4e2e-a52c-ca17f499bbba@denx.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240703_023614_769344_7DA293F3 X-CRM114-Status: GOOD ( 12.16 ) 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 2024-07-03 10:24:26 +0200, Geert Uytterhoeven wrote: > Niklas: commit 54bf0c27380b95a2 ("arm64: dts: renesas: r8a779g0: Use > MDIO node for all AVB devices") did keep the reset-gpios property in > the PHY node. I guess it should be moved one level up? It's possible to have a rest-gpios property both in the mdio node and the phy node. The former resets the whole bus while the later a single PHY, at least that's my understanding. I think it would be more correct to move the reset-gpios up one level for r8a779g0. However as it already was in the PHY node and this functioned as it should I kept it there. The need for the mdio node was to avoid a device specific property added in the BSP to reset the whole bus. At the moment I can't recall the two different call sites for when the two resets are called. Maybe if we move it to the mdio node we can avoid setting the PHY id in the compatible string as we could then always probe the PHY correctly. I will look into it. -- Kind Regards, Niklas Söderlund