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 <tr...@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
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
