Message ID | 20150826035426.GA30213@home.buserror.net |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote: > The following changes since commit c851a2458fbc12495f4f786d4eabb612850a5143: > > Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400) > > are available in the git repository at: > > git://git.denx.de/u-boot-nand-flash.git > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543: > > mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -0500) > Applied to u-boot/master, thanks! But please note that I now see: +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too big Which I'm hoping can be tuned to re-fit size constraints again easily enough.
On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote: > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote: > > > The following changes since commit > > c851a2458fbc12495f4f786d4eabb612850a5143: > > > > Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400) > > > > are available in the git repository at: > > > > git://git.denx.de/u-boot-nand-flash.git > > > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543: > > > > mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 - > > 0500) > > > > Applied to u-boot/master, thanks! > > But please note that I now see: > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too > big > > Which I'm hoping can be tuned to re-fit size constraints again easily > enough. They built OK for me, so they must be close enough that the toolchain makes a difference. -Scott
On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote: > On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote: > > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote: > > > > > The following changes since commit > > > c851a2458fbc12495f4f786d4eabb612850a5143: > > > > > > Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400) > > > > > > are available in the git repository at: > > > > > > git://git.denx.de/u-boot-nand-flash.git > > > > > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543: > > > > > > mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 - > > > 0500) > > > > > > > Applied to u-boot/master, thanks! > > > > But please note that I now see: > > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too > > big > > > > Which I'm hoping can be tuned to re-fit size constraints again easily > > enough. > > They built OK for me, so they must be close enough that the toolchain makes a > difference. Indeed. I'm using arm-none-eabi from Debian on that test box (since it will also build the armv7m targets).
2015-08-27 3:06 GMT+09:00 Tom Rini <trini@konsulko.com>: > On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote: >> On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote: >> > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote: >> > >> > > The following changes since commit >> > > c851a2458fbc12495f4f786d4eabb612850a5143: >> > > >> > > Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400) >> > > >> > > are available in the git repository at: >> > > >> > > git://git.denx.de/u-boot-nand-flash.git >> > > >> > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543: >> > > >> > > mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 - >> > > 0500) >> > > >> > >> > Applied to u-boot/master, thanks! >> > >> > But please note that I now see: >> > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too >> > big >> > These are my boards. I will do something with them.
Hi Tom, Scott, 2015-08-27 3:06 GMT+09:00 Tom Rini <trini@konsulko.com>: > On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote: >> On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote: >> > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote: >> > >> > > The following changes since commit >> > > c851a2458fbc12495f4f786d4eabb612850a5143: >> > > >> > > Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400) >> > > >> > > are available in the git repository at: >> > > >> > > git://git.denx.de/u-boot-nand-flash.git >> > > >> > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543: >> > > >> > > mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 - >> > > 0500) >> > > >> > >> > Applied to u-boot/master, thanks! >> > >> > But please note that I now see: >> > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too >> > big >> > >> > Which I'm hoping can be tuned to re-fit size constraints again easily >> > enough. >> >> They built OK for me, so they must be close enough that the toolchain makes a >> difference. > > Indeed. I'm using arm-none-eabi from Debian on that test box (since it > will also build the armv7m targets). I've sent a patch to fix the root cause of this problem. http://patchwork.ozlabs.org/patch/511211/ Only my boards (ph1_ld4, ph1_sld8, ph1_pro4) define CONFIG_SPL_NAND_DENALI, and I tested the patch on them. So, this patch should be safe. Is it OK to apply it quickly?
On Thu, Aug 27, 2015 at 06:57:13PM +0900, Masahiro Yamada wrote: > Hi Tom, Scott, > > > 2015-08-27 3:06 GMT+09:00 Tom Rini <trini@konsulko.com>: > > On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote: > >> On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote: > >> > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote: > >> > > >> > > The following changes since commit > >> > > c851a2458fbc12495f4f786d4eabb612850a5143: > >> > > > >> > > Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400) > >> > > > >> > > are available in the git repository at: > >> > > > >> > > git://git.denx.de/u-boot-nand-flash.git > >> > > > >> > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543: > >> > > > >> > > mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 - > >> > > 0500) > >> > > > >> > > >> > Applied to u-boot/master, thanks! > >> > > >> > But please note that I now see: > >> > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too > >> > big > >> > > >> > Which I'm hoping can be tuned to re-fit size constraints again easily > >> > enough. > >> > >> They built OK for me, so they must be close enough that the toolchain makes a > >> difference. > > > > Indeed. I'm using arm-none-eabi from Debian on that test box (since it > > will also build the armv7m targets). > > > I've sent a patch to fix the root cause of this problem. > http://patchwork.ozlabs.org/patch/511211/ > > Only my boards (ph1_ld4, ph1_sld8, ph1_pro4) define CONFIG_SPL_NAND_DENALI, > and I tested the patch on them. > > So, this patch should be safe. > > > Is it OK to apply it quickly? Yes it is, thanks!