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 2BAC0CA5FFC for ; Tue, 6 Oct 2026 16:14:06 +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=C8wch9WjDUqzsgZiqy7IMvtDNpZ7CUulCuk5xqtXDNU=; b=Xk0gDg+6oCKnsq 5O7QMXjP6b3id3rHgcc5orq1nol0azNFGhBqJz2Rc735Y2c2G0UuwZN+SVf6qLp8Xd2AaWZHyaO8i BvBzQGzQzcRzetb/c+EEmu48gDhQn4O4HVjcupJ2v4r0t1HynIbVWXticsylrX+wU9/3URbuJeC6u 49uVILoV7HWdIkI1fBeFv17YPHlQrS9ouFllHZDo/Lqso0etpqgMZDsvFUWvGxVT+bdsbO7BLIpV1 9HIH8STVCRXSQCT7jQJZFNujI0hfgR9oy4uDO0SN+LVXjg6OX4PtDctJG9MtCxpFUbCPAkXU3xMi6 +yoBqftP5oTjdXyFLYTw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xE7np-000000017KH-32wG; Tue, 06 Oct 2026 16:14:05 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xE7np-000000017K1-0Fxr; Tue, 06 Oct 2026 16:14:05 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 7F94E43CC4; Tue, 6 Oct 2026 16:14:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 330131F0089B; Tue, 6 Oct 2026 16:14:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791303244; bh=txP3ruVYCLkbMDGy07Vcni7w37L1LO/1WObGN9fDR6g=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=g1xdCTZWc+2keXIczvFXqEioig8hAdmMVgPi0UUdiJ1jt/m+HcLtmBHFVGhNbyH6P v2CxRkYvuRtEcGfObj3btZB08LnhgnVJ4OtTSNV/kFLzcMOjhTei9WhhWVKT7ISpBk 5v0UuzTVRln7IpACNwOxWxRIAAc/E3SjyElDIZCE08/xHh8WV2Y5rfiCUf3sEAIFzA imIC8VzpHbrfCc+he7/NinmJ5gUcaF8GlEStOrKf0l4hE3lq9XwUOwaEkaSUfGZf16 DuHsrWUkTS3a3pUwLA8mSPPsir6UiTFxPUXw4sAG3y0j0+9TkgmXJsDkO9ZQJWDXTH /rHc88pYFlhhQ== Date: Tue, 6 Oct 2026 11:14:03 -0500 From: Rob Herring To: Shawn Lin Cc: Coia Prant , Neil Armstrong , Manivannan Sadhasivam , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Vinod Koul Subject: Re: [PATCH v12 2/2] phy: rockchip: naneng-combphy: add SGMII MAC selection for RK3568 Message-ID: <20261006161403.GA2469169-robh@kernel.org> References: <20261006074749.2010233-1-coiaprant@gmail.com> <20261006074749.2010233-3-coiaprant@gmail.com> <08736f4d-a728-4b1e-a28e-882648a412e0@rock-chips.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <08736f4d-a728-4b1e-a28e-882648a412e0@rock-chips.com> X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org T24gVHVlLCBPY3QgMDYsIDIwMjYgYXQgMTA6NTM6MjJQTSArMDgwMCwgU2hhd24gTGluIHdyb3Rl Ogo+IEhpIENvaWEKPiAKPiDlnKggMjAyNi8xMC8wNiDmmJ/mnJ/kuowgMTU6NDcsIENvaWEgUHJh bnQg5YaZ6YGTOgo+ID4gT24gUkszNTY4LCB0aGUgU0dNSUkgaW50ZXJmYWNlIGNhbiBiZSByb3V0 ZWQgdG8gZWl0aGVyIEdNQUMwIG9yCj4gPiBHTUFDMSB2aWEgdGhlIEdSRiByZWdpc3RlciBwaXBl X3NnbWlpX21hY19zZWwuCj4gPiAKPiA+IEFkZCBzdXBwb3J0IGZvciB0aGlzIHNlbGVjdGlvbiBi eSBpbnRyb2R1Y2luZwo+ID4gdGhlICJyb2NrY2hpcCxzZ21paS1tYWMtc2VsIiBEVCBwcm9wZXJ0 eS4KPiA+IAo+ID4gIEZyb20gdGhlIFJLMzU2OCBUUk0gKFBhcnQxLCBQYWdlIDIyOSksIHRoZSBQ SVBFX0dSRl9YUENTX0NPTjAKPiA+IGJpdCAxIChwaXBlX3NnbWlpX21hY19zZWwpIGlzIGRlZmlu ZWQgYXM6Cj4gPiAKPiA+ICAgICAgMDogU0dNSUkgcm91dGVkIHRvIEdNQUMwCj4gPiAgICAgIDE6 IFNHTUlJIHJvdXRlZCB0byBHTUFDMQo+ID4gCj4gPiBUaGUgaGFyZHdhcmUgcmVzZXQgdmFsdWUg aXMgMSAoR01BQzEpLiBJZiB0aGUgcHJvcGVydHkgaXMgc2V0IHRvIDAsCj4gPiB0aGUgZHJpdmVy IHJvdXRlcyBTR01JSSB0byBHTUFDMDsgaWYgc2V0IHRvIDEgKG9yIG9taXR0ZWQpLCBpdAo+ID4g cmVtYWlucyBhdCBHTUFDMS4KPiA+IAo+IAo+IFRoYW5rcyBmb3IgeW91ciBwYXRjaC4gSSBoYXZl IGEgcXVlc3Rpb24gYmVsb3csIHBsZWFzZSBoYXZlIGEgbG9vay4KPiAKPiA+IFRoaXMgaXMgbmVj ZXNzYXJ5IGZvciBib2FyZHMgc3VjaCBhcyB0aGUgQXJpYWJvYXJkIFBob3RvbmljYXQsIHdoaWNo Cj4gPiB1c2VzIHRoZSBTR01JSSBpbnRlcmZhY2UgY29ubmVjdGVkIHRvIEdNQUMwLgo+ID4gCj4g PiBPdXQtb2YtcmFuZ2UgdmFsdWVzIGFyZSByZWplY3RlZCBieSBkdHNjaGVtYSwgc28gdGhlIGRy aXZlciBkb2VzIG5vdAo+ID4gZHVwbGljYXRlIHRoZSByYW5nZSBjaGVjay4KPiA+IAo+ID4gTGlu azogaHR0cHM6Ly9kbC5yYWR4YS5jb20vcm9jazMvZG9jcy9ody9kYXRhc2hlZXQvUm9ja2NoaXAl MjBSSzM1NjglMjBUUk0lMjBQYXJ0MSUyMFYxLjEtMjAyMTAzMDEucGRmIChQYWdlIDIyOSkKPiA+ IFNpZ25lZC1vZmYtYnk6IENvaWEgUHJhbnQgPGNvaWFwcmFudEBnbWFpbC5jb20+Cj4gPiAtLS0K PiA+ICAgZHJpdmVycy9waHkvcm9ja2NoaXAvcGh5LXJvY2tjaGlwLW5hbmVuZy1jb21icGh5LmMg fCA4ICsrKysrKysrCj4gPiAgIDEgZmlsZSBjaGFuZ2VkLCA4IGluc2VydGlvbnMoKykKPiA+IAo+ ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvcGh5L3JvY2tjaGlwL3BoeS1yb2NrY2hpcC1uYW5lbmct Y29tYnBoeS5jIGIvZHJpdmVycy9waHkvcm9ja2NoaXAvcGh5LXJvY2tjaGlwLW5hbmVuZy1jb21i cGh5LmMKPiA+IGluZGV4IDc4NDMzNTZhNGRkNDcuLjdiODY3ZTc1MjAwNjQgMTAwNjQ0Cj4gPiAt LS0gYS9kcml2ZXJzL3BoeS9yb2NrY2hpcC9waHktcm9ja2NoaXAtbmFuZW5nLWNvbWJwaHkuYwo+ ID4gKysrIGIvZHJpdmVycy9waHkvcm9ja2NoaXAvcGh5LXJvY2tjaGlwLW5hbmVuZy1jb21icGh5 LmMKPiA+IEBAIC0xODYsNiArMTg2LDcgQEAgc3RydWN0IHJvY2tjaGlwX2NvbWJwaHlfZ3JmY2Zn IHsKPiA+ICAgCXN0cnVjdCBjb21icGh5X3JlZyBwaXBlX3hwY3NfcGh5X3JlYWR5Owo+ID4gICAJ c3RydWN0IGNvbWJwaHlfcmVnIHBpcGVfcGNpZTFsMF9zZWw7Cj4gPiAgIAlzdHJ1Y3QgY29tYnBo eV9yZWcgcGlwZV9wY2llMWwxX3NlbDsKPiA+ICsJc3RydWN0IGNvbWJwaHlfcmVnIHBpcGVfc2dt aWlfbWFjX3NlbDsKPiA+ICAgCXN0cnVjdCBjb21icGh5X3JlZyB1M290ZzBfcG9ydF9lbjsKPiA+ ICAgCXN0cnVjdCBjb21icGh5X3JlZyB1M290ZzFfcG9ydF9lbjsKPiA+ICAgfTsKPiA+IEBAIC0y MTIsNiArMjEzLDcgQEAgc3RydWN0IHJvY2tjaGlwX2NvbWJwaHlfcHJpdiB7Cj4gPiAgIAlib29s IGVuYWJsZV9zc2M7Cj4gPiAgIAlib29sIGV4dF9yZWZjbGs7Cj4gPiAgIAlzdHJ1Y3QgY2xrICpy ZWZjbGs7Cj4gPiArCXUzMiBzZ21paV9tYWNfc2VsOwo+ID4gICB9Owo+ID4gICBzdGF0aWMgdm9p ZCByb2NrY2hpcF9jb21icGh5X3VwZGF0ZWwoc3RydWN0IHJvY2tjaGlwX2NvbWJwaHlfcHJpdiAq cHJpdiwKPiA+IEBAIC0zNzUsNiArMzc3LDkgQEAgc3RhdGljIGludCByb2NrY2hpcF9jb21icGh5 X3BhcnNlX2R0KHN0cnVjdCBkZXZpY2UgKmRldiwgc3RydWN0IHJvY2tjaGlwX2NvbWJwaHkKPiA+ ICAgCXByaXYtPmV4dF9yZWZjbGsgPSBkZXZpY2VfcHJvcGVydHlfcHJlc2VudChkZXYsICJyb2Nr Y2hpcCxleHQtcmVmY2xrIik7Cj4gPiArCXByaXYtPnNnbWlpX21hY19zZWwgPSAxOwo+IAo+IEJl Zm9yZSB0aGlzIHBhdGNoLCBTR01NSSB3YXMgcm91dGVkIHRvIEdNQUNYIGRlcGVuZHMgb24gYm9v dGxvYWRlcihpZgo+IHNldHRpbmcpLCBvciByb3V0ZWQgdG8gR01BQzEgYnkgY2hpcCBkZWZhdWx0 LiBXaXRob3V0IGtub3dpbmcgYW55Cj4gcG90ZW50aWFsIGNhbmRpZGF0ZXMgc2V0dGluZyBpdCB2 aWEgYm9vdGxvYWRlciwgeW91IG1pZ2h0IG5vdCBmb3JjZSBhbnkKPiBEVFMgZmlsZXMgd2l0aG91 dCAicm9ja2NoaXAsc2dtaWktbWFjLXNlbCIgdG8gcGljayBHTUFDMSBhcyBmYWxsYmFjay4KPiAK PiBNYXliZSB5b3UgY291bGQganVzdCBrZWVwIHRoZSBTR01JSSByb3V0ZSBhcy1pdCBpZiB0aGUg cHJvcGVydHkgaXMgbm90Cj4gcHJlc2VudC4gTWVhbndoaWxlLCBzaG91bGQgdmFsaWRhdGUgcm9j a2NoaXAsc2dtaWktbWFjLXNlbCBpcyBhIHZhbGlkCj4gdmFsdWUgcGFzc2luZyBmcm9tIERULgoK VGhhdCBpcyB0aGUgdHlwaWNhbCB3YXkgdHJpLXN0YXRlIHByb3BlcnRpZXMgd29yayAoYXMgb3Bw b3NlZCB0byB1c2luZyBhIApib29sZWFuKS4gSW4gdGhhdCBjYXNlLCBubyAnZGVmYXVsdCcgaW4g dGhlIHNjaGVtYSB3b3VsZCB0aGVuIGJlIAphcHByb3ByaWF0ZS4KClJvYgoKLS0gCmxpbnV4LXBo eSBtYWlsaW5nIGxpc3QKbGludXgtcGh5QGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cHM6Ly9saXN0 cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtcGh5Cg== 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 0C1D4CA5FED for ; Tue, 6 Oct 2026 16:14:15 +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=txP3ruVYCLkbMDGy07Vcni7w37L1LO/1WObGN9fDR6g=; b=OAOz1TEY+Yh6qQGf/qr3rO6oZo 6ydaGxmmaUyLjwq7LVGQJlAiNyJRU+ZFEFWdcoc3CBVxUwu6PJNa3++OdNzN0nlNusvw2pMV9XoYE 5iuGmhnCgWt7lD2xKcMvlOACL2NAU0+FfhVeJK4Q/J/uK+Jr05CrFMmCwlU0kb8e9OXuOnVzPd6uS uF3kv4pIL2jjEZo6N3YNwDKFyZyzkn+njDaCt12aPrLXOdLLtT8qBgddR6nUw/30l0f5PR3Kdnjf+ axu6DWfz5J27q7MwBEcchSmkMT+cXGvgUKuBZnAQDRbtFe12VXpitU7Aecu8MMSg2Es4tCEUIiFDO iTQNWIIA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xE7np-000000017KD-2iU6; Tue, 06 Oct 2026 16:14:05 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xE7np-000000017K1-0Fxr; Tue, 06 Oct 2026 16:14:05 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 7F94E43CC4; Tue, 6 Oct 2026 16:14:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 330131F0089B; Tue, 6 Oct 2026 16:14:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791303244; bh=txP3ruVYCLkbMDGy07Vcni7w37L1LO/1WObGN9fDR6g=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=g1xdCTZWc+2keXIczvFXqEioig8hAdmMVgPi0UUdiJ1jt/m+HcLtmBHFVGhNbyH6P v2CxRkYvuRtEcGfObj3btZB08LnhgnVJ4OtTSNV/kFLzcMOjhTei9WhhWVKT7ISpBk 5v0UuzTVRln7IpACNwOxWxRIAAc/E3SjyElDIZCE08/xHh8WV2Y5rfiCUf3sEAIFzA imIC8VzpHbrfCc+he7/NinmJ5gUcaF8GlEStOrKf0l4hE3lq9XwUOwaEkaSUfGZf16 DuHsrWUkTS3a3pUwLA8mSPPsir6UiTFxPUXw4sAG3y0j0+9TkgmXJsDkO9ZQJWDXTH /rHc88pYFlhhQ== Date: Tue, 6 Oct 2026 11:14:03 -0500 From: Rob Herring To: Shawn Lin Cc: Coia Prant , Neil Armstrong , Manivannan Sadhasivam , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Vinod Koul Subject: Re: [PATCH v12 2/2] phy: rockchip: naneng-combphy: add SGMII MAC selection for RK3568 Message-ID: <20261006161403.GA2469169-robh@kernel.org> References: <20261006074749.2010233-1-coiaprant@gmail.com> <20261006074749.2010233-3-coiaprant@gmail.com> <08736f4d-a728-4b1e-a28e-882648a412e0@rock-chips.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <08736f4d-a728-4b1e-a28e-882648a412e0@rock-chips.com> 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 Tue, Oct 06, 2026 at 10:53:22PM +0800, Shawn Lin wrote: > Hi Coia > > 在 2026/10/06 星期二 15:47, Coia Prant 写道: > > On RK3568, the SGMII interface can be routed to either GMAC0 or > > GMAC1 via the GRF register pipe_sgmii_mac_sel. > > > > Add support for this selection by introducing > > the "rockchip,sgmii-mac-sel" DT property. > > > > From the RK3568 TRM (Part1, Page 229), the PIPE_GRF_XPCS_CON0 > > bit 1 (pipe_sgmii_mac_sel) is defined as: > > > > 0: SGMII routed to GMAC0 > > 1: SGMII routed to GMAC1 > > > > The hardware reset value is 1 (GMAC1). If the property is set to 0, > > the driver routes SGMII to GMAC0; if set to 1 (or omitted), it > > remains at GMAC1. > > > > Thanks for your patch. I have a question below, please have a look. > > > This is necessary for boards such as the Ariaboard Photonicat, which > > uses the SGMII interface connected to GMAC0. > > > > Out-of-range values are rejected by dtschema, so the driver does not > > duplicate the range check. > > > > Link: https://dl.radxa.com/rock3/docs/hw/datasheet/Rockchip%20RK3568%20TRM%20Part1%20V1.1-20210301.pdf (Page 229) > > Signed-off-by: Coia Prant > > --- > > drivers/phy/rockchip/phy-rockchip-naneng-combphy.c | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/drivers/phy/rockchip/phy-rockchip-naneng-combphy.c b/drivers/phy/rockchip/phy-rockchip-naneng-combphy.c > > index 7843356a4dd47..7b867e7520064 100644 > > --- a/drivers/phy/rockchip/phy-rockchip-naneng-combphy.c > > +++ b/drivers/phy/rockchip/phy-rockchip-naneng-combphy.c > > @@ -186,6 +186,7 @@ struct rockchip_combphy_grfcfg { > > struct combphy_reg pipe_xpcs_phy_ready; > > struct combphy_reg pipe_pcie1l0_sel; > > struct combphy_reg pipe_pcie1l1_sel; > > + struct combphy_reg pipe_sgmii_mac_sel; > > struct combphy_reg u3otg0_port_en; > > struct combphy_reg u3otg1_port_en; > > }; > > @@ -212,6 +213,7 @@ struct rockchip_combphy_priv { > > bool enable_ssc; > > bool ext_refclk; > > struct clk *refclk; > > + u32 sgmii_mac_sel; > > }; > > static void rockchip_combphy_updatel(struct rockchip_combphy_priv *priv, > > @@ -375,6 +377,9 @@ static int rockchip_combphy_parse_dt(struct device *dev, struct rockchip_combphy > > priv->ext_refclk = device_property_present(dev, "rockchip,ext-refclk"); > > + priv->sgmii_mac_sel = 1; > > Before this patch, SGMMI was routed to GMACX depends on bootloader(if > setting), or routed to GMAC1 by chip default. Without knowing any > potential candidates setting it via bootloader, you might not force any > DTS files without "rockchip,sgmii-mac-sel" to pick GMAC1 as fallback. > > Maybe you could just keep the SGMII route as-it if the property is not > present. Meanwhile, should validate rockchip,sgmii-mac-sel is a valid > value passing from DT. That is the typical way tri-state properties work (as opposed to using a boolean). In that case, no 'default' in the schema would then be appropriate. Rob 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 4F7BDCA5FFC for ; Tue, 6 Oct 2026 16:14:10 +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=HEl5ZNeBluUIIuHzdPJlHoYF00cSu8oyAsJP6lVT2bI=; b=K7xHkFSzcWhGMC RQBbQvNMSureIySI61m5wWsfRTPZvoDjSqTw1AqlJyFhxDYc3lK1Jtyuubk6OTScwOYmMB4VgIMDJ W5Ak4Asu1JOHBfxyCWugXntMsthf2p+uYFGglUgMORbFdET5b3GBmuh90uACMYhcLUHtCN1wHmLvT Zf6dtj42DX4NVpkDy4zjIBOaX9lSM2HrsJaT40h4T8Xnn45LtvcYie/wAgboFoobU6EIokqypbpIO 6U0Ko39fbOBcUPlBxxAI5AnVwhOz/KdIcDQ3Jm8mFuSOBqwHpPM7zzlQ+PtSomgG4DJ07PdpOTFE8 e5HeQK1c0I9ezVtLnF/w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xE7np-000000017KL-3GGW; Tue, 06 Oct 2026 16:14:05 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xE7np-000000017K1-0Fxr; Tue, 06 Oct 2026 16:14:05 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 7F94E43CC4; Tue, 6 Oct 2026 16:14:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 330131F0089B; Tue, 6 Oct 2026 16:14:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791303244; bh=txP3ruVYCLkbMDGy07Vcni7w37L1LO/1WObGN9fDR6g=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=g1xdCTZWc+2keXIczvFXqEioig8hAdmMVgPi0UUdiJ1jt/m+HcLtmBHFVGhNbyH6P v2CxRkYvuRtEcGfObj3btZB08LnhgnVJ4OtTSNV/kFLzcMOjhTei9WhhWVKT7ISpBk 5v0UuzTVRln7IpACNwOxWxRIAAc/E3SjyElDIZCE08/xHh8WV2Y5rfiCUf3sEAIFzA imIC8VzpHbrfCc+he7/NinmJ5gUcaF8GlEStOrKf0l4hE3lq9XwUOwaEkaSUfGZf16 DuHsrWUkTS3a3pUwLA8mSPPsir6UiTFxPUXw4sAG3y0j0+9TkgmXJsDkO9ZQJWDXTH /rHc88pYFlhhQ== Date: Tue, 6 Oct 2026 11:14:03 -0500 From: Rob Herring To: Shawn Lin Cc: Coia Prant , Neil Armstrong , Manivannan Sadhasivam , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Vinod Koul Subject: Re: [PATCH v12 2/2] phy: rockchip: naneng-combphy: add SGMII MAC selection for RK3568 Message-ID: <20261006161403.GA2469169-robh@kernel.org> References: <20261006074749.2010233-1-coiaprant@gmail.com> <20261006074749.2010233-3-coiaprant@gmail.com> <08736f4d-a728-4b1e-a28e-882648a412e0@rock-chips.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <08736f4d-a728-4b1e-a28e-882648a412e0@rock-chips.com> X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org T24gVHVlLCBPY3QgMDYsIDIwMjYgYXQgMTA6NTM6MjJQTSArMDgwMCwgU2hhd24gTGluIHdyb3Rl Ogo+IEhpIENvaWEKPiAKPiDlnKggMjAyNi8xMC8wNiDmmJ/mnJ/kuowgMTU6NDcsIENvaWEgUHJh bnQg5YaZ6YGTOgo+ID4gT24gUkszNTY4LCB0aGUgU0dNSUkgaW50ZXJmYWNlIGNhbiBiZSByb3V0 ZWQgdG8gZWl0aGVyIEdNQUMwIG9yCj4gPiBHTUFDMSB2aWEgdGhlIEdSRiByZWdpc3RlciBwaXBl X3NnbWlpX21hY19zZWwuCj4gPiAKPiA+IEFkZCBzdXBwb3J0IGZvciB0aGlzIHNlbGVjdGlvbiBi eSBpbnRyb2R1Y2luZwo+ID4gdGhlICJyb2NrY2hpcCxzZ21paS1tYWMtc2VsIiBEVCBwcm9wZXJ0 eS4KPiA+IAo+ID4gIEZyb20gdGhlIFJLMzU2OCBUUk0gKFBhcnQxLCBQYWdlIDIyOSksIHRoZSBQ SVBFX0dSRl9YUENTX0NPTjAKPiA+IGJpdCAxIChwaXBlX3NnbWlpX21hY19zZWwpIGlzIGRlZmlu ZWQgYXM6Cj4gPiAKPiA+ICAgICAgMDogU0dNSUkgcm91dGVkIHRvIEdNQUMwCj4gPiAgICAgIDE6 IFNHTUlJIHJvdXRlZCB0byBHTUFDMQo+ID4gCj4gPiBUaGUgaGFyZHdhcmUgcmVzZXQgdmFsdWUg aXMgMSAoR01BQzEpLiBJZiB0aGUgcHJvcGVydHkgaXMgc2V0IHRvIDAsCj4gPiB0aGUgZHJpdmVy IHJvdXRlcyBTR01JSSB0byBHTUFDMDsgaWYgc2V0IHRvIDEgKG9yIG9taXR0ZWQpLCBpdAo+ID4g cmVtYWlucyBhdCBHTUFDMS4KPiA+IAo+IAo+IFRoYW5rcyBmb3IgeW91ciBwYXRjaC4gSSBoYXZl IGEgcXVlc3Rpb24gYmVsb3csIHBsZWFzZSBoYXZlIGEgbG9vay4KPiAKPiA+IFRoaXMgaXMgbmVj ZXNzYXJ5IGZvciBib2FyZHMgc3VjaCBhcyB0aGUgQXJpYWJvYXJkIFBob3RvbmljYXQsIHdoaWNo Cj4gPiB1c2VzIHRoZSBTR01JSSBpbnRlcmZhY2UgY29ubmVjdGVkIHRvIEdNQUMwLgo+ID4gCj4g PiBPdXQtb2YtcmFuZ2UgdmFsdWVzIGFyZSByZWplY3RlZCBieSBkdHNjaGVtYSwgc28gdGhlIGRy aXZlciBkb2VzIG5vdAo+ID4gZHVwbGljYXRlIHRoZSByYW5nZSBjaGVjay4KPiA+IAo+ID4gTGlu azogaHR0cHM6Ly9kbC5yYWR4YS5jb20vcm9jazMvZG9jcy9ody9kYXRhc2hlZXQvUm9ja2NoaXAl MjBSSzM1NjglMjBUUk0lMjBQYXJ0MSUyMFYxLjEtMjAyMTAzMDEucGRmIChQYWdlIDIyOSkKPiA+ IFNpZ25lZC1vZmYtYnk6IENvaWEgUHJhbnQgPGNvaWFwcmFudEBnbWFpbC5jb20+Cj4gPiAtLS0K PiA+ICAgZHJpdmVycy9waHkvcm9ja2NoaXAvcGh5LXJvY2tjaGlwLW5hbmVuZy1jb21icGh5LmMg fCA4ICsrKysrKysrCj4gPiAgIDEgZmlsZSBjaGFuZ2VkLCA4IGluc2VydGlvbnMoKykKPiA+IAo+ ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvcGh5L3JvY2tjaGlwL3BoeS1yb2NrY2hpcC1uYW5lbmct Y29tYnBoeS5jIGIvZHJpdmVycy9waHkvcm9ja2NoaXAvcGh5LXJvY2tjaGlwLW5hbmVuZy1jb21i cGh5LmMKPiA+IGluZGV4IDc4NDMzNTZhNGRkNDcuLjdiODY3ZTc1MjAwNjQgMTAwNjQ0Cj4gPiAt LS0gYS9kcml2ZXJzL3BoeS9yb2NrY2hpcC9waHktcm9ja2NoaXAtbmFuZW5nLWNvbWJwaHkuYwo+ ID4gKysrIGIvZHJpdmVycy9waHkvcm9ja2NoaXAvcGh5LXJvY2tjaGlwLW5hbmVuZy1jb21icGh5 LmMKPiA+IEBAIC0xODYsNiArMTg2LDcgQEAgc3RydWN0IHJvY2tjaGlwX2NvbWJwaHlfZ3JmY2Zn IHsKPiA+ICAgCXN0cnVjdCBjb21icGh5X3JlZyBwaXBlX3hwY3NfcGh5X3JlYWR5Owo+ID4gICAJ c3RydWN0IGNvbWJwaHlfcmVnIHBpcGVfcGNpZTFsMF9zZWw7Cj4gPiAgIAlzdHJ1Y3QgY29tYnBo eV9yZWcgcGlwZV9wY2llMWwxX3NlbDsKPiA+ICsJc3RydWN0IGNvbWJwaHlfcmVnIHBpcGVfc2dt aWlfbWFjX3NlbDsKPiA+ICAgCXN0cnVjdCBjb21icGh5X3JlZyB1M290ZzBfcG9ydF9lbjsKPiA+ ICAgCXN0cnVjdCBjb21icGh5X3JlZyB1M290ZzFfcG9ydF9lbjsKPiA+ICAgfTsKPiA+IEBAIC0y MTIsNiArMjEzLDcgQEAgc3RydWN0IHJvY2tjaGlwX2NvbWJwaHlfcHJpdiB7Cj4gPiAgIAlib29s IGVuYWJsZV9zc2M7Cj4gPiAgIAlib29sIGV4dF9yZWZjbGs7Cj4gPiAgIAlzdHJ1Y3QgY2xrICpy ZWZjbGs7Cj4gPiArCXUzMiBzZ21paV9tYWNfc2VsOwo+ID4gICB9Owo+ID4gICBzdGF0aWMgdm9p ZCByb2NrY2hpcF9jb21icGh5X3VwZGF0ZWwoc3RydWN0IHJvY2tjaGlwX2NvbWJwaHlfcHJpdiAq cHJpdiwKPiA+IEBAIC0zNzUsNiArMzc3LDkgQEAgc3RhdGljIGludCByb2NrY2hpcF9jb21icGh5 X3BhcnNlX2R0KHN0cnVjdCBkZXZpY2UgKmRldiwgc3RydWN0IHJvY2tjaGlwX2NvbWJwaHkKPiA+ ICAgCXByaXYtPmV4dF9yZWZjbGsgPSBkZXZpY2VfcHJvcGVydHlfcHJlc2VudChkZXYsICJyb2Nr Y2hpcCxleHQtcmVmY2xrIik7Cj4gPiArCXByaXYtPnNnbWlpX21hY19zZWwgPSAxOwo+IAo+IEJl Zm9yZSB0aGlzIHBhdGNoLCBTR01NSSB3YXMgcm91dGVkIHRvIEdNQUNYIGRlcGVuZHMgb24gYm9v dGxvYWRlcihpZgo+IHNldHRpbmcpLCBvciByb3V0ZWQgdG8gR01BQzEgYnkgY2hpcCBkZWZhdWx0 LiBXaXRob3V0IGtub3dpbmcgYW55Cj4gcG90ZW50aWFsIGNhbmRpZGF0ZXMgc2V0dGluZyBpdCB2 aWEgYm9vdGxvYWRlciwgeW91IG1pZ2h0IG5vdCBmb3JjZSBhbnkKPiBEVFMgZmlsZXMgd2l0aG91 dCAicm9ja2NoaXAsc2dtaWktbWFjLXNlbCIgdG8gcGljayBHTUFDMSBhcyBmYWxsYmFjay4KPiAK PiBNYXliZSB5b3UgY291bGQganVzdCBrZWVwIHRoZSBTR01JSSByb3V0ZSBhcy1pdCBpZiB0aGUg cHJvcGVydHkgaXMgbm90Cj4gcHJlc2VudC4gTWVhbndoaWxlLCBzaG91bGQgdmFsaWRhdGUgcm9j a2NoaXAsc2dtaWktbWFjLXNlbCBpcyBhIHZhbGlkCj4gdmFsdWUgcGFzc2luZyBmcm9tIERULgoK VGhhdCBpcyB0aGUgdHlwaWNhbCB3YXkgdHJpLXN0YXRlIHByb3BlcnRpZXMgd29yayAoYXMgb3Bw b3NlZCB0byB1c2luZyBhIApib29sZWFuKS4gSW4gdGhhdCBjYXNlLCBubyAnZGVmYXVsdCcgaW4g dGhlIHNjaGVtYSB3b3VsZCB0aGVuIGJlIAphcHByb3ByaWF0ZS4KClJvYgoKX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTGludXgtcm9ja2NoaXAgbWFpbGlu ZyBsaXN0CkxpbnV4LXJvY2tjaGlwQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmlu ZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1yb2NrY2hpcAo=