On Wed, Mar 1, 2017 at 5:29 PM, Tom Rini <tr...@konsulko.com> wrote:
> On Tue, Feb 21, 2017 at 09:51:31PM +0200, Sam Protsenko wrote:
>
>> The current size of U-Boot is already ~840 KiB.
>>
>> This patch is to increase bootloader partition size up to 1 MiB
>> and thereby fix flashing error.
>>
>> Also fix U-Boot environment address on eMMC ("reserved" partition), as
>> it's being shifted by above change.
>>
>> Signed-off-by: Sam Protsenko <semen.protse...@linaro.org>
>
> I know I'm late chiming in here, but I want to now, all the same.
> First, we need to explain _why_ things are getting so huge. The answer
> is that we're including so many (complete, from Linux) device trees. So
> we should probably update the comments around this area to note why the
> bootloader partition is so large, and then pick a good, large, and
> aligned with the underlying erase block size partition.
>
Tom,
Your comment is absolutely correct, I was just in a little bit hurry
when sending this patch. Please don't merge it yet, I'm gonna get back
to it after Linaro Connect. Things you mentioned will be added to
commit message, and we are probably going to change that partition
size one more time. So let's hold on on this change for now.
Thanks.
> --
> Tom
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot
