From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bin Meng Date: Mon, 4 Jan 2016 10:35:39 +0800 Subject: [U-Boot] Pull request: u-boot-net In-Reply-To: <20160102170929.GW4093@bill-the-cat> References: <20160102170929.GW4093@bill-the-cat> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de +Dirk On Sun, Jan 3, 2016 at 1:09 AM, Tom Rini 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. Regards, Bin