[U-Boot] [PATCH v2 4/4] ARM: fix CONFIG_SPL_MAX_SIZE semantics
Albert ARIBAUD
albert.u.boot at aribaud.net
Thu Apr 11 01:09:54 CEST 2013
On Thu, 11 Apr 2013 00:50:01 +0200, Albert ARIBAUD
<albert.u.boot at aribaud.net> wrote:
> What we could do, though, is subdivide testing based on the existence or
> non-existence of CONFIG_SPL_BSS_START_ADDR:
>
> - if CONFIG_SPL_BSS_START_ADDR exists, then we assume SPL image and
> BSS are disjoint and we test each one against its max size, as this
> patch series does;
>
> - if CONFIG_SPL_BSS_START_ADDR does not exist, then we assume SPL image
> and BSS are contiguous and we test the whole of SPL against the sum
> of CONFIG_SPL_MAX_SIZE and CONFIG_SPL_BSS_MAX_SIZE.
>
> I guess this will be considered useless complication -- after all,
> once you have artificially partitioned your SPL space into image+BSS --
> and you know from the build command how much should be allotted to each
> of them -- the worst that can happen is that a later build fails with
> an explicit error message forcing you to look at current image and BSS
> size and adjust one or both of the max values accordingly.
P.S. In any case, the proposal above will go in, if at all, as a
separate patch; the current patch series is going in right now as it is.
Amicalement,
--
Albert.
More information about the U-Boot
mailing list