[U-Boot] [PATCH v2 2/3] common: dfu: ignore reset for spl-dfu

[B, Ravi]

Hi Tom

>> 
>> 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.

Yes.

>> 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!

I observed around 7K reduced. 

Regards
Ravi