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 C3851C4332F for ; Wed, 13 Dec 2023 08:17:12 +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:From:References:To:Subject: CC: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=+xaLxKlgDV9X4wwiNasCdZIel+WMBBofilSNFv7DRvQ=; b=W5Papy5p1B8woD DC9DsQESHaLnCxuCvFGkyK0U/GZZLACv6xThtFMIX0YugSxGT8UIx+XN/P41LgeBYmJ7EVkR3JFRZ nzvmpSUBL0ITPurLDK+lZkSr+rB8cU/BU4RlXIebTtzFoD1l/FMpq7GzQAFFieN4xuyU4kGQWwjdW iEsTbVDvHiDdaqj0WJI1C2D//Zt1eQx0MSTCY4GUBJImbCAGRIZ3OPGQpxOQMCy3LXics7o7uZ97B pmsHLpk3xaDzRvBotfCtlK1PQruXhvndQLyXU2U6c8cTbv0JVog5ykCAF2/xQj/JluiUNV203Gqcu NpYtc705n3IrwtN9bhkA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rDKQ8-00E0Aq-1W; Wed, 13 Dec 2023 08:16:44 +0000 Received: from lelv0143.ext.ti.com ([198.47.23.248]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rDKQ3-00E09u-2v for linux-arm-kernel@lists.infradead.org; Wed, 13 Dec 2023 08:16:42 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by lelv0143.ext.ti.com (8.15.2/8.15.2) with ESMTP id 3BD8GXxT059954; Wed, 13 Dec 2023 02:16:33 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1702455393; bh=FStV+DQOwLOG+FKKRELWFahKxJNKTxkxhmIijEdUO3M=; h=Date:CC:Subject:To:References:From:In-Reply-To; b=v+kMrcXy8lzWgCOI60Q8ItTyzhNVJoJZ3z+KuZFNjI9Bvhj/XMo6S1yXenbJT14TF CuZor7IQ8Zh1RZU2H+SnZS7hjMIuW1EWCFPnRC+2Xi2asPD5ux9an9I3SV4uxcOv3R rQ9KnS/2G2KaPLtVzW6XXr0G+3FeHSao5uztB7V4= Received: from DFLE101.ent.ti.com (dfle101.ent.ti.com [10.64.6.22]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 3BD8GXej119101 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 13 Dec 2023 02:16:33 -0600 Received: from DFLE101.ent.ti.com (10.64.6.22) by DFLE101.ent.ti.com (10.64.6.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Wed, 13 Dec 2023 02:16:33 -0600 Received: from lelvsmtp5.itg.ti.com (10.180.75.250) by DFLE101.ent.ti.com (10.64.6.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Wed, 13 Dec 2023 02:16:33 -0600 Received: from [172.24.227.9] (uda0492258.dhcp.ti.com [172.24.227.9]) by lelvsmtp5.itg.ti.com (8.15.2/8.15.2) with ESMTP id 3BD8GTW1040313; Wed, 13 Dec 2023 02:16:30 -0600 Message-ID: <89ed9033-7fbd-4a48-9a83-0a0b3b208e3d@ti.com> Date: Wed, 13 Dec 2023 13:46:28 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird CC: , , , , , , , , , , , Subject: Re: [PATCH] arm64: dts: ti: k3-am654-icssg2: Enable PHY interrupts for ICSSG2 Content-Language: en-US To: Nishanth Menon References: <20231120063159.539306-1-s-vadapalli@ti.com> <20231204132103.ikkxjz3yxz3ynq6s@demystify> From: Siddharth Vadapalli In-Reply-To: <20231204132103.ikkxjz3yxz3ynq6s@demystify> 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-20231213_001640_029869_D528E09D X-CRM114-Status: GOOD ( 17.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-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Nishanth, Thank you for reviewing the patch. I have addressed your feedback in the v2 patch at: https://lore.kernel.org/r/20231213080216.1710730-1-s-vadapalli@ti.com/ On 04/12/23 18:51, Nishanth Menon wrote: > On 12:01-20231120, Siddharth Vadapalli wrote: >> Enable interrupt mode of operation of the DP83867 Ethernet PHY which is >> used by ICSSG2. The DP83867 PHY driver already supports interrupt handling >> for interrupts generated by the PHY. Thus, add the necessary device-tree >> support to enable it. >> >> Since the GPIO1_87 line is muxed with EXT_REFCLK1 and SYNC1_OUT, update >> the pinmux to select GPIO1_87 for routing the interrupt. >> >> Signed-off-by: Siddharth Vadapalli >> --- >> >> This patch is based on linux-next tagged next-20231120. >> >> Regards, >> Siddharth. >> >> arch/arm64/boot/dts/ti/k3-am654-icssg2.dtso | 17 +++++++++++++++-- >> 1 file changed, 15 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-am654-icssg2.dtso b/arch/arm64/boot/dts/ti/k3-am654-icssg2.dtso >> index ec8cf20ca3ac..9f723592d0f4 100644 >> --- a/arch/arm64/boot/dts/ti/k3-am654-icssg2.dtso >> +++ b/arch/arm64/boot/dts/ti/k3-am654-icssg2.dtso >> @@ -124,21 +124,34 @@ AM65X_IOPAD(0x0088, PIN_INPUT, 2) /* (AG17) PRG2_PRU0_GPO4.PRG2_RGMII1_RX_CTL */ >> }; >> }; >> >> +&main_pmx1 { >> + /* Select GPIO1_87 for ICSSG2 PHY interrupt */ >> + icssg2_phy_irq_pins_default: icssg2-phy-irq-default-pins { >> + pinctrl-single,pins = < >> + AM65X_IOPAD(0x0014, PIN_INPUT, 7) /* (A22) EXT_REFCLK1.GPIO1_87 */ >> + >; >> + }; >> +}; >> + >> &icssg2_mdio { >> status = "okay"; >> - pinctrl-names = "default"; >> - pinctrl-0 = <&icssg2_mdio_pins_default>; >> + pinctrl-names = "default", "icssg2-phy-irq"; >> + pinctrl-0 = <&icssg2_mdio_pins_default>, <&icssg2_phy_irq_pins_default>; > > why should the pins be part of mdio pinctrl instead of phy? > >> #address-cells = <1>; >> #size-cells = <0>; >> >> icssg2_phy0: ethernet-phy@0 { >> reg = <0>; >> + interrupt-parent = <&main_gpio1>; >> + interrupts = <87 0x2>; >> ti,rx-internal-delay = ; >> ti,fifo-depth = ; >> }; >> >> icssg2_phy1: ethernet-phy@3 { >> reg = <3>; >> + interrupt-parent = <&main_gpio1>; >> + interrupts = <87 0x2>; > > Shouldn't you be using macros for interrupt level like IRQ_TYPE_EDGE_FALLING? > >> ti,rx-internal-delay = ; >> ti,fifo-depth = ; >> }; >> -- >> 2.34.1 >> > -- Regards, Siddharth. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel