From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Simek <michal.simek@xilinx.com>
Date: Wed, 16 Dec 2015 09:05:43 +0100
Subject: [U-Boot] [PATCH 2/5] net: gem: Fix return value from recv
In-Reply-To: <CANr=Z=br3P2xH-28DfAte7+=FvYK272r+=4FmYCk304h4CU+tA@mail.gmail.com>
References: <f0d134bad9d670fe2a5a005d8f2d58e4308a98a2.1449834212.git.michal.simek@xilinx.com>
 <6e7a29b9075e356511e564cabcfc6da5d3b9cc14.1449834212.git.michal.simek@xilinx.com>
 <CANr=Z=bOtMK=-0D4ykLPpWv3kFubwPgSfAzLnEUk7-aeSSGaBA@mail.gmail.com>
 <CANr=Z=br3P2xH-28DfAte7+=FvYK272r+=4FmYCk304h4CU+tA@mail.gmail.com>
Message-ID: <56711B57.5000400@xilinx.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Joe,

On 15.12.2015 20:51, Joe Hershberger wrote:
> Hi Michal,
> 
> On Tue, Dec 15, 2015 at 11:47 AM, Joe Hershberger
> <joe.hershberger@gmail.com> wrote:
>> On Fri, Dec 11, 2015 at 5:43 AM, Michal Simek <michal.simek@xilinx.com> wrote:
>>> recv function should return 0 instead of frame_len not to
>>> proceed the same packet again in core.
>>>
>>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>>
>> Acked-by: Joe Hershberger <joe.hershberger@ni.com>
> 
> Oops. NAK. Didn't realize this was the top-level recv function being
> passed to the eth layer.

I expect that origin ACK was valid and you realized like for other
driver that here should be return 0 because separation with free_pkt is
done in 3/5.

Thanks,
Michal