This is an automated email from Gerrit.

Andrey Yurovsky ([email protected]) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/1874

-- gerrit

commit 217a5d3550b484e26e0ee015e66a9e252bd5ee3e
Author: Andrey Yurovsky <[email protected]>
Date:   Wed Jan 15 08:10:35 2014 -0800

    flash: samd: declare const data
    
    Make the chip information lookup tables const (thanks to Spencer Oliver for
    pointing this out).  Also fix spacing on one macro.  No functional changes.
    
    Change-Id: I053c4951c2626b8aa4541a79673b500394ef08e8
    Signed-off-by: Andrey Yurovsky <[email protected]>

diff --git a/src/flash/nor/at91samd.c b/src/flash/nor/at91samd.c
index 5de5a9a..8b0f14d 100644
--- a/src/flash/nor/at91samd.c
+++ b/src/flash/nor/at91samd.c
@@ -60,7 +60,7 @@
 #define SAMD_PROCESSOR_M0      0x01
 #define SAMD_FAMILY_D          0x00
 #define SAMD_SERIES_20         0x00
-#define SAMD_SERIES_21      0x01
+#define SAMD_SERIES_21         0x01
 
 struct samd_part {
        uint8_t id;
@@ -70,7 +70,7 @@ struct samd_part {
 };
 
 /* Known SAMD20 parts. See Table 12-8 in 42129F–SAM–10/2013 */
-static struct samd_part samd20_parts[] = {
+static const struct samd_part samd20_parts[] = {
        { 0x0, "SAMD20J18A", 256, 32 },
        { 0x1, "SAMD20J17A", 128, 16 },
        { 0x2, "SAMD20J16A", 64, 8 },
@@ -88,7 +88,7 @@ static struct samd_part samd20_parts[] = {
 };
 
 /* Known SAMD21 parts. */
-static struct samd_part samd21_parts[] = {
+static const struct samd_part samd21_parts[] = {
        { 0x0, "SAMD21J18A", 256, 32 },
        { 0x1, "SAMD21J17A", 128, 16 },
        { 0x2, "SAMD21J16A", 64, 8 },
@@ -113,12 +113,12 @@ struct samd_family {
        uint8_t processor;
        uint8_t family;
        uint8_t series;
-       struct samd_part *parts;
+       const struct samd_part *parts;
        size_t num_parts;
 };
 
 /* Known SAMD families */
-static struct samd_family samd_families[] = {
+static const struct samd_family samd_families[] = {
        { SAMD_PROCESSOR_M0, SAMD_FAMILY_D, SAMD_SERIES_20,
                samd20_parts, ARRAY_SIZE(samd20_parts) },
        { SAMD_PROCESSOR_M0, SAMD_FAMILY_D, SAMD_SERIES_21,
@@ -137,7 +137,7 @@ struct samd_info {
 
 static struct samd_info *samd_chips;
 
-static struct samd_part *samd_find_part(uint32_t id)
+static const struct samd_part *samd_find_part(uint32_t id)
 {
        uint8_t processor = (id >> 28);
        uint8_t family = (id >> 24) & 0x0F;
@@ -180,7 +180,7 @@ static int samd_probe(struct flash_bank *bank)
        uint32_t id, param;
        int res;
        struct samd_info *chip = (struct samd_info *)bank->driver_priv;
-       struct samd_part *part;
+       const struct samd_part *part;
 
        if (chip->probed)
                return ERROR_OK;

-- 

------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today. 
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to