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 96405C433F5 for ; Wed, 1 Jun 2022 11:51:42 +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: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=6enTWWl/rMWEs9fpHuCQwqq1C9VsFXeHkwoB0dQ6BGU=; b=1oiWk3dJMMV7ak iSUQHpcRV+Sl8rLIPZFUZho6TtjCIoPhCFeABuGQ8k1Qs7F2PWlQqOg5gE8D1pg8yatLDsQm2lypg YZ3FM6fFQvp5ohOJORxgVaa4fa352YLNqskFIhTyKOueKbPZ+FhBLBAUx3zx/qF2zPQbZScFhYc87 hH7wOrMuXwOZOUeVHgrxYng+D0TPpYdztbNdN8LnLvXd+5cwTXuAbRdcNAX7ttvxbdO3xMXrMry/p e6sZCdB/n5pUlWQxV9tJmhEJllG8wNefApiwlpEYzGNbymu9Wa1R4HA3//hDFuExcP1IaLuaWQkMC no3Vo0rx1xgeANPUth9A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nwMrk-00Fku8-B2; Wed, 01 Jun 2022 11:50:20 +0000 Received: from mail-ed1-x52e.google.com ([2a00:1450:4864:20::52e]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nwMrg-00FktG-OZ for linux-arm-kernel@lists.infradead.org; Wed, 01 Jun 2022 11:50:18 +0000 Received: by mail-ed1-x52e.google.com with SMTP id o10so1822133edi.1 for ; Wed, 01 Jun 2022 04:50:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=NCu09c7It8PCgWDIsBdAVtichE5PsFctC0ahaIHP8Xk=; b=KD/GF46y0L9iV/6oC8B0iJGCrA1YOd1ZDr5ZyyRZKnNDXwqFSyH7pMugTFD19FNvuK DLG3zOI1/3Fg4wUPC6NHPSQsy5OZb4b0Gzq06f1NQQgETWqm89FW05A3FR2gVOXq19JI CVJ6LZEFdk42WkWzwgK+DKTWazScy0oP4rYwVZuAJM9xGuHcCJziiPFOQ+DYiY0ziLC9 879vFCoj64XlDCsamSFbS8NsBkT6Wd/mB/bfROk0lZwGArhZOmwQjbsMRzUVQWJzSFqQ TKzfvDC846UfKtt8N47ycGDKj/fiFXjMu2XbHSJn7sd2vqALeK2+srAsZBIKPsgY1YKN 9Hvw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=NCu09c7It8PCgWDIsBdAVtichE5PsFctC0ahaIHP8Xk=; b=uSrz/rUmsyPQL8EnDXwLTtXuM4qXZy1LOuBfaB1j0FDIJzyjcbo5NeOo1gXwPDOoar VsO59N5JwC469cJGnYCwUPVFHt1SJri/2niJl/nr5Yv3ysLL7Lux9ZB73PKauLA6zDEK m16WS6uIH07xHp4S5+w7rH8xnOqMtmhK5vWOwkuHlA88k2CG0sHnnFc15Uuyraqieo/w SuwwNfdaqIESh3kFR0u6rc09oUDF/AnnyW76RYJ29RlgLI3BDe97XobW80B31hJLG7ij u7uhCHsMh23blYYuJwDFz6sVRC0d4UodhHysmshtTdQIIt4OskiY+LKhpgiLPo9MpXUW 4U5Q== X-Gm-Message-State: AOAM530YOLL6P+6JVZ57M66n6E6vAKJkJZOyJU55gnh518w5Xqj4vvSy KszVP9GKqtZX1p3iOS5Rs6TBAQ== X-Google-Smtp-Source: ABdhPJyELI2heE5E7DXJZNsKABuJGaTaxV4OxNBunIxicg2IcK642ipDPhUCnEa0+rHGU5oepmuQnQ== X-Received: by 2002:a05:6402:1851:b0:42d:c904:d73b with SMTP id v17-20020a056402185100b0042dc904d73bmr20563379edy.417.1654084213214; Wed, 01 Jun 2022 04:50:13 -0700 (PDT) Received: from [192.168.0.179] (xdsl-188-155-176-92.adslplus.ch. [188.155.176.92]) by smtp.gmail.com with ESMTPSA id e26-20020a50ec9a000000b0042ad0358c8bsm855792edr.38.2022.06.01.04.50.11 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 01 Jun 2022 04:50:12 -0700 (PDT) Message-ID: Date: Wed, 1 Jun 2022 13:50:11 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1 Subject: Re: [PATCH v1 01/15] dt-binding: remoteproc: mediatek: Support dual-core SCP Content-Language: en-US To: Tinghan Shen , Bjorn Andersson , Mathieu Poirier , Rob Herring , Krzysztof Kozlowski , Matthias Brugger , Lee Jones , Benson Leung , Guenter Roeck , Daisuke Nojiri , Sebastian Reichel , "Dustin L. Howett" , Tzung-Bi Shih , "Gustavo A. R. Silva" , Prashant Malani , Enric Balletbo i Serra , Brian Norris Cc: linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, chrome-platform@lists.linux.dev, Project_Global_Chrome_Upstream_Group@mediatek.com, weishunc@google.com References: <20220601112201.15510-1-tinghan.shen@mediatek.com> <20220601112201.15510-2-tinghan.shen@mediatek.com> From: Krzysztof Kozlowski In-Reply-To: <20220601112201.15510-2-tinghan.shen@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220601_045016_871367_633C4B1C X-CRM114-Status: GOOD ( 22.15 ) 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 01/06/2022 13:21, Tinghan Shen wrote: > The SCP co-processor is a dual-core RISC-V MCU on MT8195. > > Add a new property to identify each core and helps to find drivers > through device tree API to cooperate with each other, e.g. boot flow and > watchdog timeout flow. > > Add a new compatile for the driver of SCP 2nd core. > > Signed-off-by: Tinghan Shen > --- > .../devicetree/bindings/remoteproc/mtk,scp.yaml | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml > index eec3b9c4c713..b181786d9575 100644 > --- a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml > +++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml > @@ -20,6 +20,7 @@ properties: > - mediatek,mt8186-scp > - mediatek,mt8192-scp > - mediatek,mt8195-scp > + - mediatek,mt8195-scp-dual > > reg: > description: > @@ -57,6 +58,16 @@ properties: > memory-region: > maxItems: 1 > > + mediatek,scp-core: > + $ref: /schemas/types.yaml#/definitions/uint32-array > + description: > + The property value is a list with 2 items, a core id and a phandle uint32, not phandle. > + to the sibling SCP node. Skip this. First part is obvious from the schema, second part should be described via items. The core id represents the id of the dts node contains > + this property. The valid values of core id are 0 and 1 for dual-core SCP. > + The phandle of sibling SCP node is used to find the register settings, > + trigger core dependent callback, and invoke rproc API. Entire description did not help me to understand what's this. So far it looks like it is not a hardware property but some programming help, so it does not look like properly described in bindings. > + maxItems: 1 In description you said - two items. You need allOf:if:then disallowing this property for other variants. > + > required: > - compatible > - reg > @@ -115,6 +126,7 @@ examples: > reg-names = "sram", "cfg", "l1tcm"; > clocks = <&infracfg CLK_INFRA_SCPSYS>; > clock-names = "main"; > + mediatek,scp-core = <0 &scp_dual>; This looks like phandle, so wrong type. > > cros_ec { > mediatek,rpmsg-name = "cros-ec-rpmsg"; Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel