Re: [U-Boot] [PATCH v6 2/2] NET: Add net_busy_flag

2013-07-18 Thread Stephen Warren
On 07/18/2013 08:15 AM, Jim Lin wrote:
 This flag is to have console aware that NET transfer is running or not.

I actually wonder if the shell interpreter or command-invocation code
(to cover automatic bootcmd?) shouldn't set this flag before invoking
every command, and clear it afterward? Or perhaps, a flag should be
added to commands that would benefit from this flag being set?

But this patch is probably a reasonable way to implement it in the first
place, so we don't add complex infra-structure if it turns out that
nothing else is going to require this feature.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v6 2/2] NET: Add net_busy_flag

2013-07-18 Thread Marek Vasut
Dear Stephen Warren,

 On 07/18/2013 08:15 AM, Jim Lin wrote:
  This flag is to have console aware that NET transfer is running or not.
 
 I actually wonder if the shell interpreter or command-invocation code
 (to cover automatic bootcmd?) shouldn't set this flag before invoking
 every command, and clear it afterward? Or perhaps, a flag should be
 added to commands that would benefit from this flag being set?

Not a bad idea, but then this might be also needed for USB if the transfer is 
long etc etc
 
 But this patch is probably a reasonable way to implement it in the first
 place, so we don't add complex infra-structure if it turns out that
 nothing else is going to require this feature.

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot