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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 20DA5C001DB for ; Tue, 8 Aug 2023 17:01:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233852AbjHHRBO (ORCPT ); Tue, 8 Aug 2023 13:01:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36004 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233421AbjHHRAY (ORCPT ); Tue, 8 Aug 2023 13:00:24 -0400 Received: from mail-ed1-x52d.google.com (mail-ed1-x52d.google.com [IPv6:2a00:1450:4864:20::52d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AEF396E90 for ; Tue, 8 Aug 2023 09:00:35 -0700 (PDT) Received: by mail-ed1-x52d.google.com with SMTP id 4fb4d7f45d1cf-5234f2c6c1dso25375a12.1 for ; Tue, 08 Aug 2023 09:00:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691510404; x=1692115204; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=q6MY1hiBmbwXLoucjEj2Xi2SFMUgMZzIdibN/5/2L+I=; b=XE3FbJgkdOF/iq2UzkGREAHWzytiQow/a+OkiLcUWzD3afPgXpn4Bkn0q23aepYcPm PlDJUO4h6xktKbUpriRdA9mBfB91M3xhtYS6PKmWp1zCNjrzSKb4JY40Kp23wJR29P6+ 862U0t0fdoLrXXzKbnSNDzNezGWrLwgSxq+dDqkVyK7MQKzl4yfIFA4hQL4PdOT/gpHu AG6glwy2/ncLWP+i/cffmsO2L4FiA13ohkomBtNmbGVbuQA+iTYuZIiRDUFX3KfGKfF7 26JhF0GEUCXbMYdcHbIHTWD/B2I87yHduL0HA1KJ3d1MzlLNkcyG1YoDHWWfBmI9EdSU yU8g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691510404; x=1692115204; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=q6MY1hiBmbwXLoucjEj2Xi2SFMUgMZzIdibN/5/2L+I=; b=bUpcPpM0Cp3bYleUGHPNiEj1OXotzIduoaSpgME3kby65GBEUZRv35Jc57+DlrATdl cbSIwufMw9JvyN1LsDdHcMpQR56+THMxuDsp/tz24m5BBKG6gZGC6LTB7NT8zLXzhUxA /j6sB5nZbBoBJWjMzBg81M/ZGrVJhoWc6QAng2VF2Cmpa9HMh3M5GsoO/rjoAFFklKA2 oQG4MW+Ij1XPRr2gOz41uhztNmt3cXyidioWXQIRE9UtOQvhdN+mqQU6WNGIebZjUchK JQEG6A8IWcNqnkdFBrjEGyb+yt5omKB7Yq8XDwC20Nw7FF30hudidhyCW+ZbVEuOa2XM XmgA== X-Gm-Message-State: AOJu0YzeZx8neaxHLuk5gk1oWmV/hToH1SiOKaHiNSxugcvqW0AMQY07 NxPHE8RHTtHlOf9t7WMO7o3Nev76AFPOwHj0e04= X-Google-Smtp-Source: AGHT+IEFM3DwvG4gSvRW8WGApgoleljA2c2Cr/zQIjyrhjF0UV2hAUM2hKy+et7OFD1S77hBApF2qg== X-Received: by 2002:a17:906:54:b0:999:26d3:b815 with SMTP id 20-20020a170906005400b0099926d3b815mr13144104ejg.64.1691506434441; Tue, 08 Aug 2023 07:53:54 -0700 (PDT) Received: from [192.168.1.20] ([178.197.222.113]) by smtp.gmail.com with ESMTPSA id rk22-20020a170907215600b00982a92a849asm6796311ejb.91.2023.08.08.07.53.52 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 08 Aug 2023 07:53:53 -0700 (PDT) Message-ID: <8e7cf088-e206-746d-0360-16479b1a33c3@linaro.org> Date: Tue, 8 Aug 2023 16:53:51 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH] dt-bindings: remoteproc: pru: Add Interrupt property Content-Language: en-US To: Md Danish Anwar , Conor Dooley , MD Danish Anwar Cc: Suman Anna , Conor Dooley , Krzysztof Kozlowski , Rob Herring , Mathieu Poirier , Bjorn Andersson , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-remoteproc@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, vigneshr@ti.com, srk@ti.com, nm@ti.com References: <20230807110836.2612730-1-danishanwar@ti.com> <20230807-euphemism-trailing-ef4130dc7437@spud> <910a4a98-712a-5517-5a5b-ffb962f83463@ti.com> <1ade44f5-b3d1-dcde-6819-9e944f3b115d@linaro.org> <5ca41ad1-4f42-f71a-4b5f-f2b6def42cb1@ti.com> From: Krzysztof Kozlowski In-Reply-To: <5ca41ad1-4f42-f71a-4b5f-f2b6def42cb1@ti.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 08/08/2023 12:57, Md Danish Anwar wrote: > On 08/08/23 3:48 pm, Krzysztof Kozlowski wrote: >> On 08/08/2023 11:44, Md Danish Anwar wrote: >>>>> properties: >>>>> compatible: >>>>> @@ -171,6 +181,9 @@ examples: >>>>> <0x22400 0x100>; >>>>> reg-names = "iram", "control", "debug"; >>>>> firmware-name = "am65x-pru0_0-fw"; >>>>> + interrupt-parent = <&icssg0_intc>; >>>>> + interrupts = <16 2 2>; >>>>> + interrupt-names = "vring"; >>>>> }; >>>> >>>> These examples would probably be more helpful if they used the >>>> appropriate defines, no? >>>> >>> >>> PRUSS Interrupt controller doesn't have any appropriate defines. This doesn't >>> use GIC so defines from arm-gic.h can not be used here. These are specific to >>> PRUSS INTC. >>> >>> I think these example are OK. Please let me know if this is OK to you. >> >> But isn't "2" type of the interrupt? >> >> Best regards, >> Krzysztof >> > > As per the description of interrupts property in ti,pruss-intc.yaml [1] > > Cell 1 is PRU System event number, cell 2 is PRU channel and cell 3 is PRU > host_event (target). None of them is type of interrupt. So that's why they all > are hardcoded. I don't think we can use IRQ_TYPE macros here. OK, thanks for clarifying this. Best regards, Krzysztof