mirror of
https://source.denx.de/u-boot/u-boot.git
synced 2025-08-08 00:06:59 +02:00
dfu: allow to manage DFU on several devices
Add support of DFU for several interface/device with one command. The format for "dfu_alt_info" in this case is : - <interface> <dev>'='alternate list (';' separated) - each interface is separated by '&' The previous behavior is always supported. One example for NOR (bootloaders) + NAND (rootfs in UBI): U-Boot> env set dfu_alt_info \ "sf 0:0:10000000:0=spl part 0 1;u-boot part 0 2; \ u-boot-env part 0 3&nand 0=UBI partubi 0,3" U-Boot> dfu 0 list DFU alt settings list: dev: SF alt: 0 name: spl layout: RAW_ADDR dev: SF alt: 1 name: ssbl layout: RAW_ADDR dev: SF alt: 2 name: u-boot-env layout: RAW_ADDR dev: NAND alt: 3 name: UBI layout: RAW_ADDR U-Boot> dfu 0 $> dfu-util -l Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\ intf=0, alt=3, name="UBI", serial="002700333338511934383330" Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\ intf=0, alt=2, name="u-boot-env", serial="002700333338511934383330" Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\ intf=0, alt=1, name="u-boot", serial="002700333338511934383330" Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\ intf=0, alt=0, name="spl", serial="002700333338511934383330" Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
This commit is contained in:
parent
9ada683055
commit
febabe3ed4
21
cmd/dfu.c
21
cmd/dfu.c
@ -21,23 +21,28 @@
|
|||||||
static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
|
static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
|
||||||
{
|
{
|
||||||
|
|
||||||
if (argc < 4)
|
if (argc < 2)
|
||||||
return CMD_RET_USAGE;
|
return CMD_RET_USAGE;
|
||||||
|
|
||||||
#ifdef CONFIG_DFU_OVER_USB
|
#ifdef CONFIG_DFU_OVER_USB
|
||||||
char *usb_controller = argv[1];
|
char *usb_controller = argv[1];
|
||||||
#endif
|
#endif
|
||||||
#if defined(CONFIG_DFU_OVER_USB) || defined(CONFIG_DFU_OVER_TFTP)
|
#if defined(CONFIG_DFU_OVER_USB) || defined(CONFIG_DFU_OVER_TFTP)
|
||||||
char *interface = argv[2];
|
char *interface = NULL;
|
||||||
char *devstring = argv[3];
|
char *devstring = NULL;
|
||||||
|
|
||||||
|
if (argc >= 4) {
|
||||||
|
interface = argv[2];
|
||||||
|
devstring = argv[3];
|
||||||
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
int ret = 0;
|
int ret = 0;
|
||||||
#ifdef CONFIG_DFU_OVER_TFTP
|
#ifdef CONFIG_DFU_OVER_TFTP
|
||||||
unsigned long addr = 0;
|
unsigned long addr = 0;
|
||||||
if (!strcmp(argv[1], "tftp")) {
|
if (!strcmp(argv[1], "tftp")) {
|
||||||
if (argc == 5)
|
if (argc == 5 || argc == 3)
|
||||||
addr = simple_strtoul(argv[4], NULL, 0);
|
addr = simple_strtoul(argv[argc - 1], NULL, 0);
|
||||||
|
|
||||||
return update_tftp(addr, interface, devstring);
|
return update_tftp(addr, interface, devstring);
|
||||||
}
|
}
|
||||||
@ -48,7 +53,7 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
|
|||||||
goto done;
|
goto done;
|
||||||
|
|
||||||
ret = CMD_RET_SUCCESS;
|
ret = CMD_RET_SUCCESS;
|
||||||
if (argc > 4 && strcmp(argv[4], "list") == 0) {
|
if (strcmp(argv[argc - 1], "list") == 0) {
|
||||||
dfu_show_entities();
|
dfu_show_entities();
|
||||||
goto done;
|
goto done;
|
||||||
}
|
}
|
||||||
@ -67,7 +72,7 @@ U_BOOT_CMD(dfu, CONFIG_SYS_MAXARGS, 1, do_dfu,
|
|||||||
"Device Firmware Upgrade",
|
"Device Firmware Upgrade",
|
||||||
""
|
""
|
||||||
#ifdef CONFIG_DFU_OVER_USB
|
#ifdef CONFIG_DFU_OVER_USB
|
||||||
"<USB_controller> <interface> <dev> [list]\n"
|
"<USB_controller> [<interface> <dev>] [list]\n"
|
||||||
" - device firmware upgrade via <USB_controller>\n"
|
" - device firmware upgrade via <USB_controller>\n"
|
||||||
" on device <dev>, attached to interface\n"
|
" on device <dev>, attached to interface\n"
|
||||||
" <interface>\n"
|
" <interface>\n"
|
||||||
@ -77,7 +82,7 @@ U_BOOT_CMD(dfu, CONFIG_SYS_MAXARGS, 1, do_dfu,
|
|||||||
#ifdef CONFIG_DFU_OVER_USB
|
#ifdef CONFIG_DFU_OVER_USB
|
||||||
"dfu "
|
"dfu "
|
||||||
#endif
|
#endif
|
||||||
"tftp <interface> <dev> [<addr>]\n"
|
"tftp [<interface> <dev>] [<addr>]\n"
|
||||||
" - device firmware upgrade via TFTP\n"
|
" - device firmware upgrade via TFTP\n"
|
||||||
" on device <dev>, attached to interface\n"
|
" on device <dev>, attached to interface\n"
|
||||||
" <interface>\n"
|
" <interface>\n"
|
||||||
|
@ -42,16 +42,25 @@ Environment variables:
|
|||||||
separated string of information on each alternate:
|
separated string of information on each alternate:
|
||||||
dfu_alt_info="<alt1>;<alt2>;....;<altN>"
|
dfu_alt_info="<alt1>;<alt2>;....;<altN>"
|
||||||
|
|
||||||
|
when only several device are used, the format is:
|
||||||
|
- <interface> <dev>'='alternate list (';' separated)
|
||||||
|
- each interface is separated by '&'
|
||||||
|
dfu_alt_info=\
|
||||||
|
"<interface1> <dev1>=<alt1>;....;<altN>&"\
|
||||||
|
"<interface2> <dev2>=<altN+1>;....;<altM>&"\
|
||||||
|
...\
|
||||||
|
"<interfaceI> <devI>=<altY+1>;....;<altZ>&"
|
||||||
|
|
||||||
"dfu_bufsiz" : size of the DFU buffer, when absent, use
|
"dfu_bufsiz" : size of the DFU buffer, when absent, use
|
||||||
CONFIG_SYS_DFU_DATA_BUF_SIZE (8MiB by default)
|
CONFIG_SYS_DFU_DATA_BUF_SIZE (8MiB by default)
|
||||||
|
|
||||||
"dfu_hash_algo" : name of the hash algorithm to use
|
"dfu_hash_algo" : name of the hash algorithm to use
|
||||||
|
|
||||||
Commands:
|
Commands:
|
||||||
dfu <USB_controller> <interface> <dev> list
|
dfu <USB_controller> [<interface> <dev>] list
|
||||||
list the alternate device defined in "dfu_alt_info"
|
list the alternate device defined in "dfu_alt_info"
|
||||||
|
|
||||||
dfu <USB_controller> <interface> <dev>
|
dfu <USB_controller> [<interface> <dev>]
|
||||||
start the dfu stack on the USB instance with the selected medium
|
start the dfu stack on the USB instance with the selected medium
|
||||||
backend and use the "dfu_alt_info" variable to configure the
|
backend and use the "dfu_alt_info" variable to configure the
|
||||||
alternate setting and link each one with the medium
|
alternate setting and link each one with the medium
|
||||||
@ -95,6 +104,18 @@ Commands:
|
|||||||
|
|
||||||
with <partid> is the MTD partition index
|
with <partid> is the MTD partition index
|
||||||
|
|
||||||
|
<interface> and <dev> are absent:
|
||||||
|
the dfu command to use multiple devices
|
||||||
|
cmd: dfu 0 list
|
||||||
|
cmd: dfu 0
|
||||||
|
"dfu_alt_info" variable provides the list of <interface> <dev> with
|
||||||
|
alternate list separated by '&' with the same format for each <alt>
|
||||||
|
mmc <dev>=<alt1>;....;<altN>
|
||||||
|
nand <dev>=<alt1>;....;<altN>
|
||||||
|
ram <dev>=<alt1>;....;<altN>
|
||||||
|
sf <dev>=<alt1>;....;<altN>
|
||||||
|
|
||||||
|
|
||||||
Host tools:
|
Host tools:
|
||||||
When U-Boot runs the dfu stack, the DFU host tools can be used
|
When U-Boot runs the dfu stack, the DFU host tools can be used
|
||||||
to send/receive firmwares on each configurated alternate.
|
to send/receive firmwares on each configurated alternate.
|
||||||
@ -103,7 +124,7 @@ Host tools:
|
|||||||
specifications(http://dfu-util.sourceforge.net/) which works with U-Boot.
|
specifications(http://dfu-util.sourceforge.net/) which works with U-Boot.
|
||||||
|
|
||||||
Usage:
|
Usage:
|
||||||
Example for firmware located in eMMC or SD card, with:
|
Example 1: firmware located in eMMC or SD card, with:
|
||||||
- alternate 1 (alt=1) for SPL partition (GPT partition 1)
|
- alternate 1 (alt=1) for SPL partition (GPT partition 1)
|
||||||
- alternate 2 (alt=2) for U-Boot partition (GPT partition 2)
|
- alternate 2 (alt=2) for U-Boot partition (GPT partition 2)
|
||||||
|
|
||||||
@ -147,3 +168,34 @@ Usage:
|
|||||||
|
|
||||||
To request a DFU detach and reset the USB connection:
|
To request a DFU detach and reset the USB connection:
|
||||||
$> dfu-util -a 0 -e -R
|
$> dfu-util -a 0 -e -R
|
||||||
|
|
||||||
|
|
||||||
|
Example 2: firmware located in NOR (sf) and NAND, with:
|
||||||
|
- alternate 1 (alt=1) for SPL partition (NOR GPT partition 1)
|
||||||
|
- alternate 2 (alt=2) for U-Boot partition (NOR GPT partition 2)
|
||||||
|
- alternate 3 (alt=3) for U-Boot-env partition (NOR GPT partition 3)
|
||||||
|
- alternate 4 (alt=4) for UBI partition (NAND GPT partition 1)
|
||||||
|
|
||||||
|
U-Boot> env set dfu_alt_info \
|
||||||
|
"sf 0:0:10000000:0=spl part 0 1;u-boot part 0 2; \
|
||||||
|
u-boot-env part 0 3&nand 0=UBI partubi 0,1"
|
||||||
|
|
||||||
|
U-Boot> dfu 0 list
|
||||||
|
|
||||||
|
DFU alt settings list:
|
||||||
|
dev: SF alt: 0 name: spl layout: RAW_ADDR
|
||||||
|
dev: SF alt: 1 name: ssbl layout: RAW_ADDR
|
||||||
|
dev: SF alt: 2 name: u-boot-env layout: RAW_ADDR
|
||||||
|
dev: NAND alt: 3 name: UBI layout: RAW_ADDR
|
||||||
|
|
||||||
|
U-Boot> dfu 0
|
||||||
|
|
||||||
|
$> dfu-util -l
|
||||||
|
Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\
|
||||||
|
intf=0, alt=3, name="UBI", serial="002700333338511934383330"
|
||||||
|
Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\
|
||||||
|
intf=0, alt=2, name="u-boot-env", serial="002700333338511934383330"
|
||||||
|
Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\
|
||||||
|
intf=0, alt=1, name="u-boot", serial="002700333338511934383330"
|
||||||
|
Found DFU: [0483:5720] ver=9999, devnum=96, cfg=1,\
|
||||||
|
intf=0, alt=0, name="spl", serial="002700333338511934383330"
|
||||||
|
@ -53,6 +53,54 @@ static int dfu_find_alt_num(const char *s)
|
|||||||
return ++i;
|
return ++i;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* treat dfu_alt_info with several interface information
|
||||||
|
* to allow DFU on several device with one command,
|
||||||
|
* the string format is
|
||||||
|
* interface devstring'='alternate list (';' separated)
|
||||||
|
* and each interface separated by '&'
|
||||||
|
*/
|
||||||
|
int dfu_config_interfaces(char *env)
|
||||||
|
{
|
||||||
|
struct dfu_entity *dfu;
|
||||||
|
char *s, *i, *d, *a, *part;
|
||||||
|
int ret = -EINVAL;
|
||||||
|
int n = 1;
|
||||||
|
|
||||||
|
s = env;
|
||||||
|
for (; *s; s++) {
|
||||||
|
if (*s == ';')
|
||||||
|
n++;
|
||||||
|
if (*s == '&')
|
||||||
|
n++;
|
||||||
|
}
|
||||||
|
ret = dfu_alt_init(n, &dfu);
|
||||||
|
if (ret)
|
||||||
|
return ret;
|
||||||
|
|
||||||
|
s = env;
|
||||||
|
while (s) {
|
||||||
|
ret = -EINVAL;
|
||||||
|
i = strsep(&s, " ");
|
||||||
|
if (!i)
|
||||||
|
break;
|
||||||
|
d = strsep(&s, "=");
|
||||||
|
if (!d)
|
||||||
|
break;
|
||||||
|
a = strsep(&s, "&");
|
||||||
|
if (!a)
|
||||||
|
a = s;
|
||||||
|
do {
|
||||||
|
part = strsep(&a, ";");
|
||||||
|
ret = dfu_alt_add(dfu, i, d, part);
|
||||||
|
if (ret)
|
||||||
|
return ret;
|
||||||
|
} while (a);
|
||||||
|
}
|
||||||
|
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
|
||||||
int dfu_init_env_entities(char *interface, char *devstr)
|
int dfu_init_env_entities(char *interface, char *devstr)
|
||||||
{
|
{
|
||||||
const char *str_env;
|
const char *str_env;
|
||||||
@ -69,7 +117,11 @@ int dfu_init_env_entities(char *interface, char *devstr)
|
|||||||
}
|
}
|
||||||
|
|
||||||
env_bkp = strdup(str_env);
|
env_bkp = strdup(str_env);
|
||||||
|
if (!interface && !devstr)
|
||||||
|
ret = dfu_config_interfaces(env_bkp);
|
||||||
|
else
|
||||||
ret = dfu_config_entities(env_bkp, interface, devstr);
|
ret = dfu_config_entities(env_bkp, interface, devstr);
|
||||||
|
|
||||||
if (ret) {
|
if (ret) {
|
||||||
pr_err("DFU entities configuration failed!\n");
|
pr_err("DFU entities configuration failed!\n");
|
||||||
pr_err("(partition table does not match dfu_alt_info?)\n");
|
pr_err("(partition table does not match dfu_alt_info?)\n");
|
||||||
@ -83,6 +135,7 @@ done:
|
|||||||
|
|
||||||
static unsigned char *dfu_buf;
|
static unsigned char *dfu_buf;
|
||||||
static unsigned long dfu_buf_size;
|
static unsigned long dfu_buf_size;
|
||||||
|
static enum dfu_device_type dfu_buf_device_type;
|
||||||
|
|
||||||
unsigned char *dfu_free_buf(void)
|
unsigned char *dfu_free_buf(void)
|
||||||
{
|
{
|
||||||
@ -100,6 +153,10 @@ unsigned char *dfu_get_buf(struct dfu_entity *dfu)
|
|||||||
{
|
{
|
||||||
char *s;
|
char *s;
|
||||||
|
|
||||||
|
/* manage several entity with several contraint */
|
||||||
|
if (dfu_buf && dfu->dev_type != dfu_buf_device_type)
|
||||||
|
dfu_free_buf();
|
||||||
|
|
||||||
if (dfu_buf != NULL)
|
if (dfu_buf != NULL)
|
||||||
return dfu_buf;
|
return dfu_buf;
|
||||||
|
|
||||||
@ -118,6 +175,7 @@ unsigned char *dfu_get_buf(struct dfu_entity *dfu)
|
|||||||
printf("%s: Could not memalign 0x%lx bytes\n",
|
printf("%s: Could not memalign 0x%lx bytes\n",
|
||||||
__func__, dfu_buf_size);
|
__func__, dfu_buf_size);
|
||||||
|
|
||||||
|
dfu_buf_device_type = dfu->dev_type;
|
||||||
return dfu_buf;
|
return dfu_buf;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user