On Wed, Feb 08, 2017 at 02:46:22PM -0600, Andrew F. Davis wrote:
On 02/08/2017 02:40 PM, Tom Rini wrote:
On Wed, Feb 08, 2017 at 12:42:33PM -0600, Andrew F. Davis wrote:
On 02/08/2017 07:07 AM, Lokesh Vutla wrote:
On Wednesday 08 February 2017 06:26 PM, Tom Rini wrote:
On Wed, Feb 08, 2017 at 06:03:31PM +0530, Lokesh Vutla wrote:
- Enable SPL_DM on all DRA7 and AM57xx based platforms.
- SPL uses a generic dts on all platforms with same defconfig.
- After this series SPL size is increased by ~7KB.
That's not a bad size growth, esp here where we can afford it. Did you
Yeah, but I am worried about this size on am335x.
I am also, I'm working to save about 30 bytes right now to include another boot mode into HS SPL, +7Kb will simply not work. It looks like you also get some of your saving from enabling simple_malloc, this is already enabled for AM335x-HS, so the size increase will likely be greater on the platform that needs it the most :(
I know I said it only the other day but switching from gcc-4.x/5.x to 6.x saves a bunch of space, even on am335x hs. I guess my request is that you pick up a gcc-6.x (Debian/unstable, Fedora 25 or I assume Linaro) compiler for ARM and see converting am335x HS to SPL_DT is anywhere close to fitting in the constraint.
I'm already using Linaro's gcc-6.x, I don't believe SPL fits anymore using the old compilers as it stands today.
You're enforcing the max size constraint today via the appropriate config option, right?