From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (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 5D7DC3148D0 for ; Thu, 30 Apr 2026 15:37:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777563452; cv=none; b=UxXVdsy39eX4xC1OWvkpbbH5MHgR87OsLyt2ObSZzP9VB8GXUSsHEQU9Xyczc0JqsZyyFhjVL/K/wWmQtCbNPLsf2RKvQqv+X0m1a11kkmROoY/bGhzw/pJbOEpKDIUJQ5wiugmO0cfeRxGkJSJM7O76fZmLhnRcS7SmSzt46NE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777563452; c=relaxed/simple; bh=D8ctGVVQEch5tunpDf4+eVEVa+9ZQoaz/XYhpFXnBKk=; h=Mime-Version:Content-Type:Date:Message-Id:Subject:From:To:Cc: References:In-Reply-To; b=UiSQDqCME4elYgP2I+le+CL0ereStdlLyMZ/3gwdB2OrJibhwQfVs3we8NPARFqCO9G23XW2Z6ZljJRH497bBU7mczi9e++No+s22JvooFZERBxiRCh7iOrJ3vr8uhTtC8747QX0vNMWmzvtHffkM45aWGXXBA5kwu76ihA3iHw= 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=HipJ19fU; arc=none smtp.client-ip=209.85.128.46 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="HipJ19fU" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-488ff90d6c7so10548395e9.2 for ; Thu, 30 Apr 2026 08:37:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1777563446; x=1778168246; darn=vger.kernel.org; h=in-reply-to:references:cc:to:from:subject:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=boh2FGqR0WpDSM6dhSTP3kihWh+3GkokB90DFYaafvo=; b=HipJ19fUAEq33OWW/Q5wLCcOkQz4TNsdlYRpHK2OgdU1xSTGX5gYTi5ltEpc9KxlyG rWk4XWAYn/0Kh4r96B+/Ml36YkDJ+KoZl6I6CQXXlNMJrozrmbIMTNO+1QiPY/+X/nEF lVcw3tXGvvESxcFtQOcWGO/Uy4U4Vrqs6S6WqtnN8VYi5Vj4SurG1M20D+e1RRKxAPKJ o3+zdVz3KxLhTRBjslmdaSOT1Lokn2xvSTEQI8JdmfGqehNzrWK9V1RRk4unKtJ1BI6/ q3E4tQ1FrtQ7z1ZQcR0wYJpaQNHeynZrLDo8uiXFyo3AT6gokrT/inex1mKruQ1e/HkK mY3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777563446; x=1778168246; h=in-reply-to:references:cc:to:from:subject:message-id:date :content-transfer-encoding:mime-version:x-gm-gg:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=boh2FGqR0WpDSM6dhSTP3kihWh+3GkokB90DFYaafvo=; b=QjkQnC7NgRjzd1h3uZyM35jQOeJgUddAwWd15xKuMJ+7hPJYzCtioZUpMhCIQenU2E smAgpo/oCrd4MXcMCtP9+3A7lcomGvuBt2ngxldW4OVKaRZ6+JtWZ7Mju0P4hTZdLOl8 KpAd8nGSpDAZv/Gge9pTxZxxIsewwppwGuUBmNRPGzajJugl4iJ3VONxZceDYRTux99r Eilb2GiMyNS6uDRrBnkouJyutstbs9AVjV9TAdN+V2NzHMY3SuXwIYcya80kG4onzDYZ 8uJzPu1HqsL91/YzTxf3zFuzsUmcFU3y6VLMyCAkqylnVe+mzWFqp61dDEAkk28kTWZ/ TCRw== X-Forwarded-Encrypted: i=1; AFNElJ+4DEZwhvSM+a8Y8tAx/SSwRP0NXikrv7jzsJ8FpDF53YqsZAsbzmtjeCUIkJEjm0g8WYSaFE6Djl0/tKs=@vger.kernel.org X-Gm-Message-State: AOJu0Yy9M9S22AbFShJwJ7eMRs+f9g0Ksn1kf/geTNHcHMmb44gjqy6Y Vi05I5rZpd6wmc2vIFvLJph3mcxQrLU/d0+90CWE5pDkti3c4w2FhcbUloOXTeCGjzY= X-Gm-Gg: AeBDievwjt++RHIilv2tc03LFyXAyil10OVm2/FCEioVJKs0mywZeCB7Pr2FCuHeW6Y rbmmr/tIbbfwyxbwX7f/1Yg/FY/vG2RZxiuBpWKyI4dEL3TLF6D4o8nP565jr94TjkhkcWrYsQq wEIFXCojYS+B4PkT1rkuMqW5ohlfwIoAqtCtjYO32duMIWY4QhU2tkknM5rsxwZLqwkbHXphNqY shtTHlwjWmwcWBcEXCZ1mAK2oldVUHa+1qz/QH3CUhn7Z23v1535vBc3UgXOuGXxVahlxHC+jC1 2zg6K5qsFp9fjiKLWlfT/g9mAYBFBeQcD/hHC0HuZwcHA33cU/CB35nqPYfrj1jBx1rqm/myZfB wkhMOFUpODNMm4Ck6Rb/nO1htkrAatQmLi+ZXX+Yr210XuyUTcvWU6jyw5FSs8BXysnIZewLvVE 3Q0kU6Mv4wgDyJn6QcfRPn++LwC7jT73W8It50Z4iik1loQp62oN2f+r7mJuU6QfmxPsD2VScQp R1oRIlefutczhbOhL5wE0leQA== X-Received: by 2002:a05:600c:c058:b0:488:ac01:72de with SMTP id 5b1f17b1804b1-48a83d6a890mr42190595e9.5.1777563445839; Thu, 30 Apr 2026 08:37:25 -0700 (PDT) Received: from localhost ([2a00:2381:fd67:101:b0e1:a8:4337:b2a4]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-48a7b912869sm46165045e9.6.2026.04.30.08.37.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 30 Apr 2026 08:37:25 -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: Thu, 30 Apr 2026 16:37:24 +0100 Message-Id: Subject: Re: [PATCH] arm64: dts: exynos850: Add SRAM node From: "Alexey Klimov" To: "Krzysztof Kozlowski" , "Rob Herring" , "Krzysztof Kozlowski" Cc: , , , , "Sam Protsenko" , "Conor Dooley" , "Alim Akhtar" X-Mailer: aerc 0.20.0 References: <20260413-exynos850_sram-v1-1-7fda5b7fb7d4@linaro.org> <2ff077e1-8983-4a41-bb21-5e4140545aa3@kernel.org> <4c6a92e0-15a1-4f82-afc9-542f5ad9d2df@kernel.org> In-Reply-To: <4c6a92e0-15a1-4f82-afc9-542f5ad9d2df@kernel.org> On Tue Apr 14, 2026 at 10:08 AM BST, Krzysztof Kozlowski wrote: > On 14/04/2026 11:00, Alexey Klimov wrote: >> On Mon Apr 13, 2026 at 4:23 PM BST, Krzysztof Kozlowski wrote: >>> On 13/04/2026 16:52, Alexey Klimov wrote: >>>> SRAM is used by the ACPM protocol to retrieve the ACPM channels >>>> information and configuration data. Add the SRAM node. >>>> >>>> Signed-off-by: Alexey Klimov >>>> --- >>>> arch/arm64/boot/dts/exynos/exynos850.dtsi | 8 ++++++++ >>>> 1 file changed, 8 insertions(+) >>>> >>>> diff --git a/arch/arm64/boot/dts/exynos/exynos850.dtsi b/arch/arm64/bo= ot/dts/exynos/exynos850.dtsi >>>> index cb55015c8dce..cf4a6168846c 100644 >>>> --- a/arch/arm64/boot/dts/exynos/exynos850.dtsi >>>> +++ b/arch/arm64/boot/dts/exynos/exynos850.dtsi >>>> @@ -910,6 +910,14 @@ spi_2: spi@11d20000 { >>>> }; >>>> }; >>>> }; >>>> + >>>> + apm_sram: sram@2039000 { >>>> + compatible =3D "mmio-sram"; >>>> + reg =3D <0x0 0x2039000 0x40000>; >>>> + #address-cells =3D <1>; >>>> + #size-cells =3D <1>; >>>> + ranges =3D <0x0 0x0 0x2039000 0x40000>; >>> >>> You miss here children. >>=20 >> Thank you! I guess I should convert it to smth like this: >>=20 >> apm_sram: sram@2039000 { >> compatible =3D "mmio-sram"; >> reg =3D <0x0 0x2039000 0x40000>; >> ranges =3D <0x0 0x0 0x2039000 0x40000>; >> #address-cells =3D <1>; >> #size-cells =3D <1>; >>=20 >> acpm_sram_region: sram-section@0 { >> reg =3D <0x0 0x40000>; > > This covers entire block, so feels pointless. Maybe requirement of > children should be dropped. What's the point of having children? Why > does the driver need them? Thinking further about this. I was looking at the schema file and different commits and it the right approach could be: - add separate compatible for mmio-sram region for this case. Similar to scmi and scpi, i'd say it should be "exynos,acpm-sram" or "samsung,acpm-sram"; - update schema sram.yaml to exclude requirement for having child nodes and properties like ranges, address and size cells Similar like in commit, for instance, git show d0fe6491ddd22 (or git show 2d4a22e79a5fd). - then this addition becomes: apm_sram: sram@2039000 { compatible =3D "exynos,acpm-sram"; reg =3D <0x0 0x2039000 0x40000>; }; and will be consumed as a whole region (also as Tudor mentioned in the other email). And then we should update gs101 dts as well. Thoughts? I honestly don't know why be default schema requires child nodes unless it is specifically excluded. Could be a good question to Rob? Or maybe we should really drop a requirement for having children from schema. >> And then later reference shmem =3D &acpm_sram_region from acpm node. >>=20 >>> Also, 'ranges' should be after 'reg'. >>=20 >> Thanks, will fix this. >>=20 >> FWIW this commit is a copy of commit 48e7821b26904 >> https://lore.kernel.org/r/20250207-gs101-acpm-dt-v4-1-230ba8663a2d@linar= o.org > > > Huh, we should fix that one as well. Best regardss, Alexey