warnings: use more 'const' for char *
authorØyvind Harboe <oyvind.harboe@zylin.com>
Wed, 29 Dec 2010 21:07:21 +0000 (22:07 +0100)
committerØyvind Harboe <oyvind.harboe@zylin.com>
Wed, 29 Dec 2010 21:07:39 +0000 (22:07 +0100)
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
25 files changed:
src/flash/mflash.h
src/flash/nand/core.h
src/flash/nand/driver.h
src/flash/nor/at91sam7.c
src/flash/nor/core.h
src/flash/nor/driver.h
src/flash/nor/lpc288x.c
src/flash/nor/pic32mx.c
src/flash/nor/stellaris.c
src/flash/nor/tms470.c
src/helper/bin2char.c
src/helper/command.c
src/helper/command.h
src/pld/pld.h
src/server/server.c
src/server/server.h
src/target/algorithm.h
src/target/arm9tdmi.c
src/target/arm_disassembler.c
src/target/armv7m.c
src/target/dsp563xx.c
src/target/dsp563xx.h
src/target/etm.h
src/target/register.h
src/target/target_type.h

index bb3cd05..0520c5f 100644 (file)
@@ -34,7 +34,7 @@ struct mflash_gpio_num
 
 struct mflash_gpio_drv
 {
-       char *name;
+       const char *name;
        int (*set_gpio_to_output) (struct mflash_gpio_num gpio);
        int (*set_gpio_output_val) (struct mflash_gpio_num gpio, uint8_t val);
 };
index a8e7466..709c37b 100644 (file)
@@ -59,7 +59,7 @@ struct nand_ecclayout {
 
 struct nand_device
 {
-       char *name;
+       const char *name;
        struct nand_flash_controller *controller;
        void *controller_priv;
        struct nand_manufacturer *manufacturer;
@@ -91,12 +91,12 @@ enum
 struct nand_manufacturer
 {
        int id;
-       char *name;
+       const char *name;
 };
 
 struct nand_info
 {
-       char *name;
+       const char *name;
        int id;
        int page_size;
        int chip_size;
index fe73e90..deb41b2 100644 (file)
@@ -35,7 +35,7 @@ struct nand_device;
 struct nand_flash_controller
 {
        /** Driver name that is used to select it from configuration files. */
-       char *name;
+       const char *name;
 
     const struct command_registration *commands;
 
index 3fabe47..ad6ee6c 100644 (file)
@@ -125,7 +125,7 @@ struct at91sam7_flash_bank
        uint16_t cidr_nvpsiz2;
        uint16_t cidr_eproc;
        uint16_t cidr_version;
-       char *target_name;
+       const char *target_name;
 
        /* flash auto-detection */
        uint8_t  flash_autodetection;
index ff14df0..8f7dce8 100644 (file)
@@ -74,7 +74,7 @@ struct flash_sector
  */
 struct flash_bank
 {
-       char *name;
+       const char *name;
 
        struct target *target; /**< Target to which this bank belongs. */
 
index cbf9c52..afd0528 100644 (file)
@@ -54,7 +54,7 @@ struct flash_driver
         * Gives a human-readable name of this flash driver,
         * This field is used to select and initialize the driver.
         */
-       char *name;
+       const char *name;
 
        /**
         * An array of driver-specific commands to register.  When called
index d8cecc0..589527f 100644 (file)
@@ -91,7 +91,7 @@ struct lpc288x_flash_bank
 
        /* chip id register */
        uint32_t cidr;
-       char * target_name;
+       const char * target_name;
        uint32_t cclk;
 
        uint32_t sector_size_break;
index 4b8d027..6a27702 100644 (file)
@@ -99,7 +99,7 @@ struct pic32mx_flash_bank
 
 static const struct pic32mx_devs_s {
        uint8_t devid;
-       char *name;
+       const char *name;
 } pic32mx_devs[] = {
        {0x38, "360F512L"},
        {0x34, "360F256L"},
index 12ea15b..851cab3 100644 (file)
@@ -100,7 +100,7 @@ struct stellaris_flash_bank
        uint32_t dc0;
        uint32_t dc1;
 
-       char * target_name;
+       const char * target_name;
 
        uint32_t sramsiz;
        uint32_t flshsz;
@@ -125,7 +125,7 @@ struct stellaris_flash_bank
 
 static struct {
        uint32_t partno;
-       char *partname;
+       const char *partname;
 }      StellarisParts[] =
 {
        {0x0001,"LM3S101"},
index 4ca6cd8..dd9ff5b 100644 (file)
@@ -38,7 +38,7 @@ struct tms470_flash_bank
        uint32_t technology_family;
        uint32_t rom_flash;
        uint32_t part_number;
-       char * part_name;
+       const char * part_name;
 
 };
 
@@ -129,7 +129,7 @@ static int tms470_read_part_info(struct flash_bank *bank)
        uint32_t technology_family;
        uint32_t rom_flash;
        uint32_t part_number;
-       char *part_name;
+       const char *part_name;
 
        /* we shall not rely on the caller in this test, this function allocates memory,
           thus and executing the code more than once may cause memory leak */
index 5bb4a6d..a214b63 100644 (file)
@@ -29,7 +29,7 @@ int main(int argc, char **argv)
 {
        int c;
        unsigned int n;
-       char *name;
+       const char *name;
 
        if (argc == 1) {
                fprintf(stderr, "bin2char <varname>\n");
index fd5e5f9..d5ac34b 100644 (file)
@@ -310,7 +310,7 @@ static void command_free(struct command *c)
        }
 
        if (c->name)
-               free(c->name);
+               free((void *)c->name);
        if (c->help)
                free((void*)c->help);
        if (c->usage)
index 0ee66b0..3d1cdec 100644 (file)
@@ -173,7 +173,7 @@ typedef __COMMAND_HANDLER((*command_handler_t));
 
 struct command
 {
-       char *name;
+       const char *name;
        const char *help;
        const char *usage;
        struct command *parent;
index d306246..3c7e4ff 100644 (file)
@@ -29,7 +29,7 @@ struct pld_device;
 
 struct pld_driver
 {
-       char *name;
+       const char *name;
        __PLD_DEVICE_COMMAND((*pld_device_command));
        const struct command_registration *commands;
        int (*load)(struct pld_device *pld_device, const char *filename);
index 70d9566..c70a522 100644 (file)
@@ -306,7 +306,7 @@ static int remove_services(void)
                struct service *next = c->next;
 
                if (c->name)
-                       free(c->name);
+                       free((void *)c->name);
 
                if (c->type == CONNECTION_PIPE)
                {
index 2afd712..34cf1cf 100644 (file)
@@ -57,7 +57,7 @@ typedef int (*connection_closed_handler_t)(struct connection *connection);
 
 struct service
 {
-       char *name;
+       const char *name;
        enum connection_type type;
        const char *port;
        unsigned short portnumber;
index abb8ecf..af1fd91 100644 (file)
@@ -37,7 +37,7 @@ struct mem_param
 
 struct reg_param
 {
-       char *reg_name;
+       const char *reg_name;
        uint32_t size;
        uint8_t *value;
        enum param_direction direction;
index fd50703..5c2fd9d 100644 (file)
@@ -60,7 +60,7 @@ enum arm9tdmi_vector_bit
 };
 
 static const struct arm9tdmi_vector {
-       char *name;
+       const char *name;
        uint32_t value;
 } arm9tdmi_vectors[] = {
        {"reset", ARM9TDMI_RESET_VECTOR},
index f02053f..10720f4 100644 (file)
@@ -2354,7 +2354,7 @@ static int evaluate_add_sp_pc_thumb(uint16_t opcode,
        uint8_t Rd = (opcode >> 8) & 0x7;
        uint8_t Rn;
        uint32_t SP = opcode & (1 << 11);
-       char *reg_name;
+       const char *reg_name;
 
        instruction->type = ARM_ADD;
 
index 03c9c7a..4fcde16 100644 (file)
@@ -82,7 +82,7 @@ struct reg armv7m_gdb_dummy_cpsr_reg =
  */
 static const struct {
        unsigned id;
-       char *name;
+       const char *name;
        unsigned bits;
 } armv7m_regs[] = {
        { ARMV7M_R0, "r0", 32 },
index 9563eea..10fd014 100644 (file)
@@ -173,7 +173,7 @@ static int dsp563xx_jtag_sendinstr(struct jtag_tap *tap, uint8_t * ir_in, uint8_
 static const struct
 {
        unsigned id;
-       char *name;
+       const char *name;
        unsigned bits;
        uint32_t r_cmd;
        uint32_t w_cmd;
index f8070fa..475e516 100644 (file)
@@ -53,7 +53,7 @@ struct dsp563xx_common
 struct dsp563xx_core_reg
 {
        uint32_t num;
-       char *name;
+       const char *name;
        uint32_t size;
        uint32_t r_cmd;
        uint32_t w_cmd;
index 8a482c1..4224de6 100644 (file)
@@ -139,7 +139,7 @@ struct etm_context;
 
 struct etm_capture_driver
 {
-       char *name;
+       const char *name;
        const struct command_registration *commands;
        int (*init)(struct etm_context *etm_ctx);
        trace_status_t (*status)(struct etm_context *etm_ctx);
index 0cd0ddf..4b45fd3 100644 (file)
@@ -27,7 +27,7 @@ struct target;
 
 struct reg
 {
-       char *name;
+       const char *name;
        void *value;
        bool dirty;
        bool valid;
@@ -38,7 +38,7 @@ struct reg
 
 struct reg_cache
 {
-       char *name;
+       const char *name;
        struct reg_cache *next;
        struct reg *reg_list;
        unsigned num_regs;
index 7a8ac9c..bfa7f93 100644 (file)
@@ -42,7 +42,7 @@ struct target_type
         * Name of this type of target.  Do @b not access this
         * field directly, use target_type_name() instead.
         */
-       char *name;
+       const char *name;
 
        /* poll current target status */
        int (*poll)(struct target *target);