[PATCH] ARM: dts: renesas: genmai: Fix partition size for QSPI NOR Flash

[PATCH] ARM: dts: renesas: genmai: Fix partition size for QSPI NOR Flash

[Wolfram Sang]

Second partition was too large, looks like two digits got mixed up.
Fixes:

mtd: partition "user1" extends beyond the end of device "18000000.flash" -- size truncated to 0x4000000

Fixes: 30e0a8cf886c ("ARM: dts: renesas: genmai: Add FLASH nodes")
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

As you can see, I got my Genmai board working again \o/

 arch/arm/boot/dts/renesas/r7s72100-genmai.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/renesas/r7s72100-genmai.dts b/arch/arm/boot/dts/renesas/r7s72100-genmai.dts
index 63f9f679964b..c835d61fc20d 100644
--- a/arch/arm/boot/dts/renesas/r7s72100-genmai.dts
+++ b/arch/arm/boot/dts/renesas/r7s72100-genmai.dts
@@ -54,7 +54,7 @@ partition@0 {
 
 			partition@4000000 {
 				label = "user1";
-				reg = <0x04000000 0x40000000>;
+				reg = <0x04000000 0x04000000>;
 			};
 		};
 	};
-- 
2.45.2

Re: [PATCH] ARM: dts: renesas: genmai: Fix partition size for QSPI NOR Flash

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 333 bytes --]

On Sat, Sep 14, 2024 at 08:28:44PM +0200, Wolfram Sang wrote:
> Second partition was too large, looks like two digits got mixed up.
> Fixes:

Hmm, maybe this "Fixes:" is ambiguous... perhaps we should leave it out.

> 
> mtd: partition "user1" extends beyond the end of device "18000000.flash" -- size truncated to 0x4000000

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] ARM: dts: renesas: genmai: Fix partition size for QSPI NOR Flash

[Geert Uytterhoeven]

On Sat, Sep 14, 2024 at 9:03 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> On Sat, Sep 14, 2024 at 08:28:44PM +0200, Wolfram Sang wrote:
> > Second partition was too large, looks like two digits got mixed up.
> > Fixes:
>
> Hmm, maybe this "Fixes:" is ambiguous... perhaps we should leave it out.

I like Fixes: ;-)

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v6.13.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

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