From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f53.google.com (mail-ej1-f53.google.com [209.85.218.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EF5E24A3D28 for ; Fri, 11 Sep 2026 16:21:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789143663; cv=none; b=j9QHLvjlh/t1BL3zIDjr0OT/la+sxhJF94vugb4bmXfzjN656wewXLVQwDOQ8wnbu8EQNzA1KFVQfyS5LCGfYYK94p22EsffKYmmPRMAGd1HRvjhBqhowocxX89s5XTmA3t2dgYUpEghT4tyzHZw332j2DyulVX7S1biI5cGgP0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789143663; c=relaxed/simple; bh=Fx/YTBtEiBsiwaKpigFxNiDWpt9TDDyotK/D78aJG+g=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=CSFueK1VLtU3G4ee6y80i8/CyIrB0bonzlI82sNMtp9bAitqNW87jkhg7Pld28cO/Jfkgzbg5ZzTrVONA47TaCvbPMUQYudr+AxtT+fJt6V/SiqvhNnHax5DtWQOQxPBAPrTJVuE7Bpi/S2Caa0SUCYhtiS20VPctfjIEoMzYTY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=D1KSxMOi; arc=none smtp.client-ip=209.85.218.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="D1KSxMOi" Received: by mail-ej1-f53.google.com with SMTP id a640c23a62f3a-c259e5c22ffso213597966b.1 for ; Fri, 11 Sep 2026 09:21:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1789143659; x=1789748459; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-type:content-transfer-encoding:mime-version:from:to:cc :subject:date:message-id:reply-to:content-type; bh=t3Ow0xpp1VPcxg35zy2Gpi2FWcle5fD0sYX/EFmyWr4=; b=D1KSxMOijjQCA7SiPHopfdOWWR0ltqCvRVKinHeFUWlZMFVwxAPpMKQTsgNBTwWZeu NJhAKAyZS01th0/8/DRLaVRYOsC43FEh5vTInFlM7Uhm9ftytIzlReehp+xqzMD5G4EW JTlqdrv4bdvkN5yVll/uO0eBuh4DWZ/KdYdOxkkq9mfuNBH3V4TA3oN2ZJ/lP68ypUjS dQKX7kaXs5QCAok6PEUCVk/a82ferJHK51qkLQ3ZI43NVNfrag/LFONvxF0p/rvrvUDX oqsKL3mGjNFH6gyUi7aOfohIXa/l7Ev+Z6080NM/U6B9DblH8kn32SIGMO+N6f3Dr4jJ wLng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789143659; x=1789748459; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-type:content-transfer-encoding:mime-version:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=t3Ow0xpp1VPcxg35zy2Gpi2FWcle5fD0sYX/EFmyWr4=; b=WkI21VM9zR3M8MspcuFzJk3wTAQrY7E7mY01AkvBpvpOXvIwzXnEETfAqY5Q4pGIcp z9PhiYtPiZquM+MAtErQrScjYX27zA27GPHZIeaT6GhKfoZiapfFlnhMf4JeAwwWEEKg nSYUt0a/6k7tZgRfgQvXvojtH/ZXTAbRYCKhSScJeQKixp8pDfNA/6aPt0pC+Wkk+PVI Y0ZCGzsJFzaIh0ef+a0UK5EZmhd1U/p6h0E/EO63AoKuwQlOsR0YZsQQgTHo1KuOK0Nk FLguTEWqJ6+d93ybG/lwPxAfgahDYSP9lK7lrfzB34mDl6+B3TxrA0jHt91SWgN+Pr3n 0LLw== X-Forwarded-Encrypted: i=1; AKwUvBzGB0qOG+CB/HVn4hvdt7NF823k2BvkrAg7v2ELe/UCYlfiARaoVRMZPk9P7M+elWGz6KaJ67RaZb9Qu/I=@vger.kernel.org X-Gm-Message-State: AFuF++nQhFBHs9gIo0qCEMn9JIvx7ZI+D7UEoHcOkS44/Kq+JkobDLRr hywdsRBzsFGnVRDbUAbQBNobAx2ye0yYzzF7vzDe60if8zosTurF+dikhkpXlWVLZIM= X-Gm-Gg: AYBFou20zfrch4wKyD7zRa2+sNBuz2N7nxoHNdjUXwi8vdEjHaDig2cENUQPLs370yt 5B8KW5zedvGP5zX+A/XtyRyORRp15cG7Z+96esz4TScX5L0hOjPHgh7LjJhb2hX+KgjRHqJ7sk2 hNfdP3/zL4qayI97IK9GKgsBkQagQy/NDEUoUrGPPtDwrJ/qNtb54sxwsbxkCdJG2MFst6QqEmB 1PTmwDBCu85/S40lQUu3KhPrNGdeZStN1hjHaDxf8pip7qxnRDIveuagWI4TRloycDQFyoxt6Cj LbuWX9MRxhJWuIAtbd7TJ8ffdYsYuEaEkT7lhcphK12qLumnWJrjFXYdOWCCUF9xBPSjqK7aWRX frsmmS4bz5NhXhC+29hR7WFPE5HfA/1F37Gx1P5K+3A8eOI9wjtAWyiPBzr4V7Fvl8MVFui5riX 7KaoLN1m8UVxKMekFWUTZOJNYuJhqdIGgQtVVbZmxonI7po1i4k5TSjw8f5zL4U6yvzrxnUs6QY PLENESGPmRwQfmX2BcavdoXtwX6Ap/5yrb3UmQ+yVvYvXbc0a8mh6IGBvie X-Received: by 2002:a17:907:5c9:b0:c26:1691:b36d with SMTP id a640c23a62f3a-c2966a6e21fmr221558666b.38.1789143659129; Fri, 11 Sep 2026 09:20:59 -0700 (PDT) Received: from localhost ([94.4.85.166]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c2965c4e248sm109763966b.2.2026.09.11.09.20.57 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 11 Sep 2026 09:20:58 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Fri, 11 Sep 2026 17:20:57 +0100 Message-Id: Cc: "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Alim Akhtar" , "Peter Griffin" , "Tudor Ambarus" , , , , , "Krzysztof Kozlowski" Subject: Re: [PATCH v3 1/2] dt-bindings: sram: Document samsung,exynos850-acpm-sram From: "Alexey Klimov" To: "Sam Protsenko" , "Alexey Klimov" X-Mailer: aerc 0.21.0 References: <20260727-exynos850_sram-v3-0-8627a49188ec@linaro.org> <20260727-exynos850_sram-v3-1-8627a49188ec@linaro.org> In-Reply-To: On Fri Sep 11, 2026 at 1:42 AM BST, Sam Protsenko wrote: > On Mon, Jul 27, 2026 at 8:23=E2=80=AFAM Alexey Klimov wrote: >> >> Document "samsung,exynos850-acpm-sram" which represents an SRAM region u= sed >> for ACPM communication on Samsung-based SoCs. >> >> The SRAM is consumed as a whole, is not split up using child nodes and >> properties related to this are not required. >> >> Reviewed-by: Krzysztof Kozlowski >> Signed-off-by: Alexey Klimov >> --- >> Documentation/devicetree/bindings/sram/sram.yaml | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/sram/sram.yaml b/Document= ation/devicetree/bindings/sram/sram.yaml >> index b83fc6831947..b9c4d4f3199b 100644 >> --- a/Documentation/devicetree/bindings/sram/sram.yaml >> +++ b/Documentation/devicetree/bindings/sram/sram.yaml >> @@ -43,6 +43,7 @@ properties: >> - qcom,rpm-msg-ram >> - qcom,shikra-imem >> - rockchip,rk3288-pmu-sram >> + - samsung,exynos850-acpm-sram > > Is there a reason we can't just use "mmio-sram" for Exynos850 ACPM > needs? I can see it's used in the gs101 device tree like this. If > there is a reason for this, I think it might be a good idea to > document it in the commit message (probably in both patches). But isn't it already in this commit message? "The SRAM is consumed as a whole, is not split up using child nodes and properties related to this are not required." Also: https://lore.kernel.org/all/5797be19-8886-45a2-bfce-eb7d5f0df6c2@lina= ro.org/ Do you want it to be re-phrased smth like this: "OSPM ACPM should parse the SRAM and discover essential data (e.g. queue offsets) for establishing ACPM communication. At this point it is not expected for any software entities apart ACPM to use SRAM so the SRAM is consumed as a whole, is not split up using child nodes and properties related to this are not required." or "The OSPM/ACPM IPC should parse the SRAM at runtime to find configuration data (e.g., queue offsets). Because this SRAM region is dedicated exclusively to ACPM IPC communication and is not shared with other software entities, it is represented as a monolithic SRAM block without requiring sub-node partitioning." ? I hope if there will be other software entities then we will be able to fix it later. >Also, > should "mmio-sram" be added as a backup option (after comma) in your > [PATCH 2/2]? Sure, I can add "mmio-sram". The compatible =3D "rockchip,rk3288-pmu-sram", "mmio-sram" and compatible =3D "qcom,rpm-msg-ram", "mmio-sram" are used in such way. Best regards, Alexey