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

[Tom Rini]

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.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160103/a3875c20/attachment.sig>