Re: [PATCH v2 1/2] common: misc: add soc_uid_hex to globalvar

[Sascha Hauer <s.hauer@pengutronix.de>]

On Wed, Apr 22, 2026 at 11:19:02AM +0200, Fabian Pflug wrote:
> On Wed, 2026-04-22 at 11:16 +0200, Sascha Hauer wrote:
> > On Wed, Apr 22, 2026 at 10:52:34AM +0200, Fabian Pflug wrote:
> > > 
> > > There already is
> > > 
> > > BAREBOX_MAGICVAR(global.soc_uid_hex, "SoC Unique ID (in hex)");
> > > 
> > > And since it is a single value and not a byte array, I think byte array is the wrong representation.
> > > Using a byte-array for TLV does not work with the TLV tooling. This expects a single value.
> > 
> > You are right, byte array is not the correct term here. What I wanted to
> > make clear is that it is the raw buffer content with the first byte in
> > the buffer being the first two digits of the string, just to make the
> > difference between these two clearer:
> > 
> > global.soc_uid=0011223344556677		"SoC Unique ID"
> > global.soc_uid_hex=7766554433221100	"SoC Unique ID (in hex)"
> > 
> > Both are obviously hex values, which one means what?
> 
> maybe "Raw SoC Unique ID (vendor independent representation)"?

I think

"Raw SoC Unique ID representation"

is enough. It's not necessarily the vendors that made up the Linux representation.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |