Aw: Re: Re: [RFC] arm64: dts: use size of reserved partition for bl2

[Frank Wunderlich]

> Gesendet: Montag, 03. April 2023 um 16:04 Uhr
> Von: "AngeloGioacchino Del Regno" <angelogioacchino.delregno at collabora.com>
> An: "Frank Wunderlich" <frank-w at public-files.de>
> Cc: "Frank Wunderlich" <linux at fw-web.de>, linux-mediatek at lists.infradead.org, "Rob Herring" <robh+dt at kernel.org>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt at linaro.org>, "Matthias Brugger" <matthias.bgg at gmail.com>, devicetree at vger.kernel.org, linux-kernel at vger.kernel.org, linux-arm-kernel at lists.infradead.org
> Betreff: Re: Aw: Re: [RFC] arm64: dts: use size of reserved partition for bl2
>
> Il 03/04/23 15:56, Frank Wunderlich ha scritto:
> >> Gesendet: Montag, 03. April 2023 um 14:43 Uhr
> >> Von: "AngeloGioacchino Del Regno" <angelogioacchino.delregno at collabora.com>
> >> Il 03/04/23 12:58, Frank Wunderlich ha scritto:
> >>> From: Frank Wunderlich <frank-w at public-files.de>
> >>>
> >>> To store uncompressed bl2 more space is required than partition is
> >>> actually defined.
> >>>
> >>> Signed-off-by: Frank Wunderlich <frank-w at public-files.de>
> >>
> >> If this doesn't get changed anymore, I'm fine with it... but a question arises:
> >> did you send patches to add your BPI-r3 board(s) to upstream u-boot?
> >
> > currently i use the rfb dts for r3 in uboot: arch/arm/dts/mt7986a-sd-rfb.dts
> >
> > this file in upstream uboot has no partitions defined
> >
> > https://source.denx.de/u-boot/u-boot/-/blob/master/arch/arm/dts/mt7986a-sd-rfb.dts#L144
> >
> > but i added them there too and i wrote content by offsets to main device (not to partitions).
> >
> > https://github.com/frank-w/u-boot/blob/2023-04-bpi/arch/arm/dts/mt7986a-sd-rfb.dts#L154
> >
> > so yes basicly it needs to be send there too, maybe as additional dts.
> >
>
> I strongly encourage you to send this change to u-boot as well as changing the
> kernel devicetree... and when you do, you can add a reference to the list and/or
> commit @ u-boot on the commit that you'll send here for the kernel... but there's
> another question...

imho uboot needs an own dts for the board (copy from rfb) for this because of another change
(tick_dly) and i have no rfb for testing.

> ...what happens if you run the kernel on a device with an older partition layout?
> Does anything break?
> I don't think that anything does, since this is read-only anyway, but just to be
> sure... :-)

i have no information about the second partition from the old layout...it was only named reserved.

as openwrt also uses 0x40000 it should be save to use this value here.

if the bl2 does not fit into the bl2-partition (cut off), board simply does not boot,
thats why i want to increase it now. my current bl2 is 197K for nor and i had this problem.

> Besides, you have to fix the commit description, but you knew that already.

sorry missed the mt7986: part in title, or did you mean another part?

i put the additional information below --- to not be visible when patch will be applied.

> P.S.: When you post a RFC, giving it the right commit title helps sometimes! :-P

yes, missed the mt7986-part here, just wanted to hear comments first if thisis the right way