[U-Boot] [PATCH 1/9] net: Revert "tftp: adjust settings to be suitable for 100Mbit ethernet"

Joe Hershberger joe.hershberger at gmail.com
Tue Aug 25 18:05:02 CEST 2015


Hi Bin,

On Tue, Aug 25, 2015 at 4:26 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
> Hi Joe,
>
> On Tue, Aug 25, 2015 at 3:22 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
>> Testing either pch_gbe or e1000 driver via tftp command on Intel
>> Crown Bay board, shows the following failure.
>>
>>     TFTP error: 'Unsupported option(s) requested' (8)
>>
>> It turns out commit 620776d causes this. Revert this commit for now.
>
> Please check http://lists.denx.de/pipermail/u-boot/2015-August/225187.html
> on why this commit should be reverted.
>
> Let me know if you have different thoughts (eg: I can respin a v2 to
> explicitly mention in the commit message that commit 620776d is a spec
> violation to RTC 2349 thus we need revert it)

I agree we should revert it for this release. Please send a v2 that
describes as much detail from the RFC as needed and references the RFC
as well. We will revert it and figure out if there is a compliant way
we can improve performance in a high-load situation.

By pushing this to next release we will get much more testing and not
risk losing compatibility in this release.

-Joe


More information about the U-Boot mailing list