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 1F086C77B60 for ; Wed, 26 Apr 2023 13:22:57 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:CC:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=u83hn0j2NPX2rCcjit0IEwNpkMY63DOh/No3IfiKjr0=; b=Ilki8gv24To4jO N8LGXJVkB0Hd+a36qWbY4BVY0b6loY2cRESoHtvVSSBAN+PN6gxgIZPq+NXqnOJ1bK5a12VyWyyAF 3KvwYNPXK7E4CNZL777dBcOaxAeekYtLL6tLZxLPQcJ5KDDbRjy/4ebp+s1B4YkASasJ2U3APDBom O0Py/Ab+/+uJ6zCmw/vZXCiYWIjvOPXs2ONdH7qxTUNrAvLyp9o0hbSy8iu9Yo3KkWtuZS7msn3mk aH5lkez6gRy78XvoajmHRuttExyHJ+Tw81YE6/heYHCtQPkSbDXkVDuHODLcVFeR0Iqj0N83v5eX4 yXJffCmibDFEa1IE10WQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1prf5m-0043x4-0y; Wed, 26 Apr 2023 13:21:54 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1prf5j-0043wW-1p for linux-arm-kernel@lists.infradead.org; Wed, 26 Apr 2023 13:21:53 +0000 Received: from lelv0265.itg.ti.com ([10.180.67.224]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 33QDLkW4096750; Wed, 26 Apr 2023 08:21:46 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1682515306; bh=QdepxYuSwTdOmEf8k5cCZjOM29OP5/kofyPKcNTFUVo=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=k7thEKT9zPmrKcNVux/rKwIuW3iqOkvhengLrEOwOaV5GtVzjPD7gSEj+rMVe6KRZ QGF4D57JMY5EdrCSumYXs2M2kBqd0+iviDOj+jAtx2fiWvFRE32q6Y4l8dfHUHK2es yMYzjbH8BSG52dU74Z7vtxg9jr89NDVennyHlkPU= Received: from DLEE112.ent.ti.com (dlee112.ent.ti.com [157.170.170.23]) by lelv0265.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 33QDLku7022065 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 26 Apr 2023 08:21:46 -0500 Received: from DLEE112.ent.ti.com (157.170.170.23) by DLEE112.ent.ti.com (157.170.170.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.16; Wed, 26 Apr 2023 08:21:45 -0500 Received: from lelv0327.itg.ti.com (10.180.67.183) by DLEE112.ent.ti.com (157.170.170.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.16 via Frontend Transport; Wed, 26 Apr 2023 08:21:45 -0500 Received: from [10.250.35.77] (ileaxei01-snat.itg.ti.com [10.180.69.5]) by lelv0327.itg.ti.com (8.15.2/8.15.2) with ESMTP id 33QDLjfG001825; Wed, 26 Apr 2023 08:21:45 -0500 Message-ID: <163cfe60-4b14-fe5c-29d9-323e7ea495d5@ti.com> Date: Wed, 26 Apr 2023 08:21:45 -0500 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 Subject: Re: [RFC PATCH 1/2] arm64: dts: ti: k3-j721s2-main: Add main CPSW2G devicetree node Content-Language: en-US To: Nishanth Menon , Siddharth Vadapalli CC: , , , , , , , , References: <20230426105718.118806-1-s-vadapalli@ti.com> <20230426105718.118806-2-s-vadapalli@ti.com> <20230426125927.itfd76cpvy32zur7@scrimmage> From: Andrew Davis In-Reply-To: <20230426125927.itfd76cpvy32zur7@scrimmage> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230426_062151_683242_CB12F3F7 X-CRM114-Status: GOOD ( 18.41 ) 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: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 4/26/23 7:59 AM, Nishanth Menon wrote: > On 16:27-20230426, Siddharth Vadapalli wrote: >> From: Kishon Vijay Abraham I >> >> TI's J721S2 SoC has a MAIN CPSW2G instance of the CPSW Ethernet Switch. >> Add devicetree node for it. >> >> Signed-off-by: Kishon Vijay Abraham I >> Signed-off-by: Siddharth Vadapalli >> --- >> arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi | 69 ++++++++++++++++++++++ >> 1 file changed, 69 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi >> index 6629b2989180..14dfef7b0758 100644 >> --- a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi >> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi >> @@ -51,6 +51,12 @@ usb_serdes_mux: mux-controller@0 { >> mux-reg-masks = <0x0 0x8000000>; /* USB0 to SERDES0 lane 1/3 mux */ >> }; >> >> + phy_gmii_sel_cpsw: phy@34 { >> + compatible = "ti,am654-phy-gmii-sel"; >> + reg = <0x34 0x4>; >> + #phy-cells = <1>; >> + }; > > > See this thread: https://lore.kernel.org/all/76da0b98-3274-b047-db11-ecabc117ae11@ti.com/ > "ti,am654-phy-gmii-sel" already has "reg", so this DT is good as is. Adding the driver fallback to use that when the parent is not a "syscon" node should be as easy as this: --- a/drivers/phy/ti/phy-gmii-sel.c +++ b/drivers/phy/ti/phy-gmii-sel.c @@ -435,9 +435,12 @@ static int phy_gmii_sel_probe(struct platform_device *pdev) priv->regmap = syscon_node_to_regmap(node->parent); if (IS_ERR(priv->regmap)) { - ret = PTR_ERR(priv->regmap); - dev_err(dev, "Failed to get syscon %d\n", ret); - return ret; + priv->regmap = device_node_to_regmap(node); + if (IS_ERR(priv->regmap)) { + ret = PTR_ERR(priv->regmap); + dev_err(dev, "Failed to get syscon %d\n", ret); + return ret; + } } I'll send this patch when the window opens. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel