Hi, On 4/22/22 14:02, Andrew Davis wrote: > On 4/21/22 3:36 PM, Dave Gerlach wrote: >> Add documentation for the ti,ctx-memory-region property which is a >> phandle to a reserved-memory carveout to be used by the ti_sci driver >> storage of low power mode memory context. This is optional for normal >> system operation but required to enabled suspend-to-mem usage of Deep >> Sleep state. >> >> Signed-off-by: Dave Gerlach <d-gerlach@xxxxxx> >> --- >> .../devicetree/bindings/arm/keystone/ti,sci.yaml | 9 +++++++++ >> 1 file changed, 9 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml >> index 34f5f877d444..ec88aa88a2a0 100644 >> --- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml >> +++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml >> @@ -61,6 +61,15 @@ properties: >> mboxes: >> minItems: 2 >> >> + ti,ctx-memory-region: >> + description: >> + Phandle to the reserved memory node to be associated with the >> + ti-sci device, to be used for saving low power context. The >> + reserved memory node should be a carveout node, and should >> + be defined as per the bindings in >> + Documentation/devicetree/bindings/reserved-memory/reserved-memory.yaml >> + $ref: /schemas/types.yaml#/definitions/string >> + > > > Why does this have to be yet another reserved carveout region, > should be dynamically allocated. > This must be a fixed address in order to support other low power modes which have not yet been introduced. Regards, Dave > Andrew > > >> ti,system-reboot-controller: >> description: Determines If system reboot can be triggered by SoC reboot >> type: boolean
