diff mbox series

[v4,1/5] dt-bindings: ti, sci: Add lpm region and firmware-name

Message ID 20221116181307.198209-2-g-vlaev@ti.com (mailing list archive)
State New, archived
Headers show
Series firmware: ti_sci: Introduce system suspend support | expand

Commit Message

Georgi Vlaev Nov. 16, 2022, 6:13 p.m. UTC 
From: Dave Gerlach <d-gerlach@ti.com>

Add documentation for the lpm region which tells the ti-sci driver where
to load the FS Stub low power mode firmware and also the firmware-name
which tells the driver which binary to load. Both of these are optional
for normal system operation but required to enabled suspend-to-mem usage
of Deep Sleep state.

Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
Signed-off-by: Vibhore Vardhan <vibhore@ti.com>
Signed-off-by: Georgi Vlaev <g-vlaev@ti.com>
---
 .../bindings/arm/keystone/ti,sci.yaml         | 21 +++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

Comments

Krzysztof Kozlowski Nov. 18, 2022, 12:59 p.m. UTC | #1 
On 16/11/2022 19:13, Georgi Vlaev wrote:
> From: Dave Gerlach <d-gerlach@ti.com>
> 
> Add documentation for the lpm region which tells the ti-sci driver where
> to load the FS Stub low power mode firmware and also the firmware-name
> which tells the driver which binary to load. Both of these are optional
> for normal system operation but required to enabled suspend-to-mem usage
> of Deep Sleep state.
> 

I think you got here Rob's tag after sending v4.

Reviewed-by: Rob Herring <robh@kernel.org>

Best regards,
Krzysztof
Georgi Vlaev Nov. 21, 2022, 10:26 p.m. UTC | #2 
Hi,

On 11/18/22 14:59, Krzysztof Kozlowski wrote:
> On 16/11/2022 19:13, Georgi Vlaev wrote:
>> From: Dave Gerlach <d-gerlach@ti.com>
>>
>> Add documentation for the lpm region which tells the ti-sci driver where
>> to load the FS Stub low power mode firmware and also the firmware-name
>> which tells the driver which binary to load. Both of these are optional
>> for normal system operation but required to enabled suspend-to-mem usage
>> of Deep Sleep state.
>>
> 
> I think you got here Rob's tag after sending v4.
> 
> Reviewed-by: Rob Herring <robh@kernel.org>
> 

I will pick it up in v5.
Thanks.

> Best regards,
> Krzysztof
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
index 34f5f877d444..4c5213de24bc 100644
--- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
+++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
@@ -42,12 +42,19 @@  properties:
           - const: ti,am654-sci
 
   reg-names:
-    description: |
-      Specifies the debug messages memory mapped region that is optionally
-      made available from TI-SCI controller.
-    const: debug_messages
+    items:
+      - const: debug_messages
+      - const: lpm
+    minItems: 1
 
   reg:
+    items:
+      - description: |
+          Specifies the debug messages memory mapped region that is optionally
+          made available from TI-SCI controller.
+      - description: |
+          Specifies the lpm memory mapped region where the FS Stub low power
+          firmware is to be loaded.
     minItems: 1
 
   mbox-names:
@@ -83,6 +90,12 @@  properties:
     type: object
     $ref: /schemas/reset/ti,sci-reset.yaml#
 
+  firmware-name:
+    description: |
+      Name of binary of FS Stub low power firmware located on the
+      firmware search path.
+    $ref: /schemas/types.yaml#/definitions/string
+
 required:
   - compatible
   - mbox-names