Re: [Buildroot] [PATCH 1/3] configs/pine64: bump U-Boot version to 2024.07

From: Mattia Narducci <mattianarducci1@gmail.com>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Jagan Teki <jagan@amarulasolutions.com>, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/3] configs/pine64: bump U-Boot version to 2024.07
Date: Wed, 04 Sep 2024 21:54:37 +0200	[thread overview]
Message-ID: <4949491.31r3eYUQgx@localhost> (raw)
In-Reply-To: <20240903215308.27bb1d2c@windsurf>

On Tue, 03 Sep 2024 21:53:08 CEST Thomas Petazzoni wrote:
> On Sun, 01 Sep 2024 18:42:58 +0200
> Mattia Narducci <mattianarducci1@gmail.com> wrote:
> >  	partition u-boot {
> >  		in-partition-table = "no"
> > -		image = "u-boot.itb"
> > -		offset = 40K
> > -		size = 1M # 1MB - 40KB
> > +		image = "u-boot-sunxi-with-spl.bin"
> > +		offset = 8K
> > +		size = 1016K # 1MB - 8KB
> 
> Here this "size" field is a bit silly. We don't really care that the
> size is 1016KB. What we care is that the offset is 8 KB, and that the
> next partition is aligned on a good boundary.
> 
> So could you test something like:
> 
>   https://patchwork.ozlabs.org/project/buildroot/patch/tencent_92439641515553684D208B3331676FB36C05@qq.com/
> 
> instead?

Sure! I have seen this approach used for other boards and thought it
was fine, but the offset is definitely clearer.

-- 
Mattia

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot