Re: [PATCH 0/3] Add eFuse access for ZynqMP

2024-05-15 Thread Marek BehĂșn
On Tue, 14 May 2024 16:04:13 +0200
lukas.funke-...@weidmueller.com wrote:

> From: Lukas Funke 
> 
> 
> This series adds a driver to read and write ZynqMP eFuses [1]. The
> driver can be accessed by the 'efuse_read' and 'efuse_write' subcommands
> of the 'zynqmp' command.

Vendor specific commands aren't great.

There is the 'fuse' command in u-boot. You need to implement the
  fuse_read()
  fuse_sense()
  fuse_prog()
  fuse_override()
functions.

See for example arch/arm/mach-mvebu/efuse.c, or other implementations.

Please don't invent new vendor specific commands in new code, it is an
antipattern.

Marek


[PATCH 0/3] Add eFuse access for ZynqMP

2024-05-14 Thread lukas . funke-oss
From: Lukas Funke 


This series adds a driver to read and write ZynqMP eFuses [1]. The
driver can be accessed by the 'efuse_read' and 'efuse_write' subcommands
of the 'zynqmp' command.

Example:

=> zynqmp efuse_read 0xc 0xc
: 85 66 b1 32 43 f2 4a 02 00 00 00 40  .f.https://docs.amd.com/r/en-US/ug1085-zynq-ultrascale-trm/eFUSE



Lukas Funke (3):
  firmware: zynqmp: Add support to access efuses
  amd64: zynqmp: Add command to program efuses
  drivers: misc: Add driver to access ZynqMP efuses

 board/xilinx/zynqmp/cmds.c | 101 ++
 drivers/firmware/firmware-zynqmp.c |  31 +
 drivers/misc/Kconfig   |   8 ++
 drivers/misc/Makefile  |   1 +
 drivers/misc/zynqmp_efuse.c| 213 +
 include/zynqmp_firmware.h  |   2 +
 6 files changed, 356 insertions(+)
 create mode 100644 drivers/misc/zynqmp_efuse.c

-- 
2.30.2