flash: constify driver data structures 93/2293/3
authorSpencer Oliver <spen@spen-soft.co.uk>
Thu, 11 Sep 2014 21:08:34 +0000 (22:08 +0100)
committerSpencer Oliver <spen@spen-soft.co.uk>
Mon, 6 Oct 2014 11:57:32 +0000 (11:57 +0000)
Change-Id: Ia5c3de48119f036e1d7a41be62a672a6fb37e59b
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/2293
Tested-by: jenkins
src/flash/nor/fm3.c
src/flash/nor/non_cfi.c
src/flash/nor/nrf51.c

index d4bd5b9c120cf8ccb9ca0f1ef27daee49541f4a0..d3d143f60b35dc24d2af126ea439b9cc4086b58f 100644 (file)
@@ -240,7 +240,7 @@ static int fm3_erase(struct flash_bank *bank, int first, int last)
        /* R0 keeps Flash Sequence address 1     (u32FlashSeq1)    */
        /* R1 keeps Flash Sequence address 2     (u32FlashSeq2)    */
        /* R2 keeps Flash Offset address         (ofs)                     */
        /* R0 keeps Flash Sequence address 1     (u32FlashSeq1)    */
        /* R1 keeps Flash Sequence address 2     (u32FlashSeq2)    */
        /* R2 keeps Flash Offset address         (ofs)                     */
-       const uint8_t fm3_flash_erase_sector_code[] = {
+       static const uint8_t fm3_flash_erase_sector_code[] = {
                                                /*    *(uint16_t*)u32FlashSeq1 = 0xAA; */
                0xAA, 0x24,             /*        MOVS  R4, #0xAA              */
                0x04, 0x80,             /*        STRH  R4, [R0, #0]           */
                                                /*    *(uint16_t*)u32FlashSeq1 = 0xAA; */
                0xAA, 0x24,             /*        MOVS  R4, #0xAA              */
                0x04, 0x80,             /*        STRH  R4, [R0, #0]           */
@@ -849,7 +849,7 @@ static int fm3_chip_erase(struct flash_bank *bank)
        /* RAMCODE used for fm3 Flash chip erase:                                  */
        /* R0 keeps Flash Sequence address 1     (u32FlashSeq1)    */
        /* R1 keeps Flash Sequence address 2     (u32FlashSeq2)    */
        /* RAMCODE used for fm3 Flash chip erase:                                  */
        /* R0 keeps Flash Sequence address 1     (u32FlashSeq1)    */
        /* R1 keeps Flash Sequence address 2     (u32FlashSeq2)    */
-       const uint8_t fm3_flash_erase_chip_code[] = {
+       static const uint8_t fm3_flash_erase_chip_code[] = {
                                                /*    *(uint16_t*)u32FlashSeq1 = 0xAA; */
                0xAA, 0x22,             /*        MOVS  R2, #0xAA              */
                0x02, 0x80,             /*        STRH  R2, [R0, #0]           */
                                                /*    *(uint16_t*)u32FlashSeq1 = 0xAA; */
                0xAA, 0x22,             /*        MOVS  R2, #0xAA              */
                0x02, 0x80,             /*        STRH  R2, [R0, #0]           */
index 9735bdbe930318724b74a78858a9999af3ba9036..72f11e55382a6f73b72016659f6e206c333fc53a 100644 (file)
@@ -33,7 +33,7 @@
 #define ERASE_REGION(num, size) (((size/256) << 16) | (num-1))
 
 /* non-CFI compatible flashes */
 #define ERASE_REGION(num, size) (((size/256) << 16) | (num-1))
 
 /* non-CFI compatible flashes */
-static struct non_cfi non_cfi_flashes[] = {
+static const struct non_cfi non_cfi_flashes[] = {
        {
                .mfr = CFI_MFR_SST,
                .id = 0xd4,
        {
                .mfr = CFI_MFR_SST,
                .id = 0xd4,
@@ -472,7 +472,7 @@ void cfi_fixup_non_cfi(struct flash_bank *bank)
 {
        unsigned int mask;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
 {
        unsigned int mask;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       struct non_cfi *non_cfi = non_cfi_flashes;
+       const struct non_cfi *non_cfi = non_cfi_flashes;
 
        if (cfi_info->x16_as_x8)
                mask = 0xFF;
 
        if (cfi_info->x16_as_x8)
                mask = 0xFF;
index adb4004d982fa1f5df05e09d2305f0c518242472..711d6fbaadee713529410b5ef5820380c3d1d401 100644 (file)
@@ -920,8 +920,9 @@ static int nrf51_info(struct flash_bank *bank, char *buf, int buf_size)
        if (res != ERROR_OK)
                return res;
 
        if (res != ERROR_OK)
                return res;
 
-       struct {
-               uint32_t address, value;
+       static struct {
+               const uint32_t address;
+               uint32_t value;
        } ficr[] = {
                { .address = NRF51_FICR_CODEPAGESIZE    },
                { .address = NRF51_FICR_CODESIZE        },
        } ficr[] = {
                { .address = NRF51_FICR_CODEPAGESIZE    },
                { .address = NRF51_FICR_CODESIZE        },

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)