Hi Biju,
On Sun, Sep 1, 2024 at 8:34 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote:
> Update partition table for spi-nor flash, so that we can flash bootloaders
> in Linux by executing the below commands:
> flash_erase /dev/mtd0 0 0
> flash_erase /dev/mtd1 0 0
> mtd_debug write /dev/mtd0 0 ${BL2_FILE_SIZE} ${BL2_IMAGE}
> mtd_debug write /dev/mtd1 512 ${FIP_FILE_SIZE} ${FIP_IMAGE}
>
> Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> ---
> v4:
> * New patch.
Thanks for your patch!
> --- a/arch/arm64/boot/dts/renesas/rzg2l-smarc-som.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg2l-smarc-som.dtsi
> @@ -341,11 +341,18 @@ partitions {
> #address-cells = <1>;
> #size-cells = <1>;
>
> - boot@0 {
> - reg = <0x00000000 0x2000000>;
> - read-only;
> + partition@0 {
> + label = "bl2";
> + reg = <0x00000000 0x0001c000>;
> };
> - user@2000000 {
> +
> + partition@1d000 { /* fip is at offset 0x200 */
Is this 4 KiB gap between the two partitions intentional?
If yes, I think it deserves a comment.
Same comment for rzg2lc-smarc-som.dtsi.
The rest LGTM.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
