u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: nvmem: u-boot,env: add basic NVMEM cells
@ 2022-07-03  8:48 Rafał Miłecki
  2022-08-16  9:26 ` Rafał Miłecki
  2022-09-13 17:45 ` Rob Herring
  0 siblings, 2 replies; 4+ messages in thread
From: Rafał Miłecki @ 2022-07-03  8:48 UTC (permalink / raw)
  To: Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski
  Cc: Tom Rini, Ahmad Fatoum, linux-arm-kernel, u-boot, devicetree,
	linux-kernel, Rafał Miłecki

From: Rafał Miłecki <rafal@milecki.pl>

U-Boot doesn't have cells at hardcoded addresses. They are stored in
internal format. It's still important to define relevant cells in DT so
NVMEM consumers can reference them.

Update binding to allow including basic cells as NVMEM device subnodes.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
 .../devicetree/bindings/nvmem/u-boot,env.yaml       | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/Documentation/devicetree/bindings/nvmem/u-boot,env.yaml b/Documentation/devicetree/bindings/nvmem/u-boot,env.yaml
index e70b2a60cb9a..e96bca99f2d9 100644
--- a/Documentation/devicetree/bindings/nvmem/u-boot,env.yaml
+++ b/Documentation/devicetree/bindings/nvmem/u-boot,env.yaml
@@ -24,6 +24,8 @@ description: |
   Right now only flash partition case is covered but it may be extended to e.g.
   UBI volumes in the future.
 
+  Variables can be defined as NVMEM device subnodes.
+
 maintainers:
   - Rafał Miłecki <rafal@milecki.pl>
 
@@ -40,6 +42,14 @@ properties:
   reg:
     maxItems: 1
 
+  bootcmd:
+    type: object
+    description: Command to use for automatic booting
+
+  ethaddr:
+    type: object
+    description: Ethernet interface's MAC address
+
 additionalProperties: false
 
 examples:
@@ -58,5 +68,8 @@ examples:
         env: partition@40000 {
             compatible = "u-boot,env";
             reg = <0x40000 0x10000>;
+
+            mac: ethaddr {
+            };
         };
     };
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: nvmem: u-boot,env: add basic NVMEM cells
  2022-07-03  8:48 [PATCH] dt-bindings: nvmem: u-boot,env: add basic NVMEM cells Rafał Miłecki
@ 2022-08-16  9:26 ` Rafał Miłecki
  2022-08-30 21:11   ` Tom Rini
  2022-09-13 17:45 ` Rob Herring
  1 sibling, 1 reply; 4+ messages in thread
From: Rafał Miłecki @ 2022-08-16  9:26 UTC (permalink / raw)
  To: Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski, Tom Rini
  Cc: Ahmad Fatoum, linux-arm-kernel, u-boot, devicetree, linux-kernel,
	Rafał Miłecki

On 3.07.2022 10:48, Rafał Miłecki wrote:
> U-Boot doesn't have cells at hardcoded addresses. They are stored in
> internal format. It's still important to define relevant cells in DT so
> NVMEM consumers can reference them.
> 
> Update binding to allow including basic cells as NVMEM device subnodes.

Ping :)


For a reference you can see Broadcom's NVRAM (identical feature):

084973e944bec ("dt-bindings: nvmem: brcm,nvram: add basic NVMEM cells")
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=084973e944bec21804f8afb0515b25434438699a

c8442f0fb09ca ("ARM: dts: BCM5301X: Add Ethernet MAC address to Luxul XWR-3150")
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=c8442f0fb09ca3d842b9b23d1d0650f649fd10f8

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: nvmem: u-boot,env: add basic NVMEM cells
  2022-08-16  9:26 ` Rafał Miłecki
@ 2022-08-30 21:11   ` Tom Rini
  0 siblings, 0 replies; 4+ messages in thread
From: Tom Rini @ 2022-08-30 21:11 UTC (permalink / raw)
  To: Rafał Miłecki
  Cc: Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski,
	Ahmad Fatoum, linux-arm-kernel, u-boot, devicetree, linux-kernel,
	Rafał Miłecki

[-- Attachment #1: Type: text/plain, Size: 969 bytes --]

On Tue, Aug 16, 2022 at 11:26:37AM +0200, Rafał Miłecki wrote:
> On 3.07.2022 10:48, Rafał Miłecki wrote:
> > U-Boot doesn't have cells at hardcoded addresses. They are stored in
> > internal format. It's still important to define relevant cells in DT so
> > NVMEM consumers can reference them.
> > 
> > Update binding to allow including basic cells as NVMEM device subnodes.
> 
> Ping :)
> 
> 
> For a reference you can see Broadcom's NVRAM (identical feature):
> 
> 084973e944bec ("dt-bindings: nvmem: brcm,nvram: add basic NVMEM cells")
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=084973e944bec21804f8afb0515b25434438699a
> 
> c8442f0fb09ca ("ARM: dts: BCM5301X: Add Ethernet MAC address to Luxul XWR-3150")
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=c8442f0fb09ca3d842b9b23d1d0650f649fd10f8

Seems fine to me.

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: nvmem: u-boot,env: add basic NVMEM cells
  2022-07-03  8:48 [PATCH] dt-bindings: nvmem: u-boot,env: add basic NVMEM cells Rafał Miłecki
  2022-08-16  9:26 ` Rafał Miłecki
@ 2022-09-13 17:45 ` Rob Herring
  1 sibling, 0 replies; 4+ messages in thread
From: Rob Herring @ 2022-09-13 17:45 UTC (permalink / raw)
  To: Rafał Miłecki
  Cc: Rob Herring, Rafał Miłecki, linux-kernel,
	Krzysztof Kozlowski, linux-arm-kernel, Srinivas Kandagatla,
	u-boot, Ahmad Fatoum, devicetree, Tom Rini

On Sun, 03 Jul 2022 10:48:43 +0200, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
> 
> U-Boot doesn't have cells at hardcoded addresses. They are stored in
> internal format. It's still important to define relevant cells in DT so
> NVMEM consumers can reference them.
> 
> Update binding to allow including basic cells as NVMEM device subnodes.
> 
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
>  .../devicetree/bindings/nvmem/u-boot,env.yaml       | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 

Sorry, I guess I was thinking this depended on or interacted with one of 
the other nvmem series.

Applied, thanks!

Rob

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-09-13 17:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-03  8:48 [PATCH] dt-bindings: nvmem: u-boot,env: add basic NVMEM cells Rafał Miłecki
2022-08-16  9:26 ` Rafał Miłecki
2022-08-30 21:11   ` Tom Rini
2022-09-13 17:45 ` Rob Herring

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).