[U-Boot] Proposed fastboot changes
I've made some changes to the fastboot handlers to meet my needs. If there is interest, I can prepare patches and submit them upstream. The changes are: BUGFIX: Currently the 'fastboot format [partition]' command (run from PC) formats the partition smaller than expected. Change getvar_partition_size() to return partition size in # of bytes instead of the current behavior of # of blocks. The 'fastboot format [partition]' command expect u-boot to return the size in number of bytes in order to format the partition correctly. Make the a fastboot oem command handler with a dynamic lookup like the base fastboot command handler currently is. This would allow easily adding new oem commands in the baseline software (such as below). It would also allow for board files to extend the oem handler to handle OEM specific cases. I would use a weakly defined default oem command handler (empty function) which a board file can optionally override to handle board specific commands. Currently the 'fastboot oem format' command (gpt write mmc # $partitions) is supported. I'd like to add a 'fastboot oem verify' (gpt verify mmc # $partitions) to help the flashing PC detect when a board needs to be re-formatted. -Doug -- This email and any attachments are for the exclusive use of the intended recipient(s) and may contain confidential and/or privileged information. Inadvertent disclosure of this message does not constitute a waiver of any privilege, right or remedy. If you are not the intended recipient, please do not directly or indirectly use, disclose or distribute this message, and please contact the sender and delete this email, any attachments and all copies. Climate and its affiliates may use, read or archive email communications (including attachments) through its computer network, as permitted by applicable law. Climate and its affiliates (or an external service provider) may also scan emails and attachments on its computer network to ensure systems operate efficiently and to minimize security risks. Thank you. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Proposed fastboot changes
On Mon, Jan 14, 2019 at 8:49 PM Douglas Zobel wrote: > > I've made some changes to the fastboot handlers to meet my needs. If there > is interest, I can prepare patches and submit them upstream. The changes are: > > BUGFIX: Currently the 'fastboot format [partition]' command (run from PC) > formats the partition smaller than expected. Change getvar_partition_size() > to return partition size in # of bytes instead of the current behavior of # > of blocks. The 'fastboot format [partition]' command expect u-boot to return > the size in number of bytes in order to format the partition correctly. > > Make the a fastboot oem command handler with a dynamic lookup like the base > fastboot command handler currently is. This would allow easily adding new > oem commands in the baseline software (such as below). It would also allow > for board files to extend the oem handler to handle OEM specific cases. I > would use a weakly defined default oem command handler (empty function) which > a board file can optionally override to handle board specific commands. > > Currently the 'fastboot oem format' command (gpt write mmc # $partitions) is > supported. I'd like to add a 'fastboot oem verify' (gpt verify mmc # > $partitions) to help the flashing PC detect when a board needs to be > re-formatted. > I'd no need for it, but when I did all of the changes to add UDP support, my intention was that you'd implement oem command using linker lists, at that point you can plug in whatever you need. -- Alex Kiernan ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Proposed fastboot changes
On Tue, Jan 15, 2019 at 4:12 AM Alex Kiernan wrote: > On Mon, Jan 14, 2019 at 8:49 PM Douglas Zobel > wrote: > > > > I've made some changes to the fastboot handlers to meet my needs. If > there is interest, I can prepare patches and submit them upstream. The > changes are: > > > > BUGFIX: Currently the 'fastboot format [partition]' command (run from > PC) formats the partition smaller than expected. Change > getvar_partition_size() to return partition size in # of bytes instead of > the current behavior of # of blocks. The 'fastboot format [partition]' > command expect u-boot to return the size in number of bytes in order to > format the partition correctly. > > > > Make the a fastboot oem command handler with a dynamic lookup like the > base fastboot command handler currently is. This would allow easily adding > new oem commands in the baseline software (such as below). It would also > allow for board files to extend the oem handler to handle OEM specific > cases. I would use a weakly defined default oem command handler (empty > function) which a board file can optionally override to handle board > specific commands. > > > > Currently the 'fastboot oem format' command (gpt write mmc # > $partitions) is supported. I'd like to add a 'fastboot oem verify' (gpt > verify mmc # $partitions) to help the flashing PC detect when a board needs > to be re-formatted. > > > > I'd no need for it, but when I did all of the changes to add UDP > support, my intention was that you'd implement oem command using > linker lists, at that point you can plug in whatever you need. > > I don't understand what you mean by linker lists. Can you give an example? -- This email and any attachments are for the exclusive use of the intended recipient(s) and may contain confidential and/or privileged information. Inadvertent disclosure of this message does not constitute a waiver of any privilege, right or remedy. If you are not the intended recipient, please do not directly or indirectly use, disclose or distribute this message, and please contact the sender and delete this email, any attachments and all copies. Climate and its affiliates may use, read or archive email communications (including attachments) through its computer network, as permitted by applicable law. Climate and its affiliates (or an external service provider) may also scan emails and attachments on its computer network to ensure systems operate efficiently and to minimize security risks. Thank you. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Proposed fastboot changes
On Tue, Jan 15, 2019 at 11:04 PM Douglas Zobel wrote: > > On Tue, Jan 15, 2019 at 4:12 AM Alex Kiernan wrote: >> >> On Mon, Jan 14, 2019 at 8:49 PM Douglas Zobel >> wrote: >> > >> > I've made some changes to the fastboot handlers to meet my needs. If >> > there is interest, I can prepare patches and submit them upstream. The >> > changes are: >> > >> > BUGFIX: Currently the 'fastboot format [partition]' command (run from PC) >> > formats the partition smaller than expected. Change >> > getvar_partition_size() to return partition size in # of bytes instead of >> > the current behavior of # of blocks. The 'fastboot format [partition]' >> > command expect u-boot to return the size in number of bytes in order to >> > format the partition correctly. >> > >> > Make the a fastboot oem command handler with a dynamic lookup like the >> > base fastboot command handler currently is. This would allow easily >> > adding new oem commands in the baseline software (such as below). It >> > would also allow for board files to extend the oem handler to handle OEM >> > specific cases. I would use a weakly defined default oem command handler >> > (empty function) which a board file can optionally override to handle >> > board specific commands. >> > >> > Currently the 'fastboot oem format' command (gpt write mmc # $partitions) >> > is supported. I'd like to add a 'fastboot oem verify' (gpt verify mmc # >> > $partitions) to help the flashing PC detect when a board needs to be >> > re-formatted. >> > >> >> I'd no need for it, but when I did all of the changes to add UDP >> support, my intention was that you'd implement oem command using >> linker lists, at that point you can plug in whatever you need. >> > > I don't understand what you mean by linker lists. Can you give an example? > The most obvious example is the command handling in U-Boot - include/command.h and the macro U_BOOT_CMD_MKENT. The definitions for linker lists themselves are in include/linker_lists.h. -- Alex Kiernan ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot