Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <786ae93b-b89f-0587-b084-2f93554df479@gmail.com>
Date:   Mon, 12 Jun 2023 05:23:09 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.2
Subject: Re: [PATCH] arm64: dts: mt7986: increase bl2 partition on NAND of
 Bananapi R3
Content-Language: en-US, ca-ES, es-ES
To:     Daniel Golle <daniel@makrotopia.org>, devicetree@vger.kernel.org,
        linux-mediatek@lists.infradead.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Frank Wunderlich <frank-w@public-files.de>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>,
        Conor Dooley <conor+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Weijie Gao <weijie.gao@mediatek.com>
References: <ZH9UGF99RgzrHZ88@makrotopia.org>
From:   Matthias Brugger <matthias.bgg@gmail.com>
In-Reply-To: <ZH9UGF99RgzrHZ88@makrotopia.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 06/06/2023 17:43, Daniel Golle wrote:
> The bootrom burned into the MT7986 SoC will try multiple locations on
> the SPI-NAND flash to load bl2 in case the bl2 image located at the the
> previously attempted offset is corrupt.
> 
> Use 0x100000 instead of 0x80000 as partition size for bl2 on SPI-NAND,
> allowing for up to four redundant copies of bl2 (typically sized a
> bit less than 0x40000).
> 
> Fixes: 8e01fb15b8157 ("arm64: dts: mt7986: add Bananapi R3")
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>

Applied, thanks!

> ---
>   .../boot/dts/mediatek/mt7986a-bananapi-bpi-r3-nand.dtso     | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-nand.dtso b/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-nand.dtso
> index 15ee8c568f3c3..543c13385d6e3 100644
> --- a/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-nand.dtso
> +++ b/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-nand.dtso
> @@ -29,13 +29,13 @@ partitions {
>   
>   					partition@0 {
>   						label = "bl2";
> -						reg = <0x0 0x80000>;
> +						reg = <0x0 0x100000>;
>   						read-only;
>   					};
>   
> -					partition@80000 {
> +					partition@100000 {
>   						label = "reserved";
> -						reg = <0x80000 0x300000>;
> +						reg = <0x100000 0x280000>;
>   					};
>   
>   					partition@380000 {