On Thu, Apr 27, 2017 at 05:45:20PM +0530, Ravi Babu wrote:
> The SPL-DFU feature enable to load and
> execute u-boot over usb from PC using
> dfu-util.
> Hence dfu-reset should not be issued
> when dfu-util -R switch is issued.
>
> Signed-off-by: Ravi Babu <ravib...@ti.com>
> ---
> common/dfu.c | 2 +-
> common/spl/Kconfig | 4 ++++
> drivers/dfu/dfu.c | 4 ++++
> 3 files changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/common/dfu.c b/common/dfu.c
> index 0e9f5f5..546a1ab 100644
> --- a/common/dfu.c
> +++ b/common/dfu.c
> @@ -88,7 +88,7 @@ exit:
> board_usb_cleanup(usbctrl_index, USB_INIT_DEVICE);
>
> if (dfu_reset)
> - run_command("reset", 0);
> + do_reset(NULL, 0, 0, NULL);
>
> g_dnl_clear_detach();
So this hunk drops out the need for cli stuff.
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index 1231351..f51ae2c 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -6,6 +6,9 @@ config SUPPORT_SPL
> config SUPPORT_TPL
> bool
>
> +config SPL_DFU_NO_RESET
> + bool
> +
> config SPL
> bool
> depends on SUPPORT_SPL
> @@ -646,6 +649,7 @@ config SPL_USBETH_SUPPORT
> config SPL_DFU_SUPPORT
> bool "Support DFU (Device Firmware Upgarde)"
> select SPL_HASH_SUPPORT
> + select SPL_DFU_NO_RESET
> depends on SPL_RAM_SUPPORT
> help
> This feature enables the DFU (Device Firmware Upgarde) in SPL with
> diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c
> index 8dacc1a..ceb33e3 100644
> --- a/drivers/dfu/dfu.c
> +++ b/drivers/dfu/dfu.c
> @@ -35,7 +35,11 @@ static struct hash_algo *dfu_hash_algo;
> */
> __weak bool dfu_usb_get_reset(void)
> {
> +#ifdef CONFIG_SPL_DFU_NO_RESET
> + return false;
> +#else
> return true;
> +#endif
> }
>
> static int dfu_find_alt_num(const char *s)
So do we still need the above, in order to save space? How much are we
saving here even, now? Thanks!
--
Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
