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 8005FC77B75 for ; Mon, 8 May 2023 14:17:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234451AbjEHORw (ORCPT ); Mon, 8 May 2023 10:17:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55434 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233722AbjEHORv (ORCPT ); Mon, 8 May 2023 10:17:51 -0400 Received: from mail-ej1-x633.google.com (mail-ej1-x633.google.com [IPv6:2a00:1450:4864:20::633]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3E062A0 for ; Mon, 8 May 2023 07:17:50 -0700 (PDT) Received: by mail-ej1-x633.google.com with SMTP id a640c23a62f3a-96649b412easo229559966b.0 for ; Mon, 08 May 2023 07:17:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1683555469; x=1686147469; 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=ownDs96VmZuf2SD3gS4UazEGTd4JsZ/cGQWhog0hNx8=; b=iu+iZGfPr4HbdXLGQeG5JZH80mbVi4fy7axe2WAwYlOvsmJ3zh2Z5gbnd+Jq//v/RJ hgkUFpB/STejIWxcxxYvXlsQzOd7/aN1Rc7JBXrfLo2FesYT8SOGtYs3VgzN7OXS8amJ ab4KD//BjGTO+eoplp2ZCIMc0gtGbg5EaJqt6yUi39vpfNAue9xvU/THJPbTmsuq9gD2 Kb6XtVEKaqleJROnJRNQf4UNxNAwrwrrT+T9w0fWotk8JA5XZdpIqrfK2mUhOAr50Ofj onwKO8p7VFAOku/1HDv8vlyqmSORhuMlo/BK1wfQghwAMW0VlBq12IaQ/ubTIIJUqWHp VuVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683555469; x=1686147469; 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=ownDs96VmZuf2SD3gS4UazEGTd4JsZ/cGQWhog0hNx8=; b=Tqz8IhiSnxRjbr5IgLI5tzYqL7EIbK2NYdhgU78UEBsfyeiJ/1Wo+rJ5wxU+X2EbVU ouH6+YVqCDczwAYY9Xf/GkKc1/BWTRy7kZlesFJy/aF6BiDq0X5UPzWlxPWN2847wUD2 FnZWTvnOvFHiN2LCmXaPH3aS7MgT/mAW8t6jpRwMEFAURX96X76VBc1QLFH5Ct0LpiOe ezOJWxSVMLr7/o5Bfrg42yykXVuvfdjut2LezmlubGVSQ+5dmmKNRjCTaqlPQyroaY7g B7Gamiomd1Q/8hz40ATreSn+x3rN7u+TMXcZD6vXr4DUSTy1MAYdwzkfbeyb3kDkIEBt 9QKg== X-Gm-Message-State: AC+VfDzGfZs29i6KcMM+JVvrdm6/tE2x4jpGM4+6CD5OfbvfgTaW3/br ADLtdkjvcfZTNtgISUwAckl7ow== X-Google-Smtp-Source: ACHHUZ560sKldiqrIMjg1ekA7xnXwChXcannOgDV/O9OwzIlwtcHkzpXwhvGDxk9FWQR5cgmnyBKOw== X-Received: by 2002:a17:907:2d26:b0:94f:3804:5cd6 with SMTP id gs38-20020a1709072d2600b0094f38045cd6mr9550720ejc.66.1683555468736; Mon, 08 May 2023 07:17:48 -0700 (PDT) Received: from ?IPV6:2a02:810d:15c0:828:50e0:ebdf:b755:b300? ([2a02:810d:15c0:828:50e0:ebdf:b755:b300]) by smtp.gmail.com with ESMTPSA id v9-20020a170906338900b0094ee99eeb01sm21627eja.150.2023.05.08.07.17.47 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 08 May 2023 07:17:48 -0700 (PDT) Message-ID: <7e16c25d-ab17-741a-1c7c-6cf1462eef5b@linaro.org> Date: Mon, 8 May 2023 16:17:46 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH 4/5] dt-bindings: Add bindings to support DRAM MRQ GSCs Content-Language: en-US To: Peter De Schrijver , "thierry.reding@gmail.com" , Jonathan Hunter Cc: "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "conor+dt@kernel.org" , "devicetree@vger.kernel.org" , "linux-tegra@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Stefan Kristiansson References: <20230508122048.99953-1-pdeschrijver@nvidia.com> <20230508122048.99953-5-pdeschrijver@nvidia.com> <43945fad-f57b-cfd9-78bd-9ec9e6628382@linaro.org> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 08/05/2023 16:12, Peter De Schrijver wrote: > > > ________________________________________ > From: Krzysztof Kozlowski > Sent: 08 May 2023 17:04 > To: Peter De Schrijver; thierry.reding@gmail.com; Jonathan Hunter > Cc: robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; devicetree@vger.kernel.org; linux-tegra@vger.kernel.org; linux-kernel@vger.kernel.org; Stefan Kristiansson > Subject: Re: [PATCH 4/5] dt-bindings: Add bindings to support DRAM MRQ GSCs > > On 08/05/2023 14:20, Peter De Schrijver wrote: >> Add bindings for DRAM MRQ GSC support. >> >> Co-developed-by: Stefan Kristiansson >> Signed-off-by: Stefan Kristiansson >> Signed-off-by: Peter De Schrijver >> --- >> .../firmware/nvidia,tegra186-bpmp.yaml | 69 ++++++++++++++++++- >> .../nvidia,tegra264-bpmp-shmem.yaml | 40 +++++++++++ > >> Why touching two files? > > Because both are needed to support having MRQ GSCs in DRAM. Yeah, but why two in one commit? All patches are needed for full support and they are not squashed into one. > >> 2 files changed, 106 insertions(+), 3 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/reserved-memory/nvidia,tegra264-bpmp-shmem.yaml >> >> diff --git a/Documentation/devicetree/bindings/firmware/nvidia,tegra186-bpmp.yaml b/Documentation/devicetree/bindings/firmware/nvidia,tegra186-bpmp.yaml >> index 833c07f1685c..d818cfe1d783 100644 >> --- a/Documentation/devicetree/bindings/firmware/nvidia,tegra186-bpmp.yaml >> +++ b/Documentation/devicetree/bindings/firmware/nvidia,tegra186-bpmp.yaml >> @@ -57,8 +57,11 @@ description: | >> "#address-cells" or "#size-cells" property. >> >> The shared memory area for the IPC TX and RX between CPU and BPMP are >> - predefined and work on top of sysram, which is an SRAM inside the >> - chip. See ".../sram/sram.yaml" for the bindings. >> + predefined and work on top of either sysram, which is an SRAM inside the >> + chip, or in normal SDRAM. >> + See ".../sram/sram.yaml" for the bindings for the SRAM case. >> + See "../reserved-memory/nvidia,tegra264-bpmp-shmem.yaml" for bindings for >> + the SDRAM case. >> >> properties: >> compatible: >> @@ -81,6 +84,11 @@ properties: >> minItems: 2 >> maxItems: 2 >> >> + memory-region: >> + description: phandle to reserved memory region used for IPC between >> + CPU-NS and BPMP. >> + maxItems: 1 >> + >> "#clock-cells": >> const: 1 >> >> @@ -115,10 +123,16 @@ properties: >> >> additionalProperties: false >> >> +allOf: >> + - oneOf: > > Keep just oneOf and drop allOf. ??? I wrote this. Fix your mailer. > >> + - required: >> + - memory-region >> + - required: >> + - shmem >> + >> required: >> - compatible >> - mboxes >> - - shmem >> - "#clock-cells" >> - "#power-domain-cells" >> - "#reset-cells" >> @@ -184,3 +198,52 @@ examples: >> #thermal-sensor-cells = <1>; >> }; >> }; >> + >> + - | >> + #include >> + #include >> + #include >> + >> + hsp_top0: hsp@3c00000 { >> + compatible = "nvidia,tegra186-hsp"; >> + reg = <0x03c00000 0xa0000>; >> + interrupts = ; >> + interrupt-names = "doorbell"; >> + #mbox-cells = <2>; > >> Why HSP example is here? > > Because it's referred to further down the example. Not needed. > >> + }; >> + >> + reserved-memory { >> + dram_cpu_bpmp_mail: shmem@f1be0000 { >> + compatible = "nvidia,tegra264-bpmp-shmem"; >> + reg = <0x0 0xf1be0000 0x0 0x2000>; >> + no-map; >> + }; >> + }; > > Drop, fairly obvious and should be in that binding, not here. ??? Please use mailing list style of replies. Best regards, Krzysztof