Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp5025790pxj; Wed, 12 May 2021 19:53:18 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyU5h04Kve1/OWbTxIODEOB2sQBSjud1hTt0xCX4VqLHC8A1eIipvhDXrHiox9b6VTa2S2C X-Received: by 2002:a17:906:cb1:: with SMTP id k17mr40684816ejh.307.1620874398028; Wed, 12 May 2021 19:53:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620874398; cv=none; d=google.com; s=arc-20160816; b=pE7EfJmUX4vGdl5kWr/vVwDGJShCna+rAUWeYHk8tqZV5nngnoIDuKbUS4Bh4XIyRB 84ModKBVRhwf9aAAmqGJt8ZcaqnHKDJnjxCBHe/+GwJdCHeH1DJ/25y57Dfib2YpNyz3 Umti941tq4tKxzZrUgx6lh5HeNCFiWsx1ZDomRtj+15B5MGYByl0DWlTDwxBy9MVtsx9 pvjPO2JUl45aqqJxYlXDktBQTutdRozXl3Zsu26+m+P/uVVR/05n8ZAN4UZE4ZjMz1l0 ykcFMwQI5pbevp1cvg9iZMQZUH4LT0J8IXZCSlerpQnl0yv8XrqiVMdRWUDn7yca/eMA OUQw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:references:cc :to:from:subject:dkim-signature; bh=CVL24IXjC7hS/rORC2nMLVaPFRAth7V3AD9AbT6OxgY=; b=oM9IjCaFarWB1opoerx6VYAtQQsOMYnj3hXWDKLNq4AsDiUmzi8yxPa8Fu1+3NQOxF gAFXYV+HLYRcJydAY/RYRZTvx39eqZAK7w1EksoePR4k1cDoFlT7MtwqJ6vOQyZQYIZs A3ldlTOtyfPqCar1opzAQ00fc7Dnkv4RFDKgElQl0bOoFYCJ11htJj+9MsFGO/y2xdmF XluHxsAY4iGzpFtzDn/4k41f4FFrA0Gq6A1rHCGD5zRWxcSdUBALmhswL1hBznDWzh2y Zz+VnMb8cATtOHm2m1SFMIlNH9ie2iDEVnA2BLlsoAhDnoytAvGn2cYK7R1wcNWN7DSW SBrQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=ohtqtg3h; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ne33si1523266ejc.24.2021.05.12.19.52.33; Wed, 12 May 2021 19:53:18 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=ohtqtg3h; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230247AbhEMCtD (ORCPT + 99 others); Wed, 12 May 2021 22:49:03 -0400 Received: from fllv0016.ext.ti.com ([198.47.19.142]:50050 "EHLO fllv0016.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229813AbhEMCtC (ORCPT ); Wed, 12 May 2021 22:49:02 -0400 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 14D2ljqr012163; Wed, 12 May 2021 21:47:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1620874065; bh=CVL24IXjC7hS/rORC2nMLVaPFRAth7V3AD9AbT6OxgY=; h=Subject:From:To:CC:References:Date:In-Reply-To; b=ohtqtg3h70b1QcdUPjyCMG0N9JyIV82M+uk+TEsSdLZDUq8Uhq8G61Gj9M7mD2aWO h7EnD5cTwYy8nLP1fSpTWp4+Z+QG60mmeiMwOH5BKYJzYcIFtNP2jEYEUBvWuLaBW5 0jJZGF7JYZrHPLS3r2U3D37oH/50BjDGilvaKYU8= Received: from DFLE106.ent.ti.com (dfle106.ent.ti.com [10.64.6.27]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 14D2lid3070216 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 12 May 2021 21:47:44 -0500 Received: from DFLE105.ent.ti.com (10.64.6.26) by DFLE106.ent.ti.com (10.64.6.27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Wed, 12 May 2021 21:47:44 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DFLE105.ent.ti.com (10.64.6.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2 via Frontend Transport; Wed, 12 May 2021 21:47:44 -0500 Received: from [10.250.33.185] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 14D2li1V024269; Wed, 12 May 2021 21:47:44 -0500 Subject: Re: [PATCH v2 1/2] dt-bindings: remoteproc: k3-r5f: Update bindings for AM64x SoCs From: Suman Anna To: Bjorn Andersson , Mathieu Poirier , Rob Herring CC: Lokesh Vutla , , , , References: <20210327143117.1840-1-s-anna@ti.com> <20210327143117.1840-2-s-anna@ti.com> <8948a30c-1a2f-1fb0-05bb-37be9c02c5d5@ti.com> Message-ID: Date: Wed, 12 May 2021 21:47:44 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <8948a30c-1a2f-1fb0-05bb-37be9c02c5d5@ti.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On 4/19/21 8:55 AM, Suman Anna wrote: > Hi Rob, > > On 3/27/21 9:31 AM, Suman Anna wrote: >> The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with >> 2 R5F cores each, both in the MAIN voltage domain. >> >> These clusters are a revised IP version compared to those present on >> J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of >> LockStep mode. Update the K3 R5F remoteproc bindings with the compatible >> info relevant to these R5F clusters/subsystems on K3 AM64x SoCs. >> >> Signed-off-by: Suman Anna >> --- >> v2: No changes >> >> .../bindings/remoteproc/ti,k3-r5f-rproc.yaml | 31 ++++++++++++++++--- > > Looks like this patch has fallen through the cracks, can you please review and > give your ack for this patch so that Bjorn can pick up the series for 5.13? Gentle reminder, do you have any comments on this patch. Appreciate your ack so that we can get this in for 5.14? regards Suman > > regards > Suman > >> 1 file changed, 26 insertions(+), 5 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml >> index d905d614502b..130fbaacc4b1 100644 >> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml >> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml >> @@ -14,8 +14,12 @@ description: | >> processor subsystems/clusters (R5FSS). The dual core cluster can be used >> either in a LockStep mode providing safety/fault tolerance features or in a >> Split mode providing two individual compute cores for doubling the compute >> - capacity. These are used together with other processors present on the SoC >> - to achieve various system level goals. >> + capacity on most SoCs. These are used together with other processors present >> + on the SoC to achieve various system level goals. >> + >> + AM64x SoCs do not support LockStep mode, but rather a new non-safety mode >> + called "Single-CPU" mode, where only Core0 is used, but with ability to use >> + Core1's TCMs as well. >> >> Each Dual-Core R5F sub-system is represented as a single DTS node >> representing the cluster, with a pair of child DT nodes representing >> @@ -33,6 +37,7 @@ properties: >> - ti,am654-r5fss >> - ti,j721e-r5fss >> - ti,j7200-r5fss >> + - ti,am64-r5fss >> >> power-domains: >> description: | >> @@ -56,11 +61,12 @@ properties: >> >> ti,cluster-mode: >> $ref: /schemas/types.yaml#/definitions/uint32 >> - enum: [0, 1] >> description: | >> Configuration Mode for the Dual R5F cores within the R5F cluster. >> - Should be either a value of 1 (LockStep mode) or 0 (Split mode), >> - default is LockStep mode if omitted. >> + Should be either a value of 1 (LockStep mode) or 0 (Split mode) on >> + most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted; >> + and should be either a value of 0 (Split mode) or 2 (Single-CPU mode) >> + on AM64x SoCs, default is Split mode if omitted. >> >> # R5F Processor Child Nodes: >> # ========================== >> @@ -97,6 +103,7 @@ patternProperties: >> - ti,am654-r5f >> - ti,j721e-r5f >> - ti,j7200-r5f >> + - ti,am64-r5f >> >> reg: >> items: >> @@ -198,6 +205,20 @@ patternProperties: >> >> unevaluatedProperties: false >> >> +if: >> + properties: >> + compatible: >> + enum: >> + - ti,am64-r5fss >> +then: >> + properties: >> + ti,cluster-mode: >> + enum: [0, 2] >> +else: >> + properties: >> + ti,cluster-mode: >> + enum: [0, 1] >> + >> required: >> - compatible >> - power-domains >> >