From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 13/13] mips: jz47xx: Add Creator CI20 platform
Date: Mon, 12 Jun 2017 15:16:43 -0400 [thread overview]
Message-ID: <20170612191643.GC10782@bill-the-cat> (raw)
In-Reply-To: <bce855c1-c773-3ba2-120d-e6636e170760@suse.de>
On Sun, Jun 11, 2017 at 11:45:48PM +0200, Andreas F채rber wrote:
> Am 12.02.2017 um 15:03 schrieb Marek Vasut:
> > On 02/12/2017 02:24 PM, Andreas F채rber wrote:
> >> Am 12.02.2017 um 13:53 schrieb Marek Vasut:
> >>> On 02/12/2017 01:24 PM, Andreas F채rber wrote:
> >>>> Am 12.02.2017 um 12:55 schrieb Marek Vasut:
> >>>>> On 02/12/2017 12:52 PM, Andreas F채rber wrote:
> >>>>>> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y leads to this error:
> >>>>>>
> >>>>>> LD spl/u-boot-spl
> >>>>>> mipsel-suse-linux-ld.bfd: u-boot-spl section `.data' will not fit in
> >>>>>> region `.sram'
> >>>>>> mipsel-suse-linux-ld.bfd: region `.sram' overflowed by 288 bytes
> >>>>>> ../scripts/Makefile.spl:304: recipe for target 'spl/u-boot-spl' failed
> >>>>>> make[2]: *** [spl/u-boot-spl] Error 1
> >>>>>> /home/andreas/OBS/u-boot/Makefile:1342: recipe for target
> >>>>>> 'spl/u-boot-spl' failed
> >>>>>> make[1]: *** [spl/u-boot-spl] Error 2
> >>>>>> make[1]: Leaving directory '/home/andreas/OBS/u-boot/ci20'
> >>>>>> Makefile:150: recipe for target 'sub-make' failed
> >>>>>> make: *** [sub-make] Error 2
> >>>>>>
> >>>>>> I've reviewed all SPL Kconfig options and found three seemingly unneeded
> >>>>>> options defaulting to y, but I did not find a way to get this number
> >>>>>> down even a single byte with my GCC 6.3.1, and the recommended 4.8.1 was
> >>>>>> even worse (~748).
> >>>> I'm looking into a gcc7 package next, but that'll take a bit.
> >>>
> >>> I don't think that's gonna help with U-Boot's bloat. [...]
> >>
> >> Yeah, slightly down with GCC 7.0.1, but not much:
> >
> > Right, it's the new bloat ...
> >
> >> LD spl/u-boot-spl
> >> mipsel-suse-linux-ld.bfd: u-boot-spl section `.data' will not fit in
> >> region `.sram'
> >> mipsel-suse-linux-ld.bfd: region `.sram' overflowed by 264 bytes
> >> ../scripts/Makefile.spl:304: recipe for target 'spl/u-boot-spl' failed
> >> make[2]: *** [spl/u-boot-spl] Error 1
> >> /home/andreas/OBS/u-boot/Makefile:1342: recipe for target
> >> 'spl/u-boot-spl' failed
> >> make[1]: *** [spl/u-boot-spl] Error 2
> >> make[1]: Leaving directory '/home/andreas/OBS/u-boot/ci20'
> >> Makefile:150: recipe for target 'sub-make' failed
> >> make: *** [sub-make] Error 2
>
> Update: I've rebased the patchset to latest master
> (8cb3ce64f936f5dedbcfc1935c5caf31bb682474 / "Merge
> git://git.denx.de/u-boot-dm") - BMIPS caused trivial conflicts, and DM
> API changes needed to be accounted for.
>
> https://github.com/afaerber/u-boot/commits/ci20
>
> By further disabling CONFIG_SPL_RAW_IMAGE_SUPPORT in the ci20 defconfig
> I am now at 344 bytes overflow (gcc 7.1.1):
>
> LD spl/u-boot-spl
> mipsel-suse-linux-ld.bfd: u-boot-spl section `.data' will not fit in
> region `.sram'
> mipsel-suse-linux-ld.bfd: region `.sram' overflowed by 344 bytes
> ../scripts/Makefile.spl:333: recipe for target 'spl/u-boot-spl' failed
> make[2]: *** [spl/u-boot-spl] Error 1
> /home/andreas/OBS/u-boot/Makefile:1382: recipe for target
> 'spl/u-boot-spl' failed
> make[1]: *** [spl/u-boot-spl] Error 2
>
> By disabling SPL GPIO support and hardcoding a board revision instead of
> detecting it via GPIOs, I can get it down to 80 bytes.
>
> By aggressively adding #ifndef CONFIG_SPL_BUILD in ci20.c I'm down to 64
> bytes, but not sure if some of that is actually needed for SPL...
>
> Tom, do you see a chance of merging any of the drivers without the whole
> board building, so that we can reduce the rebasing work and get a common
> base for optimizing?
Yes, I would be agreeable to that, especially if say they can be enabled
also on qemu-mips so we have build but not run time testing and thus
aren't adding dead code.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170612/1aa80c0e/attachment.sig>
next prev parent reply other threads:[~2017-06-12 19:16 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-01 1:06 [U-Boot] [PATCH 01/13] serial: 16550: Add getfcr accessor Marek Vasut
2016-12-01 1:06 ` [U-Boot] [PATCH 02/13] serial: 16550: Add port type as driver data Marek Vasut
2016-12-19 21:19 ` [U-Boot] [U-Boot, " Tom Rini
2016-12-01 1:06 ` [U-Boot] [PATCH 03/13] serial: 16550: Add Ingenic JZ4780 support Marek Vasut
2016-12-03 4:26 ` Simon Glass
2016-12-19 21:20 ` [U-Boot] [U-Boot, " Tom Rini
2016-12-01 1:06 ` [U-Boot] [PATCH 04/13] mmc: Fix warning if debug() is not used Marek Vasut
2016-12-01 4:17 ` Jaehoon Chung
2016-12-01 5:14 ` Jaehoon Chung
2016-12-01 1:06 ` [U-Boot] [PATCH 05/13] mmc: Tinification of the mmc code Marek Vasut
2016-12-01 5:07 ` Jaehoon Chung
2016-12-01 1:06 ` [U-Boot] [PATCH 06/13] mmc: Add JZ47xx SD/MMC controller driver Marek Vasut
2016-12-01 5:48 ` Jaehoon Chung
2017-02-12 14:20 ` Andreas Färber
2017-02-12 14:29 ` Marek Vasut
2017-02-12 14:56 ` Andreas Färber
2016-12-01 1:06 ` [U-Boot] [PATCH 07/13] SPL: mmc: Make spl_mmc_load_image available Marek Vasut
2016-12-01 5:07 ` Jaehoon Chung
2016-12-01 1:06 ` [U-Boot] [PATCH 08/13] gpio: Add JZ47xx GPIO driver Marek Vasut
2016-12-01 1:06 ` [U-Boot] [PATCH 09/13] misc: Add JZ47xx efuse driver Marek Vasut
2016-12-01 1:06 ` [U-Boot] [PATCH 10/13] mips: Add SPL header Marek Vasut
2016-12-01 1:06 ` [U-Boot] [PATCH 11/13] mips: jz47xx: Add JZ4780 SoC support Marek Vasut
2016-12-01 1:06 ` [U-Boot] [PATCH 12/13] mips: jz47xx: Add minimal JZ MMC node Marek Vasut
2016-12-01 1:06 ` [U-Boot] [PATCH 13/13] mips: jz47xx: Add Creator CI20 platform Marek Vasut
2017-02-12 11:52 ` Andreas Färber
2017-02-12 11:55 ` Marek Vasut
2017-02-12 12:24 ` Andreas Färber
2017-02-12 12:53 ` Marek Vasut
2017-02-12 13:24 ` Andreas Färber
2017-02-12 14:03 ` Marek Vasut
2017-06-11 21:45 ` Andreas Färber
2017-06-12 19:16 ` Tom Rini [this message]
2017-02-12 14:14 ` Andreas Färber
2017-02-14 22:58 ` Tom Rini
2017-02-14 23:03 ` Marek Vasut
2017-02-14 23:11 ` Tom Rini
2017-02-15 20:46 ` Marek Vasut
2017-02-15 21:50 ` Tom Rini
2017-02-15 22:08 ` Marek Vasut
2016-12-03 4:26 ` [U-Boot] [PATCH 01/13] serial: 16550: Add getfcr accessor Simon Glass
2016-12-19 21:18 ` [U-Boot] [U-Boot,01/13] " Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170612191643.GC10782@bill-the-cat \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox