[U-Boot] [PATCH] net: Return -1 when ctrl+c is pressed

[Michal Simek]

On 08/20/2015 06:51 PM, Joe Hershberger wrote:
> Hi Michal,
> 
> On Thu, Aug 20, 2015 at 7:15 AM, Michal Simek <monstr at monstr.eu> wrote:
>> On 08/13/2015 09:46 AM, Michal Simek wrote:
>>> Current behavior is that if CTRL+C is pressed command returns 0 that was
>>> successful which is not correct behavior.
>>> The easiest test case is "tftpboot 80000 uImage && echo yes"
>>> and press CTRL+C. Then the second command is called which is incorrect.
>>>
>>> Error log:
>>> zynq-uboot> tftpb 80000 uImage && echo yes
>>> Gem.e000b000:7 is connected to Gem.e000b000.  Reconnecting to
>>> Gem.e000b000
>>> Gem.e000b000 Waiting for PHY auto negotiation to complete....... done
>>> Using Gem.e000b000 device
>>> TFTP from server 192.168.0.102; our IP address is 192.168.0.101
>>> Filename 'uImage'.
>>> Load address: 0x80000
>>> Loading: ################
>>> Abort
>>> yes
>>> zynq-uboot>
>>>
>>> This patch adds -1 return value when CTRL+C is pressed.
>>>
>>> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
>>> ---
>>>
>>> Not sure if -1 is the right error code but I have parsed the code and it
>>> looks like that -1 is standard behavior for CTRL+C. But I am happy to
>>> change this behavior.
>>>
>>> ---
>>>  net/net.c | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/net/net.c b/net/net.c
>>> index 67e0ad297cf5..738a0db8c2ba 100644
>>> --- a/net/net.c
>>> +++ b/net/net.c
>>> @@ -565,6 +565,7 @@ restart:
>>>                       /* include a debug print as well incase the debug
>>>                          messages are directed to stderr */
>>>                       debug_cond(DEBUG_INT_STATE, "--- net_loop Abort!\n");
>>> +                     ret = -1;
> 
> Please replace this with -EINTR.
> 
>>>                       goto done;
>>>               }
>>>
>>>
>>
>> Joe, Tom: Any update on this one?
> 
> Slipped through... thanks for the reminder.

v2 sent.

Thanks,
Michal