[PATCH v3 4/5] cmd/arm: meson: sm: introduce efusedump command

[Neil Armstrong]

Le 22/12/2022 à 13:57, Alexey Romanov via groups.io a écrit :
> Using this command user can print efuse memory:
> 
> $ sm efusedump 0 10
>    00000000: ff 00 31 00 00 ff 66 00 00 00      ..1...f...
> 
> Signed-off-by: Alexey Romanov <avromanov at sberdevices.ru>
> Reviewed-by: Mattijs Korpershoek <mkorpershoek at baylibre.com>
> ---
>   cmd/meson/sm.c | 36 +++++++++++++++++++++++++++++++++++-
>   1 file changed, 35 insertions(+), 1 deletion(-)
> 
> diff --git a/cmd/meson/sm.c b/cmd/meson/sm.c
> index c2b57707258..d5eb1d6c084 100644
> --- a/cmd/meson/sm.c
> +++ b/cmd/meson/sm.c
> @@ -5,6 +5,7 @@
>    * Author: Beniamino Galvani <b.galvani at gmail.com>
>    * Author: Vyacheslav Bocharov <adeep at lexina.in>
>    * Author: Neil Armstrong <narmstrong at baylibre.com>
> + * Author: Alexey Romanov <avromanov at sberdevices.ru>
>    */
>   
>   #include <command.h>
> @@ -119,11 +120,43 @@ static int do_efuse_write(struct cmd_tbl *cmdtp, int flag, int argc,
>   	return CMD_RET_SUCCESS;
>   }
>   
> +static int do_efuse_dump(struct cmd_tbl *cmdtp, int flag, int argc,
> +			char *const argv[])
> +{
> +	ulong offset, size;
> +	u8 *buffer;
> +	int ret;
> +
> +	if (argc != 3)
> +		return CMD_RET_USAGE;
> +
> +	offset = simple_strtoul(argv[1], NULL, 0);
> +	size = simple_strtoul(argv[2], NULL, 0);
> +	buffer = malloc(size);
> +	if (!buffer) {
> +		pr_err("Failed to allocate %lu bytes\n", size);
> +		return CMD_RET_FAILURE;
> +	}
> +
> +	ret = meson_sm_read_efuse(offset, (void *)buffer, size);
> +	if (ret != size) {
> +		ret = CMD_RET_FAILURE;
> +		goto free_buffer;
> +	}
> +
> +	print_buffer(0, buffer, 1, size, 0);
> +
> +free_buffer:
> +	free(buffer);
> +	return ret;
> +}
> +
>   static struct cmd_tbl cmd_sm_sub[] = {
>   	U_BOOT_CMD_MKENT(serial, 2, 1, do_sm_serial, "", ""),
>   	U_BOOT_CMD_MKENT(reboot_reason, 1, 1, do_sm_reboot_reason, "", ""),
>   	U_BOOT_CMD_MKENT(efuseread, 4, 1, do_efuse_read, "", ""),
>   	U_BOOT_CMD_MKENT(efusewrite, 4, 0, do_efuse_write, "", ""),
> +	U_BOOT_CMD_MKENT(efusedump, 3, 1, do_efuse_dump, "", ""),
>   };
>   
>   static int do_sm(struct cmd_tbl *cmdtp, int flag, int argc,
> @@ -152,5 +185,6 @@ U_BOOT_CMD(
>   	"serial <address> - read chip unique id to memory address\n"
>   	"sm reboot_reason [name] - get reboot reason and store to environment\n"
>   	"sm efuseread <offset> <size> <address> - read efuse to memory address\n"
> -	"sm efusewrite <offset> <size> <address> - write into efuse from memory address"
> +	"sm efusewrite <offset> <size> <address> - write into efuse from memory address\n"
> +	"sm efusedump <offset> <size> - dump efuse data range to console"
>   );

Reviewed-by: Neil Armstrong <neil.armstrong at linaro.org>