This is an automated email from Gerrit.

"Tomas Vanek <van...@fbl.cz>" just uploaded a new patch set to Gerrit, which 
you can find at https://review.openocd.org/c/openocd/+/8443

-- gerrit

commit 3fcfddb93aaa27c45ab6760b758d21d0876f3038
Author: Luke Wren <l...@raspberrypi.com>
Date:   Fri Jun 28 16:41:48 2024 +0100

    flash/nor/rp2040: Fix up ROM table lookup for RP2350 A2
    
    which has 16-bit well-known pointers.
    
    Change-Id: Ia0838a0b062f73a9c5751abb48f1b4d55100bd1d
    Signed-off-by: Tomas Vanek <van...@fbl.cz>

diff --git a/src/flash/nor/rp2040.c b/src/flash/nor/rp2040.c
index c20fe88ae8..dc7595f303 100644
--- a/src/flash/nor/rp2040.c
+++ b/src/flash/nor/rp2040.c
@@ -331,8 +331,8 @@ static int rp2xxx_lookup_rom_symbol(struct target *target, 
uint16_t tag, uint16_
                /* Distinguish old-style RP2350 ROM table (A0, and earlier A1 
builds)
                   based on position of table -- a high address means it is 
shared with
                   RISC-V, i.e. new-style. */
-               uint32_t table_ptr;
-               err = target_read_u32(target, BOOTROM_MAGIC_ADDR + 4, 
&table_ptr);
+               uint16_t table_ptr;
+               err = target_read_u16(target, BOOTROM_MAGIC_ADDR + 4, 
&table_ptr);
                if (err != ERROR_OK)
                        return err;
                if (table_ptr < 0x7c00)

-- 

Reply via email to