From: Simon Glass <sjg@chromium.org>
To: devicetree@vger.kernel.org
Cc: Dhaval Sharma <dhaval@rivosinc.com>,
Mark Rutland <mark.rutland@arm.com>, Gua Guo <gua.guo@intel.com>,
Tom Rini <trini@konsulko.com>, Rob Herring <robh@kernel.org>,
U-Boot Mailing List <u-boot@lists.denx.de>,
ron minnich <rminnich@gmail.com>,
Ard Biesheuvel <ardb@kernel.org>,
Lean Sheng Tan <sheng.tan@9elements.com>,
Maximilian Brune <maximilian.brune@9elements.com>,
Chiu Chasel <chasel.chiu@intel.com>,
lkml <linux-kernel@vger.kernel.org>,
Yunhui Cui <cuiyunhui@bytedance.com>,
linux-acpi@vger.kernel.org, Guo Dong <guo.dong@intel.com>,
Simon Glass <sjg@chromium.org>
Subject: [PATCH v4 3/4] schemas: Add a schema for memory map
Date: Tue, 29 Aug 2023 13:18:08 -0600 [thread overview]
Message-ID: <20230829191812.135759-3-sjg@chromium.org> (raw)
In-Reply-To: <20230829191812.135759-1-sjg@chromium.org>
The Devicetree specification skips over handling of a logical view of
the memory map, pointing users to the UEFI specification.
It is common to split firmware into 'Platform Init', which does the
initial hardware setup and a "Payload" which selects the OS to be booted.
Thus an handover interface is required between these two pieces.
Where UEFI boot-time services are not available, but UEFI firmware is
present on either side of this interface, information about memory usage
and attributes must be presented to the "Payload" in some form.
This aims to provide an initial schema for this mapping.
Note that this is separate from the existing /memory and /reserved-memory
nodes, since it is mostly concerned with what the memory is used for. It
may cover only a small fraction of available memory.
For now, no attempt is made to create an exhaustive binding, so there are
some example types listed. This can be completed once this has passed
initial review.
The compatible string is not included, since the node name is enough to
indicate the purpose of a node, as per the existing reserved-memory
schema.
This binding does not include a binding for the memory 'attribute'
property, defined by EFI_BOOT_SERVICES.GetMemoryMap(). It may be useful
to have that as well, but perhaps not as a bit mask.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
Changes in v4:
- Make use of the reserved-memory node instead of creating a new one
Changes in v3:
- Reword commit message again
- cc a lot more people, from the FFI patch
- Split out the attributes into the /memory nodes
Changes in v2:
- Reword commit message
dtschema/schemas/memory-map.yaml | 61 +++++++++++++++++++
.../reserved-memory/common-reserved.yaml | 53 ++++++++++++++++
2 files changed, 114 insertions(+)
create mode 100644 dtschema/schemas/memory-map.yaml
create mode 100644 dtschema/schemas/reserved-memory/common-reserved.yaml
diff --git a/dtschema/schemas/memory-map.yaml b/dtschema/schemas/memory-map.yaml
new file mode 100644
index 0000000..4b06583
--- /dev/null
+++ b/dtschema/schemas/memory-map.yaml
@@ -0,0 +1,61 @@
+# SPDX-License-Identifier: BSD-2-Clause
+# Copyright 2023 Google LLC
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/memory-map.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: /memory-map nodes
+description: |
+ Common properties always required in /memory-map nodes. These nodes are
+ intended to resolve the nonchalant clause 3.4.1 ("/memory node and UEFI")
+ in the Devicetree Specification.
+
+maintainers:
+ - Simon Glass <sjg@chromium.org>
+
+properties:
+ $nodename:
+ const: 'memory-map'
+
+patternProperties:
+ "^([a-z][a-z0-9\\-]+@[0-9a-f]+)?$":
+ type: object
+ additionalProperties: false
+
+ properties:
+ reg:
+ minItems: 1
+ maxItems: 1024
+
+ usage:
+ $ref: /schemas/types.yaml#/definitions/string
+ description: |
+ Describes the usage of the memory region, e.g.:
+
+ "acpi-reclaim", "acpi-nvs", "bootcode", "bootdata", "bootdata",
+ "runtime-code", "runtime-data".
+
+ See enum EFI_MEMORY_TYPE in "Unified Extensible Firmware Interface
+ (UEFI) Specification" for all the types. For now there are not
+ listed here.
+
+ required:
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ memory-map {
+ acpi@f0000 {
+ reg = <0xf0000 0x4000>;
+ usage = "acpi-reclaim";
+ };
+
+ runtime@12300000 {
+ reg = <0x12300000 0x28000>;
+ usage = "runtime-code";
+ };
+ };
+...
diff --git a/dtschema/schemas/reserved-memory/common-reserved.yaml b/dtschema/schemas/reserved-memory/common-reserved.yaml
new file mode 100644
index 0000000..70cdead
--- /dev/null
+++ b/dtschema/schemas/reserved-memory/common-reserved.yaml
@@ -0,0 +1,53 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/reserved-memory/common-reserved.yaml#
+$schema: http://devicetree.org/meta-schemas/base.yaml#
+
+title: Common memory reservations
+
+description: |
+ Specifies that the reserved memory region can be used for the purpose
+ indicated by its node name.
+
+ Clients may reuse this reserved memory if they understand what it is for.
+
+maintainers:
+ - Simon Glass <sjg@chromium.org>
+
+# always select the core schema
+select: true
+
+properties:
+ $nodename:
+ oneOf:
+ - acpi-reclaim
+ - acpi-nvs
+ - boot-code
+ - boot-data
+ - runtime-code
+ - runtime-data
+
+ reg:
+ description: region of memory that is reserved for the purpose indicated
+ by the node name.
+
+required:
+ - reg
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ boot-code@12340000 {
+ reg = <0x12340000 0x00800000>;
+ };
+
+ boot-data@43210000 {
+ reg = <0x43210000 0x00800000>;
+ };
+ };
--
2.42.0.rc2.253.gd59a3bf2b4-goog
next prev parent reply other threads:[~2023-08-29 19:19 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-29 19:18 [PATCH v4 1/4] Add reserved-memory Simon Glass
2023-08-29 19:18 ` [PATCH v4 2/4] Bring in other reserved-memory files Simon Glass
2023-08-29 21:05 ` Rob Herring
2023-08-29 19:18 ` Simon Glass [this message]
2023-08-29 19:18 ` [PATCH v4 4/4] memory: Add ECC property Simon Glass
2023-08-29 21:37 ` Rob Herring
[not found] ` <CAMWxwJ13-itCyyKaH21nncaX2OUesGwpfMNHocvDEKHzZ1_F4Q@mail.gmail.com>
2023-09-06 14:48 ` Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230829191812.135759-3-sjg@chromium.org \
--to=sjg@chromium.org \
--cc=ardb@kernel.org \
--cc=chasel.chiu@intel.com \
--cc=cuiyunhui@bytedance.com \
--cc=devicetree@vger.kernel.org \
--cc=dhaval@rivosinc.com \
--cc=gua.guo@intel.com \
--cc=guo.dong@intel.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=maximilian.brune@9elements.com \
--cc=rminnich@gmail.com \
--cc=robh@kernel.org \
--cc=sheng.tan@9elements.com \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).