[U-Boot] Pull request: u-boot-net

[Bin Meng]

+Simon,

Hi Tom,

On Mon, Jan 4, 2016 at 11:46 AM, Tom Rini <trini at konsulko.com> wrote:
> On Mon, Jan 04, 2016 at 10:35:39AM +0800, Bin Meng wrote:
>> +Dirk
>>
>> On Sun, Jan 3, 2016 at 1:09 AM, Tom Rini <trini at konsulko.com> wrote:
>> > On Tue, Dec 22, 2015 at 11:58:01AM -0600, Joe Hershberger wrote:
>> >
>> >> A few patches that came in during the merge window and appear harmless.
>> >>
>> >> These cause no additional build warnings or errors.
>> >>
>> >> Thanks,
>> >> -Joe
>> >>
>> >> The following changes since commit 4832e17787acb29734d895751bc7a594908aecc6:
>> >>
>> >>   Merge branch 'master' of git://www.denx.de/git/u-boot-microblaze
>> >> (2015-12-18 07:28:24 -0500)
>> >>
>> >> are available in the git repository at:
>> >>
>> >>
>> >>   git://git.denx.de/u-boot-net.git master
>> >>
>> >> for you to fetch changes up to 140bc33e05382545b762ef51d6fc31dd5b6ec82c:
>> >>
>> >>   net: e1000: Mark _disable_wr() and _write_status() as __maybe_unused
>> >> (2015-12-21 20:01:57 -0600)
>> >>
>> >> ----------------------------------------------------------------
>> >> Bin Meng (5):
>> >>       fdt: Deprecate "usbethaddr" usage in fdt_fixup_ethernet()
>> >>       fdt: Rewrite the logic in fdt_fixup_ethernet()
>> >>       net: e1000: Remove dead codes wrapped by #if 0
>> >>       net: e1000: Remove CONFIG_MVBC_1G
>> >>       net: e1000: Mark _disable_wr() and _write_status() as __maybe_unused
>> >>
>> >> Fabio Estevam (1):
>> >>       include: net: Simplify the usage of __always_inline
>> >>
>> >>  common/fdt_support.c    | 64 ++++++++++++++++++++--------------------
>> >>  drivers/net/e1000.c     | 77 +------------------------------------------------
>> >>  drivers/net/e1000.h     | 31 --------------------
>> >>  drivers/net/e1000_spi.c |  9 +++---
>> >>  include/net.h           |  9 +++---
>> >>  5 files changed, 42 insertions(+), 148 deletions(-)
>> >
>> > I think at this stage in the release we need to rework this and drop the
>> > fdt patches, or defer the whole thing until we can figure out some way
>> > to fit iocon in with the ELDK 5.6 toolchains.  Thanks!
>>
>> I cannot reproduce this with the kernel.org powerpc gcc 4.9 toolchain.
>> I am downloading ELDK 5.6 toolchain to see what's going on there.
>> IMHO, we should increase the iocon's U-Boot size to 512K. I see from
>> its iocon.h file its flash size is 64MB, which has plenty of spaces to
>> store a bigger U-Boot. Even if we try our best to rework this fdt
>> patch, there is still possibility to break iocon sometime in the
>> future by some other random fixes or new features. This to me is a
>> tedious process.
>
> I don't know why this is so hard to reproduce, I also see it with the
> debian/unstable gcc 5.x toolchain I do beleive.
>

Still downloading ELDK toolchains so cannot comment ELDK compiler
behavior :) One question, what's our recommended toolchains for
U-Boot? I have been using kernel.org toolchains as mentioned in the
buildman doc and some other ones mentioned in the moveconfig doc.

The simple fix is to change change iocon to a more larger size since
it has a 64MB flash. Dirk, can you please comment?

Regards,
Bin