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 68E5BC001B0 for ; Thu, 10 Aug 2023 16:11:17 +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=T8yV4kkyxhJjsf4j3zqCTCZD6kUMzSMiQCyRxr/xoKA=; b=gNe607LkIguirN JTPCwJ0M23gy+6JRtfJE9Zktn95yt/f8y6v0MIgPnHBgZm/4Jy2KbxQYP6fCVcpfxJnvm+3frzHLo THDuDRWCAlapmpJ4chskzvh//l5dTdS0I+wMFTL7VKS6EMoOjYEJwbbECQudrrqT5Ep1wmrnccHD7 FvQid9AppeBPqY/Y/wf2ELo+I7Dg8luU5P0/RmMcTQ5XcVKTyf+ov8a7hNftI8ocVSESBkQe0LMwV JDG5iBjA7UN74V4ydE7xOuuSin6SSKUiUSgnTEA1yDaohrny0h33xKCdNVFLCOcFE0WeBE33Zs+X7 zl7BGsbSPgRfUkYkR7Ew==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qU8FQ-0089Mm-2k; Thu, 10 Aug 2023 16:10:52 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qU8FO-0089ML-00 for linux-arm-kernel@lists.infradead.org; Thu, 10 Aug 2023 16:10:51 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 8003466197; Thu, 10 Aug 2023 16:10:49 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B37CFC433C7; Thu, 10 Aug 2023 16:10:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1691683848; bh=hU3zEJqh3v/qDDuwBKbxUUN5LenjFbRiKXk6hZBPbkY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ov/ISR/VUh/E/BVRWOz0QgqVAECA8Syehms+PpjS07xKA5ZI4OqwY/vYO29RAgZ7h dMAU7YmZWhRsa/3euTGYup3zU1fvMTUcrjz77+Ur1N8y6GtGlntBPqLM5TChgU7ThY OxDPsQYgxk/tvNAaCHRqt4tczeafnCEjkZXnDC/PNP7NBEiBKf1FiN+Tsq96u/2FR6 K1wTqHVUhhngmhhalIBcvQF2+Sd1ALTB11SbYC44LuKc2ZVtZGQh0X1RBSPsOw5Ego ZfzW4Kt95N/6WpNykRZNlwS/G3BUBmnlJBM5bpSa9nrSIKhPWqmhf7RT3vwva3nWfZ Oshf8AHsJxJ9A== Date: Thu, 10 Aug 2023 23:59:06 +0800 From: Jisheng Zhang To: Conor Dooley Cc: "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Giuseppe Cavallaro , Alexandre Torgue , Jose Abreu , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH net-next v2 09/10] dt-bindings: net: snps,dwmac: add per channel irq support Message-ID: References: <20230807164151.1130-1-jszhang@kernel.org> <20230807164151.1130-10-jszhang@kernel.org> <20230808-clapper-corncob-0af7afa65752@spud> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230808-clapper-corncob-0af7afa65752@spud> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230810_091050_125292_BB9D170C X-CRM114-Status: GOOD ( 19.75 ) 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 On Tue, Aug 08, 2023 at 08:39:58AM +0100, Conor Dooley wrote: > On Tue, Aug 08, 2023 at 12:41:50AM +0800, Jisheng Zhang wrote: > > The IP supports per channel interrupt, add support for this usage case. > > > > Signed-off-by: Jisheng Zhang > > --- > > .../devicetree/bindings/net/snps,dwmac.yaml | 33 +++++++++++++++++++ > > 1 file changed, 33 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml > > index 5d81042f5634..5a63302ad200 100644 > > --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml > > +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml > > @@ -109,6 +109,7 @@ properties: > > - description: The interrupt that occurs when Rx exits the LPI state > > - description: The interrupt that occurs when Safety Feature Correctible Errors happen > > - description: The interrupt that occurs when Safety Feature Uncorrectible Errors happen > > + - description: All of the rx/tx per-channel interrupts > > > > interrupt-names: > > minItems: 1 > > @@ -118,6 +119,38 @@ properties: > > - const: eth_lpi > > - const: sfty_ce > > - const: sfty_ue > > + - const: rx0 > > + - const: rx1 > > + - const: rx2 > > + - const: rx3 > > + - const: rx4 > > + - const: rx5 > > + - const: rx6 > > + - const: rx7 > > + - const: rx8 > > + - const: rx9 > > + - const: rx10 > > + - const: rx11 > > + - const: rx12 > > + - const: rx13 > > + - const: rx14 > > + - const: rx15 > > + - const: tx0 > > + - const: tx1 > > + - const: tx2 > > + - const: tx3 > > + - const: tx4 > > + - const: tx5 > > + - const: tx6 > > + - const: tx7 > > + - const: tx8 > > + - const: tx9 > > + - const: tx10 > > + - const: tx11 > > + - const: tx12 > > + - const: tx13 > > + - const: tx14 > > + - const: tx15 > > I don't think Rob's comment about having added 2 interrupts but 32 > interrupt names has been resolved. I misunderstood Rob's comment. Now I'm not sure whether dt-binding can support regex or something or not, or let ask for advice in the following way: how could I write the dt-binding in this case? I didn't find similar examples so far. I'm not sure listing possible description and const properties for all channel interrupts is suitable. > Did you actually test putting this many interrupts into a node? > AFAICT, any more than 6 will cause complaints. I tried 12rx and 12tx interrupts in a node, didn't see dtc warning. so I guess the complaints are from dtb check? I will try to reproduce them. > > Thanks, > Conor. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel