]> git.sur5r.net Git - openocd/commitdiff
- code reformat (Thanks Pavel Chromy)
authorntfreak <ntfreak@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Tue, 26 Feb 2008 19:51:18 +0000 (19:51 +0000)
committerntfreak <ntfreak@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Tue, 26 Feb 2008 19:51:18 +0000 (19:51 +0000)
git-svn-id: svn://svn.berlios.de/openocd/trunk@354 b42882b7-edfa-0310-969c-e2dbd0fdcd60

src/flash/tms470.c
src/flash/tms470.h

index c3f3e8ccddd4534b5f23430cc790aba5aa1c2f08..491426ef3a0ee3342cbc1520b888cf7178570d28 100644 (file)
 #include <string.h>
 #include <unistd.h>
 
-int 
-tms470_register_commands( struct command_context_s *cmd_ctx );
-
-int 
-tms470_flash_bank_command( struct command_context_s *cmd_ctx, 
-                           char *cmd, 
-                           char **args, 
-                           int argc, 
-                           struct flash_bank_s *bank );
-
-int 
-tms470_erase( struct flash_bank_s *bank, 
-              int first, 
-              int last );
-
-int 
-tms470_protect( struct flash_bank_s *bank, 
-                int set, 
-                int first, 
-                int last );
-
-int 
-tms470_write( struct flash_bank_s *bank, 
-              u8 *buffer, 
-              u32 offset, 
-              u32 count );
-
-int 
-tms470_probe( struct flash_bank_s *bank );
-
-int 
-tms470_auto_probe( struct flash_bank_s *bank );
-
-int 
-tms470_erase_check( struct flash_bank_s *bank );
-
-int 
-tms470_protect_check( struct flash_bank_s *bank );
-
-int 
-tms470_info( struct flash_bank_s *bank, 
-             char *buf, 
-             int buf_size );
-
-flash_driver_t tms470_flash =
-  {
-    .name =               "tms470",
-    .register_commands =  tms470_register_commands,
-    .flash_bank_command = tms470_flash_bank_command,
-    .erase =              tms470_erase,
-    .protect =            tms470_protect,
-    .write =              tms470_write,
-    .probe =              tms470_probe,
-    .auto_probe =         tms470_auto_probe,
-    .erase_check =        tms470_erase_check,
-    .protect_check =      tms470_protect_check,
-    .info =               tms470_info
-  };
+int tms470_register_commands(struct command_context_s *cmd_ctx);
+int tms470_flash_bank_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct flash_bank_s *bank);
+int tms470_erase(struct flash_bank_s *bank, int first, int last);
+int tms470_protect(struct flash_bank_s *bank, int set, int first, int last);
+int tms470_write(struct flash_bank_s *bank, u8 * buffer, u32 offset, u32 count);
+int tms470_probe(struct flash_bank_s *bank);
+int tms470_auto_probe(struct flash_bank_s *bank);
+int tms470_erase_check(struct flash_bank_s *bank);
+int tms470_protect_check(struct flash_bank_s *bank);
+int tms470_info(struct flash_bank_s *bank, char *buf, int buf_size);
+
+flash_driver_t tms470_flash = {
+       .name = "tms470",
+       .register_commands = tms470_register_commands,
+       .flash_bank_command = tms470_flash_bank_command,
+       .erase = tms470_erase,
+       .protect = tms470_protect,
+       .write = tms470_write,
+       .probe = tms470_probe,
+       .auto_probe = tms470_auto_probe,
+       .erase_check = tms470_erase_check,
+       .protect_check = tms470_protect_check,
+       .info = tms470_info
+};
 
 /* ---------------------------------------------------------------------- 
                       Internal Support, Helpers
    ---------------------------------------------------------------------- */
 
-const flash_sector_t TMS470R1A256_SECTORS[] =
-  {
-    { 0x00000000, 0x00002000, -1, -1 },
-    { 0x00002000, 0x00002000, -1, -1 },
-    { 0x00004000, 0x00002000, -1, -1 },
-    { 0x00006000, 0x00002000, -1, -1 },
-    { 0x00008000, 0x00008000, -1, -1 },
-    { 0x00010000, 0x00008000, -1, -1 },
-    { 0x00018000, 0x00008000, -1, -1 },
-    { 0x00020000, 0x00008000, -1, -1 },
-    { 0x00028000, 0x00008000, -1, -1 },
-    { 0x00030000, 0x00008000, -1, -1 },
-    { 0x00038000, 0x00002000, -1, -1 },
-    { 0x0003A000, 0x00002000, -1, -1 },
-    { 0x0003C000, 0x00002000, -1, -1 },
-    { 0x0003E000, 0x00002000, -1, -1 },
-  };
+const flash_sector_t TMS470R1A256_SECTORS[] = {
+       {0x00000000, 0x00002000, -1, -1},
+       {0x00002000, 0x00002000, -1, -1},
+       {0x00004000, 0x00002000, -1, -1},
+       {0x00006000, 0x00002000, -1, -1},
+       {0x00008000, 0x00008000, -1, -1},
+       {0x00010000, 0x00008000, -1, -1},
+       {0x00018000, 0x00008000, -1, -1},
+       {0x00020000, 0x00008000, -1, -1},
+       {0x00028000, 0x00008000, -1, -1},
+       {0x00030000, 0x00008000, -1, -1},
+       {0x00038000, 0x00002000, -1, -1},
+       {0x0003A000, 0x00002000, -1, -1},
+       {0x0003C000, 0x00002000, -1, -1},
+       {0x0003E000, 0x00002000, -1, -1},
+};
 
 #define TMS470R1A256_NUM_SECTORS \
   (sizeof(TMS470R1A256_SECTORS)/sizeof(TMS470R1A256_SECTORS[0]))
 
-const flash_sector_t TMS470R1A288_BANK0_SECTORS[] =
-  {
-    { 0x00000000, 0x00002000, -1, -1 },
-    { 0x00002000, 0x00002000, -1, -1 },
-    { 0x00004000, 0x00002000, -1, -1 },
-    { 0x00006000, 0x00002000, -1, -1 },
-  };
+const flash_sector_t TMS470R1A288_BANK0_SECTORS[] = {
+       {0x00000000, 0x00002000, -1, -1},
+       {0x00002000, 0x00002000, -1, -1},
+       {0x00004000, 0x00002000, -1, -1},
+       {0x00006000, 0x00002000, -1, -1},
+};
 
 #define TMS470R1A288_BANK0_NUM_SECTORS \
   (sizeof(TMS470R1A288_BANK0_SECTORS)/sizeof(TMS470R1A288_BANK0_SECTORS[0]))
 
-const flash_sector_t TMS470R1A288_BANK1_SECTORS[] =
-  {
-    { 0x00040000, 0x00010000, -1, -1 },
-    { 0x00050000, 0x00010000, -1, -1 },
-    { 0x00060000, 0x00010000, -1, -1 },
-    { 0x00070000, 0x00010000, -1, -1 },
-  };
+const flash_sector_t TMS470R1A288_BANK1_SECTORS[] = {
+       {0x00040000, 0x00010000, -1, -1},
+       {0x00050000, 0x00010000, -1, -1},
+       {0x00060000, 0x00010000, -1, -1},
+       {0x00070000, 0x00010000, -1, -1},
+};
 
 #define TMS470R1A288_BANK1_NUM_SECTORS \
   (sizeof(TMS470R1A288_BANK1_SECTORS)/sizeof(TMS470R1A288_BANK1_SECTORS[0]))
 
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_read_part_info( struct flash_bank_s *bank )
+int tms470_read_part_info(struct flash_bank_s *bank)
 {
-  tms470_flash_bank_t *tms470_info = bank->driver_priv;
-  target_t *target = bank->target;
-  u32    device_ident_reg;
-  u32    silicon_version;
-  u32    technology_family;
-  u32    rom_flash;
-  u32    part_number;
-  char * part_name;
-
-  if (target->state != TARGET_HALTED)
-    {
-      WARNING( "Cannot communicate... target not halted." );
-      return ERROR_TARGET_NOT_HALTED;
-    }
-
-  /* read and parse the device identification register */
-  target_read_u32( target, 0xFFFFFFF0, &device_ident_reg );
-
-  INFO( "device_ident_reg=0x%08x", device_ident_reg );
-  
-  if ((device_ident_reg & 7) == 0)
-    {
-      WARNING( "Cannot identify target as a TMS470 family." );
-      return ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  silicon_version = (device_ident_reg >> 12) & 0xF;
-  technology_family = (device_ident_reg >> 11) & 1;
-  rom_flash = (device_ident_reg >> 10) & 1;
-  part_number = (device_ident_reg >> 3) & 0x7f;
-
-  /*
-   * If the part number is known, determine if the flash bank is valid
-   * based on the base address being within the known flash bank
-   * ranges.  Then fixup/complete the remaining fields of the flash
-   * bank structure.
-   */
-  switch( part_number )
-    {
-    case 0x0a:
-      part_name = "TMS470R1A256";
-
-      if (bank->base >= 0x00040000)
-        {
-          ERROR( "No %s flash bank contains base address 0x%08x.", 
-                 part_name, bank->base );
-          return ERROR_FLASH_OPERATION_FAILED;
-        }
-      tms470_info->ordinal = 0;
-      bank->base           = 0x00000000;
-      bank->size           = 256*1024;
-      bank->num_sectors    = TMS470R1A256_NUM_SECTORS;
-      bank->sectors = malloc( sizeof( TMS470R1A256_SECTORS ) );
-      if (!bank->sectors)
-        {
-          return ERROR_FLASH_OPERATION_FAILED;
-        }
-      (void) memcpy( bank->sectors,
-                     TMS470R1A256_SECTORS,
-                     sizeof( TMS470R1A256_SECTORS ) );
-      break;
-
-    case 0x2b:
-      part_name = "TMS470R1A288";
-
-      if ((bank->base >= 0x00000000) && (bank->base < 0x00008000))
-        {
-          tms470_info->ordinal = 0;
-          bank->base           = 0x00000000;
-          bank->size           = 32*1024;
-          bank->num_sectors    = TMS470R1A288_BANK0_NUM_SECTORS;
-          bank->sectors = malloc( sizeof( TMS470R1A288_BANK0_SECTORS ) );
-          if (!bank->sectors)
-            {
-              return ERROR_FLASH_OPERATION_FAILED;
-            }
-          (void) memcpy( bank->sectors,
-                         TMS470R1A288_BANK0_SECTORS,
-                         sizeof( TMS470R1A288_BANK0_SECTORS ) );
-        }
-      else if ((bank->base >= 0x00040000) && (bank->base < 0x00080000))
-        {
-          tms470_info->ordinal = 1;
-          bank->base           = 0x00040000;
-          bank->size           = 256*1024;
-          bank->num_sectors    = TMS470R1A288_BANK1_NUM_SECTORS;
-          bank->sectors = malloc( sizeof( TMS470R1A288_BANK1_SECTORS ) );
-          if (!bank->sectors)
-            {
-              return ERROR_FLASH_OPERATION_FAILED;
-            }
-          (void) memcpy( bank->sectors,
-                         TMS470R1A288_BANK1_SECTORS,
-                         sizeof( TMS470R1A288_BANK1_SECTORS ) );
-        }
-      else
-        {
-          ERROR( "No %s flash bank contains base address 0x%08x.", 
-                 part_name, bank->base );
-          return ERROR_FLASH_OPERATION_FAILED;
-        }
-      break;
-
-    default:
-      WARNING( "Could not identify part 0x%02x as a member of the TMS470 family.", 
-               part_number );
-      return ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  /* turn off memory selects */
-  target_write_u32( target, 0xFFFFFFE4, 0x00000000 );
-  target_write_u32( target, 0xFFFFFFE0, 0x00000000 );
-
-  bank->chip_width = 32;
-  bank->bus_width  = 32;
-  
-  INFO( "Identified %s, ver=%d, core=%s, nvmem=%s.",
-        part_name,
-        silicon_version,
-        (technology_family ? "1.8v" : "3.3v"),
-        (rom_flash ? "rom" : "flash") );
-
-  tms470_info->device_ident_reg  = device_ident_reg;
-  tms470_info->silicon_version   = silicon_version;
-  tms470_info->technology_family = technology_family;
-  tms470_info->rom_flash         = rom_flash;
-  tms470_info->part_number       = part_number;
-  tms470_info->part_name         = part_name;
-
-  /*
-   * Disable reset on address access violation.
-   */
-  target_write_u32( target, 0xFFFFFFE0, 0x00004007 );
-
-  return ERROR_OK;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+       target_t *target = bank->target;
+       u32 device_ident_reg;
+       u32 silicon_version;
+       u32 technology_family;
+       u32 rom_flash;
+       u32 part_number;
+       char *part_name;
+
+       if (target->state != TARGET_HALTED)
+       {
+               WARNING("Cannot communicate... target not halted.");
+               return ERROR_TARGET_NOT_HALTED;
+       }
+
+       /* read and parse the device identification register */
+       target_read_u32(target, 0xFFFFFFF0, &device_ident_reg);
+
+       INFO("device_ident_reg=0x%08x", device_ident_reg);
+
+       if ((device_ident_reg & 7) == 0)
+       {
+               WARNING("Cannot identify target as a TMS470 family.");
+               return ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       silicon_version = (device_ident_reg >> 12) & 0xF;
+       technology_family = (device_ident_reg >> 11) & 1;
+       rom_flash = (device_ident_reg >> 10) & 1;
+       part_number = (device_ident_reg >> 3) & 0x7f;
+
+       /*
+        * If the part number is known, determine if the flash bank is valid
+        * based on the base address being within the known flash bank
+        * ranges.  Then fixup/complete the remaining fields of the flash
+        * bank structure.
+        */
+       switch (part_number)
+       {
+       case 0x0a:
+               part_name = "TMS470R1A256";
+
+               if (bank->base >= 0x00040000)
+               {
+                       ERROR("No %s flash bank contains base address 0x%08x.", part_name, bank->base);
+                       return ERROR_FLASH_OPERATION_FAILED;
+               }
+               tms470_info->ordinal = 0;
+               bank->base = 0x00000000;
+               bank->size = 256 * 1024;
+               bank->num_sectors = TMS470R1A256_NUM_SECTORS;
+               bank->sectors = malloc(sizeof(TMS470R1A256_SECTORS));
+               if (!bank->sectors)
+               {
+                       return ERROR_FLASH_OPERATION_FAILED;
+               }
+               (void)memcpy(bank->sectors, TMS470R1A256_SECTORS, sizeof(TMS470R1A256_SECTORS));
+               break;
+
+       case 0x2b:
+               part_name = "TMS470R1A288";
+
+               if ((bank->base >= 0x00000000) && (bank->base < 0x00008000))
+               {
+                       tms470_info->ordinal = 0;
+                       bank->base = 0x00000000;
+                       bank->size = 32 * 1024;
+                       bank->num_sectors = TMS470R1A288_BANK0_NUM_SECTORS;
+                       bank->sectors = malloc(sizeof(TMS470R1A288_BANK0_SECTORS));
+                       if (!bank->sectors)
+                       {
+                               return ERROR_FLASH_OPERATION_FAILED;
+                       }
+                       (void)memcpy(bank->sectors, TMS470R1A288_BANK0_SECTORS, sizeof(TMS470R1A288_BANK0_SECTORS));
+               }
+               else if ((bank->base >= 0x00040000) && (bank->base < 0x00080000))
+               {
+                       tms470_info->ordinal = 1;
+                       bank->base = 0x00040000;
+                       bank->size = 256 * 1024;
+                       bank->num_sectors = TMS470R1A288_BANK1_NUM_SECTORS;
+                       bank->sectors = malloc(sizeof(TMS470R1A288_BANK1_SECTORS));
+                       if (!bank->sectors)
+                       {
+                               return ERROR_FLASH_OPERATION_FAILED;
+                       }
+                       (void)memcpy(bank->sectors, TMS470R1A288_BANK1_SECTORS, sizeof(TMS470R1A288_BANK1_SECTORS));
+               }
+               else
+               {
+                       ERROR("No %s flash bank contains base address 0x%08x.", part_name, bank->base);
+                       return ERROR_FLASH_OPERATION_FAILED;
+               }
+               break;
+
+       default:
+               WARNING("Could not identify part 0x%02x as a member of the TMS470 family.", part_number);
+               return ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       /* turn off memory selects */
+       target_write_u32(target, 0xFFFFFFE4, 0x00000000);
+       target_write_u32(target, 0xFFFFFFE0, 0x00000000);
+
+       bank->chip_width = 32;
+       bank->bus_width = 32;
+
+       INFO("Identified %s, ver=%d, core=%s, nvmem=%s.", part_name, silicon_version, (technology_family ? "1.8v" : "3.3v"), (rom_flash ? "rom" : "flash"));
+
+       tms470_info->device_ident_reg = device_ident_reg;
+       tms470_info->silicon_version = silicon_version;
+       tms470_info->technology_family = technology_family;
+       tms470_info->rom_flash = rom_flash;
+       tms470_info->part_number = part_number;
+       tms470_info->part_name = part_name;
+
+       /*
+        * Disable reset on address access violation.
+        */
+       target_write_u32(target, 0xFFFFFFE0, 0x00004007);
+
+       return ERROR_OK;
 }
 
 /* ---------------------------------------------------------------------- */
@@ -280,1003 +229,912 @@ tms470_read_part_info( struct flash_bank_s *bank )
 u32 keysSet = 0;
 u32 flashKeys[4];
 
-int 
-tms470_handle_flash_keyset_command( struct command_context_s * cmd_ctx, 
-                                    char *                     cmd, 
-                                    char **                    args, 
-                                    int                        argc )
+int tms470_handle_flash_keyset_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
 {
-  if (argc > 4)
-    {
-      command_print( cmd_ctx, "tms470 flash_keyset <key0> <key1> <key2> <key3>" );
-      return ERROR_INVALID_ARGUMENTS;
-    }
-  else if (argc == 4)
-    {
-      int i;
-
-      for( i=0; i<4; i++ )
-        {
-          int start = (0 == strncmp( args[i], "0x", 2 )) ? 2 : 0;
-          if (1 != sscanf( &args[i][start], "%x", &flashKeys[i] ))
-            {
-              command_print( cmd_ctx, "could not process flash key %s", args[i] );
-              ERROR( "could not process flash key %s", args[i] );
-              return ERROR_INVALID_ARGUMENTS;
-            }
-        }
-
-      keysSet = 1;
-    }
-  else if (argc != 0)
-    {
-      command_print( cmd_ctx, "tms470 flash_keyset <key0> <key1> <key2> <key3>" );
-      return ERROR_INVALID_ARGUMENTS;
-    }
-
-  if (keysSet)
-    {
-      command_print( cmd_ctx, "using flash keys 0x%08x, 0x%08x, 0x%08x, 0x%08x",
-                     flashKeys[0], flashKeys[1], flashKeys[2], flashKeys[3] );
-    }
-  else
-    {
-      command_print( cmd_ctx, "flash keys not set" );
-    }
-
-  return ERROR_OK;
+       if (argc > 4)
+       {
+               command_print(cmd_ctx, "tms470 flash_keyset <key0> <key1> <key2> <key3>");
+               return ERROR_INVALID_ARGUMENTS;
+       }
+       else if (argc == 4)
+       {
+               int i;
+
+               for (i = 0; i < 4; i++)
+               {
+                       int start = (0 == strncmp(args[i], "0x", 2)) ? 2 : 0;
+
+                       if (1 != sscanf(&args[i][start], "%x", &flashKeys[i]))
+                       {
+                               command_print(cmd_ctx, "could not process flash key %s", args[i]);
+                               ERROR("could not process flash key %s", args[i]);
+                               return ERROR_INVALID_ARGUMENTS;
+                       }
+               }
+
+               keysSet = 1;
+       }
+       else if (argc != 0)
+       {
+               command_print(cmd_ctx, "tms470 flash_keyset <key0> <key1> <key2> <key3>");
+               return ERROR_INVALID_ARGUMENTS;
+       }
+
+       if (keysSet)
+       {
+               command_print(cmd_ctx, "using flash keys 0x%08x, 0x%08x, 0x%08x, 0x%08x", flashKeys[0], flashKeys[1], flashKeys[2], flashKeys[3]);
+       }
+       else
+       {
+               command_print(cmd_ctx, "flash keys not set");
+       }
+
+       return ERROR_OK;
 }
 
-const u32 FLASH_KEYS_ALL_ONES[] = { 0xFFFFFFFF, 0xFFFFFFFF, 
-                                    0xFFFFFFFF, 0xFFFFFFFF, };
+const u32 FLASH_KEYS_ALL_ONES[] = { 0xFFFFFFFF, 0xFFFFFFFF,
+       0xFFFFFFFF, 0xFFFFFFFF,
+};
 
-const u32 FLASH_KEYS_ALL_ZEROS[] = { 0x00000000, 0x00000000, 
-                                     0x00000000, 0x00000000, };
+const u32 FLASH_KEYS_ALL_ZEROS[] = { 0x00000000, 0x00000000,
+       0x00000000, 0x00000000,
+};
 
 const u32 FLASH_KEYS_MIX1[] = { 0xf0fff0ff, 0xf0fff0ff,
-                                0xf0fff0ff, 0xf0fff0ff };
+       0xf0fff0ff, 0xf0fff0ff
+};
 
 const u32 FLASH_KEYS_MIX2[] = { 0x0000ffff, 0x0000ffff,
-                                0x0000ffff, 0x0000ffff };
+       0x0000ffff, 0x0000ffff
+};
 
 /* ---------------------------------------------------------------------- */
 
 int oscMHz = 12;
 
-int
-tms470_handle_osc_megahertz_command( struct command_context_s * cmd_ctx, 
-                                     char *                     cmd, 
-                                     char **                    args, 
-                                     int                        argc )
+int tms470_handle_osc_megahertz_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
 {
-  if (argc > 1)
-    {
-      command_print( cmd_ctx, "tms470 osc_megahertz <MHz>" );
-      return ERROR_INVALID_ARGUMENTS;
-    }
-  else if (argc == 1)
-    {
-      sscanf( args[0], "%d", &oscMHz );
-    }
-
-  if (oscMHz <= 0)
-    {
-      ERROR( "osc_megahertz must be positive and non-zero!" );
-      command_print( cmd_ctx, "osc_megahertz must be positive and non-zero!" );
-      oscMHz = 12;
-      return ERROR_INVALID_ARGUMENTS;
-    }
-
-  command_print( cmd_ctx, "osc_megahertz=%d", oscMHz );
-
-  return ERROR_OK;
+       if (argc > 1)
+       {
+               command_print(cmd_ctx, "tms470 osc_megahertz <MHz>");
+               return ERROR_INVALID_ARGUMENTS;
+       }
+       else if (argc == 1)
+       {
+               sscanf(args[0], "%d", &oscMHz);
+       }
+
+       if (oscMHz <= 0)
+       {
+               ERROR("osc_megahertz must be positive and non-zero!");
+               command_print(cmd_ctx, "osc_megahertz must be positive and non-zero!");
+               oscMHz = 12;
+               return ERROR_INVALID_ARGUMENTS;
+       }
+
+       command_print(cmd_ctx, "osc_megahertz=%d", oscMHz);
+
+       return ERROR_OK;
 }
 
 /* ---------------------------------------------------------------------- */
 
 int plldis = 0;
 
-int
-tms470_handle_plldis_command( struct command_context_s * cmd_ctx, 
-                              char *                     cmd, 
-                              char **                    args, 
-                              int                        argc )
+int tms470_handle_plldis_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
 {
-  if (argc > 1)
-    {
-      command_print( cmd_ctx, "tms470 plldis <0|1>" );
-      return ERROR_INVALID_ARGUMENTS;
-    }
-  else if (argc == 1)
-    {
-      sscanf( args[0], "%d", &plldis );
-      plldis = plldis ? 1 : 0;
-    }
-
-  command_print( cmd_ctx, "plldis=%d", plldis );
-
-  return ERROR_OK;
+       if (argc > 1)
+       {
+               command_print(cmd_ctx, "tms470 plldis <0|1>");
+               return ERROR_INVALID_ARGUMENTS;
+       }
+       else if (argc == 1)
+       {
+               sscanf(args[0], "%d", &plldis);
+               plldis = plldis ? 1 : 0;
+       }
+
+       command_print(cmd_ctx, "plldis=%d", plldis);
+
+       return ERROR_OK;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int
-tms470_check_flash_unlocked( target_t * target )
+int tms470_check_flash_unlocked(target_t * target)
 {
-  u32 fmbbusy;
+       u32 fmbbusy;
 
-  target_read_u32( target, 0xFFE89C08, &fmbbusy );
-  INFO( "tms470 fmbbusy=0x%08x -> %s", 
-         fmbbusy,
-         fmbbusy & 0x8000 ? "unlocked" : "LOCKED" );
-  return fmbbusy & 0x8000 ? ERROR_OK : ERROR_FLASH_OPERATION_FAILED;
+       target_read_u32(target, 0xFFE89C08, &fmbbusy);
+       INFO("tms470 fmbbusy=0x%08x -> %s", fmbbusy, fmbbusy & 0x8000 ? "unlocked" : "LOCKED");
+       return fmbbusy & 0x8000 ? ERROR_OK : ERROR_FLASH_OPERATION_FAILED;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int
-tms470_try_flash_keys( target_t *  target, 
-                       const u32 * key_set )
+int tms470_try_flash_keys(target_t * target, const u32 * key_set)
 {
-  u32 glbctrl, fmmstat;
-  int retval = ERROR_FLASH_OPERATION_FAILED;
-
-  /* set GLBCTRL.4  */
-  target_read_u32( target, 0xFFFFFFDC, &glbctrl );
-  target_write_u32( target, 0xFFFFFFDC, glbctrl | 0x10 );
-
-  /* only perform the key match when 3VSTAT is clear */
-  target_read_u32( target, 0xFFE8BC0C, &fmmstat );
-  if (!(fmmstat & 0x08))
-    {
-      unsigned i;
-      u32 fmbptr, fmbac2, orig_fmregopt;
-      
-      target_write_u32( target, 0xFFE8BC04, fmmstat & ~0x07 );
-
-      /* wait for pump ready */
-      do
-        {
-          target_read_u32( target, 0xFFE8A814, &fmbptr );
-          usleep( 1000 );
-        }
-      while( !(fmbptr & 0x0200) );
-
-      /* force max wait states */
-      target_read_u32( target, 0xFFE88004, &fmbac2 );
-      target_write_u32( target, 0xFFE88004, fmbac2 | 0xff );
-
-      /* save current access mode, force normal read mode */
-      target_read_u32( target, 0xFFE89C00, &orig_fmregopt );
-      target_write_u32( target, 0xFFE89C00, 0x00 );
-
-      for( i=0; i<4; i++ )
-        {
-          u32 tmp;
-
-          /* There is no point displaying the value of tmp, it is
-           * filtered by the chip.  The purpose of this read is to
-           * prime the unlocking logic rather than read out the value.
-           */
-          target_read_u32( target, 0x00001FF0+4*i, &tmp );
-
-          INFO( "tms470 writing fmpkey=0x%08x", key_set[i] );
-          target_write_u32( target, 0xFFE89C0C, key_set[i] );
-        }
-
-      if (ERROR_OK == tms470_check_flash_unlocked( target ))
-        {
-          /* 
-           * There seems to be a side-effect of reading the FMPKEY
-           * register in that it re-enables the protection.  So we
-           * re-enable it.
-           */
-          for( i=0; i<4; i++ )
-            {
-              u32 tmp;
-              target_read_u32( target, 0x00001FF0+4*i, &tmp );
-              target_write_u32( target, 0xFFE89C0C, key_set[i] );
-            }
-          retval = ERROR_OK;
-        }
-
-      /* restore settings */
-      target_write_u32( target, 0xFFE89C00, orig_fmregopt );
-      target_write_u32( target, 0xFFE88004, fmbac2 );
-    }
-
-  /* clear config bit */
-  target_write_u32( target, 0xFFFFFFDC, glbctrl );
-
-  return retval;
+       u32 glbctrl, fmmstat;
+       int retval = ERROR_FLASH_OPERATION_FAILED;
+
+       /* set GLBCTRL.4  */
+       target_read_u32(target, 0xFFFFFFDC, &glbctrl);
+       target_write_u32(target, 0xFFFFFFDC, glbctrl | 0x10);
+
+       /* only perform the key match when 3VSTAT is clear */
+       target_read_u32(target, 0xFFE8BC0C, &fmmstat);
+       if (!(fmmstat & 0x08))
+       {
+               unsigned i;
+               u32 fmbptr, fmbac2, orig_fmregopt;
+
+               target_write_u32(target, 0xFFE8BC04, fmmstat & ~0x07);
+
+               /* wait for pump ready */
+               do
+               {
+                       target_read_u32(target, 0xFFE8A814, &fmbptr);
+                       usleep(1000);
+               }
+               while (!(fmbptr & 0x0200));
+
+               /* force max wait states */
+               target_read_u32(target, 0xFFE88004, &fmbac2);
+               target_write_u32(target, 0xFFE88004, fmbac2 | 0xff);
+
+               /* save current access mode, force normal read mode */
+               target_read_u32(target, 0xFFE89C00, &orig_fmregopt);
+               target_write_u32(target, 0xFFE89C00, 0x00);
+
+               for (i = 0; i < 4; i++)
+               {
+                       u32 tmp;
+
+                       /* There is no point displaying the value of tmp, it is
+                        * filtered by the chip.  The purpose of this read is to
+                        * prime the unlocking logic rather than read out the value.
+                        */
+                       target_read_u32(target, 0x00001FF0 + 4 * i, &tmp);
+
+                       INFO("tms470 writing fmpkey=0x%08x", key_set[i]);
+                       target_write_u32(target, 0xFFE89C0C, key_set[i]);
+               }
+
+               if (ERROR_OK == tms470_check_flash_unlocked(target))
+               {
+                       /* 
+                        * There seems to be a side-effect of reading the FMPKEY
+                        * register in that it re-enables the protection.  So we
+                        * re-enable it.
+                        */
+                       for (i = 0; i < 4; i++)
+                       {
+                               u32 tmp;
+
+                               target_read_u32(target, 0x00001FF0 + 4 * i, &tmp);
+                               target_write_u32(target, 0xFFE89C0C, key_set[i]);
+                       }
+                       retval = ERROR_OK;
+               }
+
+               /* restore settings */
+               target_write_u32(target, 0xFFE89C00, orig_fmregopt);
+               target_write_u32(target, 0xFFE88004, fmbac2);
+       }
+
+       /* clear config bit */
+       target_write_u32(target, 0xFFFFFFDC, glbctrl);
+
+       return retval;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int
-tms470_unlock_flash( struct flash_bank_s * bank )
+int tms470_unlock_flash(struct flash_bank_s *bank)
 {
-  target_t *            target =      bank->target;
-  const u32 *           p_key_sets[5];
-  unsigned              i, key_set_count;
-
-  if (keysSet)
-    {
-      p_key_sets[0] = flashKeys;
-      p_key_sets[1] = FLASH_KEYS_ALL_ONES;
-      p_key_sets[2] = FLASH_KEYS_ALL_ZEROS;
-      p_key_sets[3] = FLASH_KEYS_MIX1;
-      p_key_sets[4] = FLASH_KEYS_MIX2;
-    }
-  else
-    {
-      key_set_count = 4;
-      p_key_sets[0] = FLASH_KEYS_ALL_ONES;
-      p_key_sets[1] = FLASH_KEYS_ALL_ZEROS;
-      p_key_sets[2] = FLASH_KEYS_MIX1;
-      p_key_sets[3] = FLASH_KEYS_MIX2;
-    }
-
-  for( i=0; i<key_set_count; i++ )
-    {
-      if (tms470_try_flash_keys( target, p_key_sets[i] ) == ERROR_OK)
-        {
-          INFO( "tms470 flash is unlocked" );
-          return ERROR_OK;
-        }
-    }
-
-  WARNING( "tms470 could not unlock flash memory protection level 2" );
-  return ERROR_FLASH_OPERATION_FAILED;
+       target_t *target = bank->target;
+       const u32 *p_key_sets[5];
+       unsigned i, key_set_count;
+
+       if (keysSet)
+       {
+               p_key_sets[0] = flashKeys;
+               p_key_sets[1] = FLASH_KEYS_ALL_ONES;
+               p_key_sets[2] = FLASH_KEYS_ALL_ZEROS;
+               p_key_sets[3] = FLASH_KEYS_MIX1;
+               p_key_sets[4] = FLASH_KEYS_MIX2;
+       }
+       else
+       {
+               key_set_count = 4;
+               p_key_sets[0] = FLASH_KEYS_ALL_ONES;
+               p_key_sets[1] = FLASH_KEYS_ALL_ZEROS;
+               p_key_sets[2] = FLASH_KEYS_MIX1;
+               p_key_sets[3] = FLASH_KEYS_MIX2;
+       }
+
+       for (i = 0; i < key_set_count; i++)
+       {
+               if (tms470_try_flash_keys(target, p_key_sets[i]) == ERROR_OK)
+               {
+                       INFO("tms470 flash is unlocked");
+                       return ERROR_OK;
+               }
+       }
+
+       WARNING("tms470 could not unlock flash memory protection level 2");
+       return ERROR_FLASH_OPERATION_FAILED;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int
-tms470_flash_initialize_internal_state_machine( struct flash_bank_s * bank )
+int tms470_flash_initialize_internal_state_machine(struct flash_bank_s *bank)
 {
-  u32 fmmac2, fmmac1, fmmaxep, k, delay, glbctrl, sysclk;
-  target_t *target = bank->target;
-  tms470_flash_bank_t * tms470_info = bank->driver_priv;
-  int result = ERROR_OK;
-
-  /*
-   * Select the desired bank to be programmed by writing BANK[2:0] of
-   * FMMAC2.
-   */
-  target_read_u32( target, 0xFFE8BC04, &fmmac2 );
-  fmmac2 &= ~0x0007;
-  fmmac2 |= (tms470_info->ordinal & 7);
-  target_write_u32( target, 0xFFE8BC04, fmmac2 );
-  DEBUG( "set fmmac2=0x%04x", fmmac2 );
-
-  /*
-   * Disable level 1 sector protection by setting bit 15 of FMMAC1.
-   */
-  target_read_u32( target, 0xFFE8BC00, &fmmac1 );
-  fmmac1 |= 0x8000;
-  target_write_u32( target, 0xFFE8BC00, fmmac1 );
-  DEBUG( "set fmmac1=0x%04x", fmmac1 );
-
-  /*
-   * FMTCREG=0x2fc0;
-   */
-  target_write_u32( target, 0xFFE8BC10, 0x2fc0 );
-  DEBUG( "set fmtcreg=0x2fc0" );
-
-  /*
-   * MAXPP=50
-   */
-  target_write_u32( target, 0xFFE8A07C, 50 );
-  DEBUG( "set fmmaxpp=50" );
-
-  /*
-   * MAXCP=0xf000+2000
-   */
-  target_write_u32( target, 0xFFE8A084, 0xf000+2000 );
-  DEBUG( "set fmmaxcp=0x%04x", 0xf000+2000 );
-
-    /*
-   * configure VHV
-   */
-  target_read_u32( target, 0xFFE8A080, &fmmaxep );
-  if (fmmaxep == 0xf000) 
-    {
-      fmmaxep = 0xf000+4095;
-      target_write_u32( target, 0xFFE8A80C, 0x9964 );
-      DEBUG( "set fmptr3=0x9964" );
-    }
-  else
-    {
-      fmmaxep = 0xa000+4095;
-      target_write_u32( target, 0xFFE8A80C, 0x9b64 );
-      DEBUG( "set fmptr3=0x9b64" );
-    }
-  target_write_u32( target, 0xFFE8A080, fmmaxep );
-  DEBUG( "set fmmaxep=0x%04x", fmmaxep );
-
-  /*
-   * FMPTR4=0xa000
-   */
-  target_write_u32( target, 0xFFE8A810, 0xa000 );
-  DEBUG( "set fmptr4=0xa000" );
-
-  /*
-   * FMPESETUP, delay parameter selected based on clock frequency.
-   *
-   * According to the TI App Note SPNU257 and flashing code, delay is
-   * int((sysclk(MHz) + 1) / 2), with a minimum of 5.  The system
-   * clock is usually derived from the ZPLL module, and selected by
-   * the plldis global.
-   */
-  target_read_u32( target, 0xFFFFFFDC, &glbctrl );
-  sysclk = (plldis ? 1 : (glbctrl & 0x08) ? 4 : 8 ) * oscMHz / (1 + (glbctrl & 7));
-  delay = (sysclk > 10) ? (sysclk + 1) / 2 : 5;
-  target_write_u32( target, 0xFFE8A018, (delay<<4)|(delay<<8) );
-  DEBUG( "set fmpsetup=0x%04x", (delay<<4)|(delay<<8) );
-
-  /*
-   * FMPVEVACCESS, based on delay.
-   */
-  k = delay|(delay<<8);
-  target_write_u32( target, 0xFFE8A05C, k );
-  DEBUG( "set fmpvevaccess=0x%04x", k );
-  
-  /*
-   * FMPCHOLD, FMPVEVHOLD, FMPVEVSETUP, based on delay.
-   */
-  k <<= 1;
-  target_write_u32( target, 0xFFE8A034, k );
-  DEBUG( "set fmpchold=0x%04x", k );
-  target_write_u32( target, 0xFFE8A040, k );
-  DEBUG( "set fmpvevhold=0x%04x", k );
-  target_write_u32( target, 0xFFE8A024, k );
-  DEBUG( "set fmpvevsetup=0x%04x", k );
-
-  /*
-   * FMCVACCESS, based on delay.
-   */
-  k = delay*16;
-  target_write_u32( target, 0xFFE8A060, k );
-  DEBUG( "set fmcvaccess=0x%04x", k );
-
-  /*
-   * FMCSETUP, based on delay.
-   */
-  k = 0x3000 | delay*20;
-  target_write_u32( target, 0xFFE8A020, k );
-  DEBUG( "set fmcsetup=0x%04x", k );
-
-  /*
-   * FMEHOLD, based on delay.
-   */
-  k = (delay*20) << 2;
-  target_write_u32( target, 0xFFE8A038, k );
-  DEBUG( "set fmehold=0x%04x", k );
-
-  /*
-   * PWIDTH, CWIDTH, EWIDTH, based on delay.
-   */
-  target_write_u32( target, 0xFFE8A050, delay*8 );
-  DEBUG( "set fmpwidth=0x%04x", delay*8 );
-  target_write_u32( target, 0xFFE8A058, delay*1000 );
-  DEBUG( "set fmcwidth=0x%04x", delay*1000 );
-  target_write_u32( target, 0xFFE8A054, delay*5400 );
-  DEBUG( "set fmewidth=0x%04x", delay*5400 );
-
-  return result;
+       u32 fmmac2, fmmac1, fmmaxep, k, delay, glbctrl, sysclk;
+       target_t *target = bank->target;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+       int result = ERROR_OK;
+
+       /*
+        * Select the desired bank to be programmed by writing BANK[2:0] of
+        * FMMAC2.
+        */
+       target_read_u32(target, 0xFFE8BC04, &fmmac2);
+       fmmac2 &= ~0x0007;
+       fmmac2 |= (tms470_info->ordinal & 7);
+       target_write_u32(target, 0xFFE8BC04, fmmac2);
+       DEBUG("set fmmac2=0x%04x", fmmac2);
+
+       /*
+        * Disable level 1 sector protection by setting bit 15 of FMMAC1.
+        */
+       target_read_u32(target, 0xFFE8BC00, &fmmac1);
+       fmmac1 |= 0x8000;
+       target_write_u32(target, 0xFFE8BC00, fmmac1);
+       DEBUG("set fmmac1=0x%04x", fmmac1);
+
+       /*
+        * FMTCREG=0x2fc0;
+        */
+       target_write_u32(target, 0xFFE8BC10, 0x2fc0);
+       DEBUG("set fmtcreg=0x2fc0");
+
+       /*
+        * MAXPP=50
+        */
+       target_write_u32(target, 0xFFE8A07C, 50);
+       DEBUG("set fmmaxpp=50");
+
+       /*
+        * MAXCP=0xf000+2000
+        */
+       target_write_u32(target, 0xFFE8A084, 0xf000 + 2000);
+       DEBUG("set fmmaxcp=0x%04x", 0xf000 + 2000);
+
+       /*
+        * configure VHV
+        */
+       target_read_u32(target, 0xFFE8A080, &fmmaxep);
+       if (fmmaxep == 0xf000)
+       {
+               fmmaxep = 0xf000 + 4095;
+               target_write_u32(target, 0xFFE8A80C, 0x9964);
+               DEBUG("set fmptr3=0x9964");
+       }
+       else
+       {
+               fmmaxep = 0xa000 + 4095;
+               target_write_u32(target, 0xFFE8A80C, 0x9b64);
+               DEBUG("set fmptr3=0x9b64");
+       }
+       target_write_u32(target, 0xFFE8A080, fmmaxep);
+       DEBUG("set fmmaxep=0x%04x", fmmaxep);
+
+       /*
+        * FMPTR4=0xa000
+        */
+       target_write_u32(target, 0xFFE8A810, 0xa000);
+       DEBUG("set fmptr4=0xa000");
+
+       /*
+        * FMPESETUP, delay parameter selected based on clock frequency.
+        *
+        * According to the TI App Note SPNU257 and flashing code, delay is
+        * int((sysclk(MHz) + 1) / 2), with a minimum of 5.  The system
+        * clock is usually derived from the ZPLL module, and selected by
+        * the plldis global.
+        */
+       target_read_u32(target, 0xFFFFFFDC, &glbctrl);
+       sysclk = (plldis ? 1 : (glbctrl & 0x08) ? 4 : 8) * oscMHz / (1 + (glbctrl & 7));
+       delay = (sysclk > 10) ? (sysclk + 1) / 2 : 5;
+       target_write_u32(target, 0xFFE8A018, (delay << 4) | (delay << 8));
+       DEBUG("set fmpsetup=0x%04x", (delay << 4) | (delay << 8));
+
+       /*
+        * FMPVEVACCESS, based on delay.
+        */
+       k = delay | (delay << 8);
+       target_write_u32(target, 0xFFE8A05C, k);
+       DEBUG("set fmpvevaccess=0x%04x", k);
+
+       /*
+        * FMPCHOLD, FMPVEVHOLD, FMPVEVSETUP, based on delay.
+        */
+       k <<= 1;
+       target_write_u32(target, 0xFFE8A034, k);
+       DEBUG("set fmpchold=0x%04x", k);
+       target_write_u32(target, 0xFFE8A040, k);
+       DEBUG("set fmpvevhold=0x%04x", k);
+       target_write_u32(target, 0xFFE8A024, k);
+       DEBUG("set fmpvevsetup=0x%04x", k);
+
+       /*
+        * FMCVACCESS, based on delay.
+        */
+       k = delay * 16;
+       target_write_u32(target, 0xFFE8A060, k);
+       DEBUG("set fmcvaccess=0x%04x", k);
+
+       /*
+        * FMCSETUP, based on delay.
+        */
+       k = 0x3000 | delay * 20;
+       target_write_u32(target, 0xFFE8A020, k);
+       DEBUG("set fmcsetup=0x%04x", k);
+
+       /*
+        * FMEHOLD, based on delay.
+        */
+       k = (delay * 20) << 2;
+       target_write_u32(target, 0xFFE8A038, k);
+       DEBUG("set fmehold=0x%04x", k);
+
+       /*
+        * PWIDTH, CWIDTH, EWIDTH, based on delay.
+        */
+       target_write_u32(target, 0xFFE8A050, delay * 8);
+       DEBUG("set fmpwidth=0x%04x", delay * 8);
+       target_write_u32(target, 0xFFE8A058, delay * 1000);
+       DEBUG("set fmcwidth=0x%04x", delay * 1000);
+       target_write_u32(target, 0xFFE8A054, delay * 5400);
+       DEBUG("set fmewidth=0x%04x", delay * 5400);
+
+       return result;
 }
-                                                
 
 /* ---------------------------------------------------------------------- */
 
-int
-tms470_flash_status( struct flash_bank_s * bank )
+int tms470_flash_status(struct flash_bank_s *bank)
 {
-  target_t *target = bank->target;
-  int result = ERROR_OK;
-  u32 fmmstat;
-
-  target_read_u32( target, 0xFFE8BC0C, &fmmstat );
-  DEBUG( "set fmmstat=0x%04x", fmmstat );
-
-  if (fmmstat & 0x0080)
-    {
-      WARNING( "tms470 flash command: erase still active after busy clear." );
-      result = ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  if (fmmstat & 0x0040)
-    {
-      WARNING( "tms470 flash command: program still active after busy clear." );
-      result = ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  if (fmmstat & 0x0020)
-    {
-      WARNING( "tms470 flash command: invalid data command." );
-      result = ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  if (fmmstat & 0x0010)
-    {
-      WARNING( "tms470 flash command: program, erase or validate sector failed." );
-      result = ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  if (fmmstat & 0x0008)
-    {
-      WARNING( "tms470 flash command: voltage instability detected." );
-      result = ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  if (fmmstat & 0x0006)
-    {
-      WARNING( "tms470 flash command: command suspend detected." );
-      result = ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  if (fmmstat & 0x0001)
-    {
-      WARNING( "tms470 flash command: sector was locked." );
-      result = ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  return result;
+       target_t *target = bank->target;
+       int result = ERROR_OK;
+       u32 fmmstat;
+
+       target_read_u32(target, 0xFFE8BC0C, &fmmstat);
+       DEBUG("set fmmstat=0x%04x", fmmstat);
+
+       if (fmmstat & 0x0080)
+       {
+               WARNING("tms470 flash command: erase still active after busy clear.");
+               result = ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       if (fmmstat & 0x0040)
+       {
+               WARNING("tms470 flash command: program still active after busy clear.");
+               result = ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       if (fmmstat & 0x0020)
+       {
+               WARNING("tms470 flash command: invalid data command.");
+               result = ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       if (fmmstat & 0x0010)
+       {
+               WARNING("tms470 flash command: program, erase or validate sector failed.");
+               result = ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       if (fmmstat & 0x0008)
+       {
+               WARNING("tms470 flash command: voltage instability detected.");
+               result = ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       if (fmmstat & 0x0006)
+       {
+               WARNING("tms470 flash command: command suspend detected.");
+               result = ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       if (fmmstat & 0x0001)
+       {
+               WARNING("tms470 flash command: sector was locked.");
+               result = ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       return result;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int
-tms470_erase_sector( struct flash_bank_s * bank, 
-                     int                   sector )
+int tms470_erase_sector(struct flash_bank_s *bank, int sector)
 {
-  u32 glbctrl, orig_fmregopt, fmbsea, fmbseb, fmmstat;
-  target_t *target = bank->target;
-  u32 flashAddr = bank->base + bank->sectors[sector].offset;
-  int result = ERROR_OK;
-
-  /* 
-   * Set the bit GLBCTRL4 of the GLBCTRL register (in the System
-   * module) to enable writing to the flash registers }.
-   */
-  target_read_u32( target, 0xFFFFFFDC, &glbctrl );
-  target_write_u32( target, 0xFFFFFFDC, glbctrl | 0x10 );
-  DEBUG( "set glbctrl=0x%08x", glbctrl | 0x10 );
-
-  /* Force normal read mode. */
-  target_read_u32( target, 0xFFE89C00, &orig_fmregopt );
-  target_write_u32( target, 0xFFE89C00, 0 );
-  DEBUG( "set fmregopt=0x%08x", 0 );
-
-  (void) tms470_flash_initialize_internal_state_machine( bank );
-  
-  /*
-   * Select one or more bits in FMBSEA or FMBSEB to disable Level 1
-   * protection for the particular sector to be erased/written.
-   */
-  if (sector < 16)
-    {
-      target_read_u32( target, 0xFFE88008, &fmbsea );
-      target_write_u32( target, 0xFFE88008, fmbsea | (1<<sector) );
-      DEBUG( "set fmbsea=0x%04x", fmbsea | (1<<sector) );
-    }
-  else
-    {
-      target_read_u32( target, 0xFFE8800C, &fmbseb );
-      target_write_u32( target, 0xFFE8800C, fmbseb | (1<<(sector-16)) );
-      DEBUG( "set fmbseb=0x%04x", fmbseb | (1<<(sector-16)) );
-    }
-  bank->sectors[sector].is_protected = 0;
-
-  /* 
-   * clear status regiser, sent erase command, kickoff erase 
-   */
-  target_write_u16( target, flashAddr, 0x0040 );
-  DEBUG( "write *(u16 *)0x%08x=0x0040", flashAddr );
-  target_write_u16( target, flashAddr, 0x0020 );
-  DEBUG( "write *(u16 *)0x%08x=0x0020", flashAddr );
-  target_write_u16( target, flashAddr, 0xffff );
-  DEBUG( "write *(u16 *)0x%08x=0xffff", flashAddr );
-
-  /*
-   * Monitor FMMSTAT, busy until clear, then check and other flags for
-   * ultimate result of the operation.
-   */
-  do
-    {
-      target_read_u32( target, 0xFFE8BC0C, &fmmstat );
-      if (fmmstat & 0x0100)
-        {
-          usleep( 1000 );
-        }
-    }
-  while( fmmstat & 0x0100 );
-
-  result = tms470_flash_status( bank );
-
-  if (sector < 16)
-    {
-      target_write_u32( target, 0xFFE88008, fmbsea );
-      DEBUG( "set fmbsea=0x%04x", fmbsea );
-      bank->sectors[sector].is_protected = 
-        fmbsea & (1<<sector) ? 0 : 1;
-    }
-  else
-    {
-      target_write_u32( target, 0xFFE8800C, fmbseb );
-      DEBUG( "set fmbseb=0x%04x", fmbseb );
-      bank->sectors[sector].is_protected = 
-        fmbseb & (1<<(sector-16)) ? 0 : 1;
-    }
-  target_write_u32( target, 0xFFE89C00, orig_fmregopt );
-  DEBUG( "set fmregopt=0x%08x", orig_fmregopt );
-  target_write_u32( target, 0xFFFFFFDC, glbctrl );
-  DEBUG( "set glbctrl=0x%08x", glbctrl );
-
-  if (result == ERROR_OK)
-    {
-      bank->sectors[sector].is_erased = 1;
-    }
-
-  return result;
+       u32 glbctrl, orig_fmregopt, fmbsea, fmbseb, fmmstat;
+       target_t *target = bank->target;
+       u32 flashAddr = bank->base + bank->sectors[sector].offset;
+       int result = ERROR_OK;
+
+       /* 
+        * Set the bit GLBCTRL4 of the GLBCTRL register (in the System
+        * module) to enable writing to the flash registers }.
+        */
+       target_read_u32(target, 0xFFFFFFDC, &glbctrl);
+       target_write_u32(target, 0xFFFFFFDC, glbctrl | 0x10);
+       DEBUG("set glbctrl=0x%08x", glbctrl | 0x10);
+
+       /* Force normal read mode. */
+       target_read_u32(target, 0xFFE89C00, &orig_fmregopt);
+       target_write_u32(target, 0xFFE89C00, 0);
+       DEBUG("set fmregopt=0x%08x", 0);
+
+       (void)tms470_flash_initialize_internal_state_machine(bank);
+
+       /*
+        * Select one or more bits in FMBSEA or FMBSEB to disable Level 1
+        * protection for the particular sector to be erased/written.
+        */
+       if (sector < 16)
+       {
+               target_read_u32(target, 0xFFE88008, &fmbsea);
+               target_write_u32(target, 0xFFE88008, fmbsea | (1 << sector));
+               DEBUG("set fmbsea=0x%04x", fmbsea | (1 << sector));
+       }
+       else
+       {
+               target_read_u32(target, 0xFFE8800C, &fmbseb);
+               target_write_u32(target, 0xFFE8800C, fmbseb | (1 << (sector - 16)));
+               DEBUG("set fmbseb=0x%04x", fmbseb | (1 << (sector - 16)));
+       }
+       bank->sectors[sector].is_protected = 0;
+
+       /* 
+        * clear status regiser, sent erase command, kickoff erase 
+        */
+       target_write_u16(target, flashAddr, 0x0040);
+       DEBUG("write *(u16 *)0x%08x=0x0040", flashAddr);
+       target_write_u16(target, flashAddr, 0x0020);
+       DEBUG("write *(u16 *)0x%08x=0x0020", flashAddr);
+       target_write_u16(target, flashAddr, 0xffff);
+       DEBUG("write *(u16 *)0x%08x=0xffff", flashAddr);
+
+       /*
+        * Monitor FMMSTAT, busy until clear, then check and other flags for
+        * ultimate result of the operation.
+        */
+       do
+       {
+               target_read_u32(target, 0xFFE8BC0C, &fmmstat);
+               if (fmmstat & 0x0100)
+               {
+                       usleep(1000);
+               }
+       }
+       while (fmmstat & 0x0100);
+
+       result = tms470_flash_status(bank);
+
+       if (sector < 16)
+       {
+               target_write_u32(target, 0xFFE88008, fmbsea);
+               DEBUG("set fmbsea=0x%04x", fmbsea);
+               bank->sectors[sector].is_protected = fmbsea & (1 << sector) ? 0 : 1;
+       }
+       else
+       {
+               target_write_u32(target, 0xFFE8800C, fmbseb);
+               DEBUG("set fmbseb=0x%04x", fmbseb);
+               bank->sectors[sector].is_protected = fmbseb & (1 << (sector - 16)) ? 0 : 1;
+       }
+       target_write_u32(target, 0xFFE89C00, orig_fmregopt);
+       DEBUG("set fmregopt=0x%08x", orig_fmregopt);
+       target_write_u32(target, 0xFFFFFFDC, glbctrl);
+       DEBUG("set glbctrl=0x%08x", glbctrl);
+
+       if (result == ERROR_OK)
+       {
+               bank->sectors[sector].is_erased = 1;
+       }
+
+       return result;
 }
 
 /* ---------------------------------------------------------------------- 
               Implementation of Flash Driver Interfaces
    ---------------------------------------------------------------------- */
 
-int 
-tms470_register_commands( struct command_context_s *cmd_ctx )
+int tms470_register_commands(struct command_context_s *cmd_ctx)
 {
-  command_t *tms470_cmd = register_command( cmd_ctx, 
-                                            NULL, 
-                                            "tms470", 
-                                            NULL, 
-                                            COMMAND_ANY, 
-                                            "applies to TI tms470 family" );
-
-  register_command( cmd_ctx, 
-                    tms470_cmd, 
-                    "flash_keyset", 
-                    tms470_handle_flash_keyset_command, 
-                    COMMAND_ANY,
-                   "tms470 flash_keyset <key0> <key1> <key2> <key3>" );
-
-  register_command( cmd_ctx,
-                    tms470_cmd,
-                    "osc_megahertz",
-                    tms470_handle_osc_megahertz_command,
-                    COMMAND_ANY,
-                    "tms470 osc_megahertz <MHz>" );
-
-  register_command( cmd_ctx,
-                    tms470_cmd,
-                    "plldis",
-                    tms470_handle_plldis_command,
-                    COMMAND_ANY,
-                    "tms470 plldis <0/1>" );
-  
-  return ERROR_OK;
+       command_t *tms470_cmd = register_command(cmd_ctx, NULL, "tms470", NULL, COMMAND_ANY, "applies to TI tms470 family");
+
+       register_command(cmd_ctx, tms470_cmd, "flash_keyset", tms470_handle_flash_keyset_command, COMMAND_ANY, "tms470 flash_keyset <key0> <key1> <key2> <key3>");
+       register_command(cmd_ctx, tms470_cmd, "osc_megahertz", tms470_handle_osc_megahertz_command, COMMAND_ANY, "tms470 osc_megahertz <MHz>");
+       register_command(cmd_ctx, tms470_cmd, "plldis", tms470_handle_plldis_command, COMMAND_ANY, "tms470 plldis <0/1>");
+
+       return ERROR_OK;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_erase( struct flash_bank_s * bank, 
-              int                   first, 
-              int                   last )
+int tms470_erase(struct flash_bank_s *bank, int first, int last)
 {
-  tms470_flash_bank_t *tms470_info = bank->driver_priv;
-  int sector, result = ERROR_OK;
-
-  if (!tms470_info->device_ident_reg)
-    {
-      tms470_read_part_info( bank );
-    }
-
-  if ((first < 0) || 
-      (first >= bank->num_sectors) ||
-      (last < 0) ||
-      (last >= bank->num_sectors) ||
-      (first > last))
-    {
-      ERROR( "Sector range %d to %d invalid.", first, last );
-      return ERROR_FLASH_SECTOR_INVALID;
-    }
-
-  result = tms470_unlock_flash( bank );
-  if (result != ERROR_OK)
-    {
-      return result;
-    }
-
-  for( sector=first; sector<=last; sector++ )
-    {
-      INFO( "Erasing tms470 bank %d sector %d...",
-            tms470_info->ordinal, sector );
-
-      result = tms470_erase_sector( bank, sector );
-
-      if (result != ERROR_OK)
-        {
-          ERROR( "tms470 could not erase flash sector." );
-          break;
-        }
-      else
-        {
-          INFO( "sector erased successfully." );
-        }
-    }
-  
-  return result;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+       int sector, result = ERROR_OK;
+
+       if (!tms470_info->device_ident_reg)
+       {
+               tms470_read_part_info(bank);
+       }
+
+       if ((first < 0) || (first >= bank->num_sectors) || (last < 0) || (last >= bank->num_sectors) || (first > last))
+       {
+               ERROR("Sector range %d to %d invalid.", first, last);
+               return ERROR_FLASH_SECTOR_INVALID;
+       }
+
+       result = tms470_unlock_flash(bank);
+       if (result != ERROR_OK)
+       {
+               return result;
+       }
+
+       for (sector = first; sector <= last; sector++)
+       {
+               INFO("Erasing tms470 bank %d sector %d...", tms470_info->ordinal, sector);
+
+               result = tms470_erase_sector(bank, sector);
+
+               if (result != ERROR_OK)
+               {
+                       ERROR("tms470 could not erase flash sector.");
+                       break;
+               }
+               else
+               {
+                       INFO("sector erased successfully.");
+               }
+       }
+
+       return result;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_protect( struct flash_bank_s * bank, 
-                int                   set, 
-                int                   first, 
-                int                   last )
+int tms470_protect(struct flash_bank_s *bank, int set, int first, int last)
 {
-  tms470_flash_bank_t *tms470_info = bank->driver_priv;
-  target_t *target = bank->target;
-  u32 fmmac2, fmbsea, fmbseb;
-  int sector;
-
-  if (!tms470_info->device_ident_reg)
-    {
-      tms470_read_part_info( bank );
-    }
-
-  if ((first < 0) || 
-      (first >= bank->num_sectors) ||
-      (last < 0) ||
-      (last >= bank->num_sectors) ||
-      (first > last))
-    {
-      ERROR( "Sector range %d to %d invalid.", first, last );
-      return ERROR_FLASH_SECTOR_INVALID;
-    }
-
-  /* enable the appropriate bank */
-  target_read_u32( target, 0xFFE8BC04, &fmmac2 );
-  target_write_u32( target, 0xFFE8BC04, 
-                    (fmmac2 & ~7) | tms470_info->ordinal );
-
-  /* get the original sector proection flags for this bank */
-  target_read_u32( target, 0xFFE88008, &fmbsea );
-  target_read_u32( target, 0xFFE8800C, &fmbseb );
-  
-  for( sector=0; sector<bank->num_sectors; sector++ )
-    {
-      if (sector < 16)
-        {
-          fmbsea = set ? fmbsea & ~(1<<sector) : 
-                         fmbsea | (1<<sector);
-          bank->sectors[sector].is_protected = set ? 1 : 0;
-        }
-      else
-        {
-          fmbseb = set ? fmbseb & ~(1<<(sector-16)) : 
-                         fmbseb | (1<<(sector-16));
-          bank->sectors[sector].is_protected = set ? 1 : 0;
-        }
-    }
-
-  /* update the protection bits */
-  target_write_u32( target, 0xFFE88008, fmbsea );
-  target_write_u32( target, 0xFFE8800C, fmbseb );
-
-  return ERROR_OK;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+       target_t *target = bank->target;
+       u32 fmmac2, fmbsea, fmbseb;
+       int sector;
+
+       if (!tms470_info->device_ident_reg)
+       {
+               tms470_read_part_info(bank);
+       }
+
+       if ((first < 0) || (first >= bank->num_sectors) || (last < 0) || (last >= bank->num_sectors) || (first > last))
+       {
+               ERROR("Sector range %d to %d invalid.", first, last);
+               return ERROR_FLASH_SECTOR_INVALID;
+       }
+
+       /* enable the appropriate bank */
+       target_read_u32(target, 0xFFE8BC04, &fmmac2);
+       target_write_u32(target, 0xFFE8BC04, (fmmac2 & ~7) | tms470_info->ordinal);
+
+       /* get the original sector proection flags for this bank */
+       target_read_u32(target, 0xFFE88008, &fmbsea);
+       target_read_u32(target, 0xFFE8800C, &fmbseb);
+
+       for (sector = 0; sector < bank->num_sectors; sector++)
+       {
+               if (sector < 16)
+               {
+                       fmbsea = set ? fmbsea & ~(1 << sector) : fmbsea | (1 << sector);
+                       bank->sectors[sector].is_protected = set ? 1 : 0;
+               }
+               else
+               {
+                       fmbseb = set ? fmbseb & ~(1 << (sector - 16)) : fmbseb | (1 << (sector - 16));
+                       bank->sectors[sector].is_protected = set ? 1 : 0;
+               }
+       }
+
+       /* update the protection bits */
+       target_write_u32(target, 0xFFE88008, fmbsea);
+       target_write_u32(target, 0xFFE8800C, fmbseb);
+
+       return ERROR_OK;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_write( struct flash_bank_s * bank, 
-              u8 *                  buffer, 
-              u32                   offset, 
-              u32                   count )
+int tms470_write(struct flash_bank_s *bank, u8 * buffer, u32 offset, u32 count)
 {
-  target_t *target = bank->target;
-  tms470_flash_bank_t *tms470_info = bank->driver_priv;
-  u32 glbctrl, fmbac2, orig_fmregopt, fmbsea, fmbseb, fmmaxpp, fmmstat;
-  int i, result = ERROR_OK;
-
-  if (!tms470_info->device_ident_reg)
-    {
-      tms470_read_part_info( bank );
-    }
-
-  INFO( "Writing %d bytes starting at 0x%08x",
-        count, bank->base + offset );
-
-  /* set GLBCTRL.4  */
-  target_read_u32( target, 0xFFFFFFDC, &glbctrl );
-  target_write_u32( target, 0xFFFFFFDC, glbctrl | 0x10 );
-
-  (void) tms470_flash_initialize_internal_state_machine( bank );
-
-  /* force max wait states */
-  target_read_u32( target, 0xFFE88004, &fmbac2 );
-  target_write_u32( target, 0xFFE88004, fmbac2 | 0xff );
-
-  /* save current access mode, force normal read mode */
-  target_read_u32( target, 0xFFE89C00, &orig_fmregopt );
-  target_write_u32( target, 0xFFE89C00, 0x00 );
-
-  /*
-   * Disable Level 1 protection for all sectors to be erased/written.
-   */
-  target_read_u32( target, 0xFFE88008, &fmbsea );
-  target_write_u32( target, 0xFFE88008, 0xffff );
-  target_read_u32( target, 0xFFE8800C, &fmbseb );
-  target_write_u32( target, 0xFFE8800C, 0xffff );
-
-  /* read MAXPP */
-  target_read_u32( target, 0xFFE8A07C, &fmmaxpp );
-
-  for( i=0; i<count; i+=2 )
-    {
-      u32 addr = bank->base + offset + i;
-      u16 word = (((u16) buffer[i]) << 8) | (u16) buffer[i+1];
-
-      if (word != 0xffff)
-        {
-          INFO( "writing 0x%04x at 0x%08x", word, addr );
-
-          /* clear status register */
-          target_write_u16( target, addr, 0x0040 );
-          /* program flash command */
-          target_write_u16( target, addr, 0x0010 );
-          /* burn the 16-bit word (big-endian) */
-          target_write_u16( target, addr, word );
-
-          /*
-           * Monitor FMMSTAT, busy until clear, then check and other flags
-           * for ultimate result of the operation.
-           */
-          do
-            {
-              target_read_u32( target, 0xFFE8BC0C, &fmmstat );
-              if (fmmstat & 0x0100)
-                {
-                  usleep( 1000 );
-                }
-            }
-          while( fmmstat & 0x0100 );
-
-          if (fmmstat & 0x3ff)
-            {
-              ERROR( "fmstat=0x%04x", fmmstat );
-              ERROR( "Could not program word 0x%04x at address 0x%08x.",
-                     word, addr );
-              result = ERROR_FLASH_OPERATION_FAILED;
-              break;
-            }
-        }
-      else
-        {
-          INFO( "skipping 0xffff at 0x%08x", addr );
-        }
-    }
-
-  /* restore */
-  target_write_u32( target, 0xFFE88008, fmbsea );
-  target_write_u32( target, 0xFFE8800C, fmbseb );
-  target_write_u32( target, 0xFFE88004, fmbac2 );
-  target_write_u32( target, 0xFFE89C00, orig_fmregopt );
-  target_write_u32( target, 0xFFFFFFDC, glbctrl );
-
-  return result;
+       target_t *target = bank->target;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+       u32 glbctrl, fmbac2, orig_fmregopt, fmbsea, fmbseb, fmmaxpp, fmmstat;
+       int i, result = ERROR_OK;
+
+       if (!tms470_info->device_ident_reg)
+       {
+               tms470_read_part_info(bank);
+       }
+
+       INFO("Writing %d bytes starting at 0x%08x", count, bank->base + offset);
+
+       /* set GLBCTRL.4  */
+       target_read_u32(target, 0xFFFFFFDC, &glbctrl);
+       target_write_u32(target, 0xFFFFFFDC, glbctrl | 0x10);
+
+       (void)tms470_flash_initialize_internal_state_machine(bank);
+
+       /* force max wait states */
+       target_read_u32(target, 0xFFE88004, &fmbac2);
+       target_write_u32(target, 0xFFE88004, fmbac2 | 0xff);
+
+       /* save current access mode, force normal read mode */
+       target_read_u32(target, 0xFFE89C00, &orig_fmregopt);
+       target_write_u32(target, 0xFFE89C00, 0x00);
+
+       /*
+        * Disable Level 1 protection for all sectors to be erased/written.
+        */
+       target_read_u32(target, 0xFFE88008, &fmbsea);
+       target_write_u32(target, 0xFFE88008, 0xffff);
+       target_read_u32(target, 0xFFE8800C, &fmbseb);
+       target_write_u32(target, 0xFFE8800C, 0xffff);
+
+       /* read MAXPP */
+       target_read_u32(target, 0xFFE8A07C, &fmmaxpp);
+
+       for (i = 0; i < count; i += 2)
+       {
+               u32 addr = bank->base + offset + i;
+               u16 word = (((u16) buffer[i]) << 8) | (u16) buffer[i + 1];
+
+               if (word != 0xffff)
+               {
+                       INFO("writing 0x%04x at 0x%08x", word, addr);
+
+                       /* clear status register */
+                       target_write_u16(target, addr, 0x0040);
+                       /* program flash command */
+                       target_write_u16(target, addr, 0x0010);
+                       /* burn the 16-bit word (big-endian) */
+                       target_write_u16(target, addr, word);
+
+                       /*
+                        * Monitor FMMSTAT, busy until clear, then check and other flags
+                        * for ultimate result of the operation.
+                        */
+                       do
+                       {
+                               target_read_u32(target, 0xFFE8BC0C, &fmmstat);
+                               if (fmmstat & 0x0100)
+                               {
+                                       usleep(1000);
+                               }
+                       }
+                       while (fmmstat & 0x0100);
+
+                       if (fmmstat & 0x3ff)
+                       {
+                               ERROR("fmstat=0x%04x", fmmstat);
+                               ERROR("Could not program word 0x%04x at address 0x%08x.", word, addr);
+                               result = ERROR_FLASH_OPERATION_FAILED;
+                               break;
+                       }
+               }
+               else
+               {
+                       INFO("skipping 0xffff at 0x%08x", addr);
+               }
+       }
+
+       /* restore */
+       target_write_u32(target, 0xFFE88008, fmbsea);
+       target_write_u32(target, 0xFFE8800C, fmbseb);
+       target_write_u32(target, 0xFFE88004, fmbac2);
+       target_write_u32(target, 0xFFE89C00, orig_fmregopt);
+       target_write_u32(target, 0xFFFFFFDC, glbctrl);
+
+       return result;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_probe( struct flash_bank_s * bank )
+int tms470_probe(struct flash_bank_s *bank)
 {
-  tms470_flash_bank_t * tms470_info = bank->driver_priv;
-
-  tms470_info->probed = 0;
-  
-  if (!tms470_info->device_ident_reg)
-    {
-      tms470_read_part_info( bank );
-    }
-  
-  tms470_info->probed = 1;
-
-  return ERROR_OK;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+
+       tms470_info->probed = 0;
+
+       if (!tms470_info->device_ident_reg)
+       {
+               tms470_read_part_info(bank);
+       }
+
+       tms470_info->probed = 1;
+
+       return ERROR_OK;
 }
 
-int 
-tms470_auto_probe( struct flash_bank_s * bank )
+int tms470_auto_probe(struct flash_bank_s *bank)
 {
-       tms470_flash_bank_t * tms470_info = bank->driver_priv;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+
        if (tms470_info->probed)
                return ERROR_OK;
        return tms470_probe(bank);
 }
+
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_erase_check( struct flash_bank_s * bank )
+int tms470_erase_check(struct flash_bank_s *bank)
 {
-  target_t *target = bank->target;
-  tms470_flash_bank_t * tms470_info = bank->driver_priv;
-  int sector, result = ERROR_OK;
-  u32 fmmac2, fmbac2, glbctrl, orig_fmregopt;
-  static u8 buffer[64*1024];
-
-  if (!tms470_info->device_ident_reg)
-    {
-      tms470_read_part_info( bank );
-    }
-
-  /* set GLBCTRL.4  */
-  target_read_u32( target, 0xFFFFFFDC, &glbctrl );
-  target_write_u32( target, 0xFFFFFFDC, glbctrl | 0x10 );
-
-  /* save current access mode, force normal read mode */
-  target_read_u32( target, 0xFFE89C00, &orig_fmregopt );
-  target_write_u32( target, 0xFFE89C00, 0x00 );
-
-  /* enable the appropriate bank */
-  target_read_u32( target, 0xFFE8BC04, &fmmac2 );
-  target_write_u32( target, 0xFFE8BC04, 
-                    (fmmac2 & ~7) | tms470_info->ordinal );
-
-  /* TCR=0 */
-  target_write_u32( target, 0xFFE8BC10, 0x2fc0 );
-
-  /* clear TEZ in fmbrdy */
-  target_write_u32( target, 0xFFE88010, 0x0b );
-
-  /* save current wait states, force max */
-  target_read_u32( target, 0xFFE88004, &fmbac2 );
-  target_write_u32( target, 0xFFE88004, fmbac2 | 0xff );
-
-  /* 
-   * The TI primitives inspect the flash memory by reading one 32-bit
-   * word at a time.  Here we read an entire sector and inspect it in
-   * an attempt to reduce the JTAG overhead.
-   */
-  for( sector=0; sector<bank->num_sectors; sector++ )
-    {
-      if (bank->sectors[sector].is_erased != 1)
-        {
-          u32 i, addr = bank->base + bank->sectors[sector].offset;
-          
-          INFO( "checking flash bank %d sector %d",
-                tms470_info->ordinal,
-                sector );
-
-          target_read_buffer( target, 
-                              addr,
-                              bank->sectors[sector].size,
-                              buffer );
-
-          bank->sectors[sector].is_erased = 1;
-          for( i=0; i<bank->sectors[sector].size; i++ )
-            {
-              if (buffer[i] != 0xff)
-                {
-                  WARNING( "tms470 bank %d, sector %d, not erased.",
-                           tms470_info->ordinal,
-                           sector );
-                  WARNING( "at location 0x%08x: flash data is 0x%02x.",
-                           addr+i, 
-                           buffer[i] );
-                           
-                  bank->sectors[sector].is_erased = 0;
-                  break;
-                }
-            }
-        }
-      if (bank->sectors[sector].is_erased != 1)
-        {
-          result = ERROR_FLASH_SECTOR_NOT_ERASED;
-          break;
-        }
-      else
-        {
-          INFO( "sector erased" );
-        }
-    }
-
-  /* reset TEZ, wait states, read mode, GLBCTRL.4 */
-  target_write_u32( target, 0xFFE88010, 0x0f );
-  target_write_u32( target, 0xFFE88004, fmbac2 );
-  target_write_u32( target, 0xFFE89C00, orig_fmregopt );
-  target_write_u32( target, 0xFFFFFFDC, glbctrl );
-
-  return result;
+       target_t *target = bank->target;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+       int sector, result = ERROR_OK;
+       u32 fmmac2, fmbac2, glbctrl, orig_fmregopt;
+       static u8 buffer[64 * 1024];
+
+       if (!tms470_info->device_ident_reg)
+       {
+               tms470_read_part_info(bank);
+       }
+
+       /* set GLBCTRL.4  */
+       target_read_u32(target, 0xFFFFFFDC, &glbctrl);
+       target_write_u32(target, 0xFFFFFFDC, glbctrl | 0x10);
+
+       /* save current access mode, force normal read mode */
+       target_read_u32(target, 0xFFE89C00, &orig_fmregopt);
+       target_write_u32(target, 0xFFE89C00, 0x00);
+
+       /* enable the appropriate bank */
+       target_read_u32(target, 0xFFE8BC04, &fmmac2);
+       target_write_u32(target, 0xFFE8BC04, (fmmac2 & ~7) | tms470_info->ordinal);
+
+       /* TCR=0 */
+       target_write_u32(target, 0xFFE8BC10, 0x2fc0);
+
+       /* clear TEZ in fmbrdy */
+       target_write_u32(target, 0xFFE88010, 0x0b);
+
+       /* save current wait states, force max */
+       target_read_u32(target, 0xFFE88004, &fmbac2);
+       target_write_u32(target, 0xFFE88004, fmbac2 | 0xff);
+
+       /* 
+        * The TI primitives inspect the flash memory by reading one 32-bit
+        * word at a time.  Here we read an entire sector and inspect it in
+        * an attempt to reduce the JTAG overhead.
+        */
+       for (sector = 0; sector < bank->num_sectors; sector++)
+       {
+               if (bank->sectors[sector].is_erased != 1)
+               {
+                       u32 i, addr = bank->base + bank->sectors[sector].offset;
+
+                       INFO("checking flash bank %d sector %d", tms470_info->ordinal, sector);
+
+                       target_read_buffer(target, addr, bank->sectors[sector].size, buffer);
+
+                       bank->sectors[sector].is_erased = 1;
+                       for (i = 0; i < bank->sectors[sector].size; i++)
+                       {
+                               if (buffer[i] != 0xff)
+                               {
+                                       WARNING("tms470 bank %d, sector %d, not erased.", tms470_info->ordinal, sector);
+                                       WARNING("at location 0x%08x: flash data is 0x%02x.", addr + i, buffer[i]);
+
+                                       bank->sectors[sector].is_erased = 0;
+                                       break;
+                               }
+                       }
+               }
+               if (bank->sectors[sector].is_erased != 1)
+               {
+                       result = ERROR_FLASH_SECTOR_NOT_ERASED;
+                       break;
+               }
+               else
+               {
+                       INFO("sector erased");
+               }
+       }
+
+       /* reset TEZ, wait states, read mode, GLBCTRL.4 */
+       target_write_u32(target, 0xFFE88010, 0x0f);
+       target_write_u32(target, 0xFFE88004, fmbac2);
+       target_write_u32(target, 0xFFE89C00, orig_fmregopt);
+       target_write_u32(target, 0xFFFFFFDC, glbctrl);
+
+       return result;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_protect_check( struct flash_bank_s * bank )
+int tms470_protect_check(struct flash_bank_s *bank)
 {
-  target_t *target = bank->target;
-  tms470_flash_bank_t * tms470_info = bank->driver_priv;
-  int sector, result = ERROR_OK;
-  u32 fmmac2, fmbsea, fmbseb;
-
-  if (!tms470_info->device_ident_reg)
-    {
-      tms470_read_part_info( bank );
-    }
-
-  /* enable the appropriate bank */
-  target_read_u32( target, 0xFFE8BC04, &fmmac2 );
-  target_write_u32( target, 0xFFE8BC04, 
-                    (fmmac2 & ~7) | tms470_info->ordinal );
-
-  target_read_u32( target, 0xFFE88008, &fmbsea );
-  target_read_u32( target, 0xFFE8800C, &fmbseb );
-  
-  for( sector=0; sector<bank->num_sectors; sector++ )
-    {
-      int protected;
-
-      if (sector < 16)
-        {
-          protected = fmbsea & (1<<sector) ? 0 : 1;
-          bank->sectors[sector].is_protected = protected;
-        }
-      else
-        {
-          protected = fmbseb & (1<<(sector-16)) ? 0 : 1;
-          bank->sectors[sector].is_protected = protected;
-        }
-
-      DEBUG( "bank %d sector %d is %s",
-             tms470_info->ordinal,
-             sector,
-             protected ? "protected" : "not protected" );
-    }
-
-  return result;
+       target_t *target = bank->target;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+       int sector, result = ERROR_OK;
+       u32 fmmac2, fmbsea, fmbseb;
+
+       if (!tms470_info->device_ident_reg)
+       {
+               tms470_read_part_info(bank);
+       }
+
+       /* enable the appropriate bank */
+       target_read_u32(target, 0xFFE8BC04, &fmmac2);
+       target_write_u32(target, 0xFFE8BC04, (fmmac2 & ~7) | tms470_info->ordinal);
+
+       target_read_u32(target, 0xFFE88008, &fmbsea);
+       target_read_u32(target, 0xFFE8800C, &fmbseb);
+
+       for (sector = 0; sector < bank->num_sectors; sector++)
+       {
+               int protected;
+
+               if (sector < 16)
+               {
+                       protected = fmbsea & (1 << sector) ? 0 : 1;
+                       bank->sectors[sector].is_protected = protected;
+               }
+               else
+               {
+                       protected = fmbseb & (1 << (sector - 16)) ? 0 : 1;
+                       bank->sectors[sector].is_protected = protected;
+               }
+
+               DEBUG("bank %d sector %d is %s", tms470_info->ordinal, sector, protected ? "protected" : "not protected");
+       }
+
+       return result;
 }
 
 /* ---------------------------------------------------------------------- */
 
-int 
-tms470_info( struct flash_bank_s * bank, 
-             char *                buf, 
-             int                   buf_size )
+int tms470_info(struct flash_bank_s *bank, char *buf, int buf_size)
 {
-  int used = 0;
-  tms470_flash_bank_t * tms470_info = bank->driver_priv;
-
-  if (!tms470_info->device_ident_reg)
-    {
-      tms470_read_part_info( bank );
-    }
-
-  if (!tms470_info->device_ident_reg)
-    {
-      (void) snprintf(buf, buf_size, "Cannot identify target as a TMS470\n");
-      return ERROR_FLASH_OPERATION_FAILED;
-    }
-
-  used += snprintf( buf, buf_size, 
-                    "\ntms470 information: Chip is %s\n",
-                    tms470_info->part_name );
-  buf += used;
-  buf_size -= used;
-
-  used += snprintf( buf, buf_size,
-                    "Flash protection level 2 is %s\n",
-                    tms470_check_flash_unlocked( bank->target ) == ERROR_OK ? "disabled" : "enabled" );
-  buf += used;
-  buf_size -= used;
-
-  return ERROR_OK;
+       int used = 0;
+       tms470_flash_bank_t *tms470_info = bank->driver_priv;
+
+       if (!tms470_info->device_ident_reg)
+       {
+               tms470_read_part_info(bank);
+       }
+
+       if (!tms470_info->device_ident_reg)
+       {
+               (void)snprintf(buf, buf_size, "Cannot identify target as a TMS470\n");
+               return ERROR_FLASH_OPERATION_FAILED;
+       }
+
+       used += snprintf(buf, buf_size, "\ntms470 information: Chip is %s\n", tms470_info->part_name);
+       buf += used;
+       buf_size -= used;
+
+       used += snprintf(buf, buf_size, "Flash protection level 2 is %s\n", tms470_check_flash_unlocked(bank->target) == ERROR_OK ? "disabled" : "enabled");
+       buf += used;
+       buf_size -= used;
+
+       return ERROR_OK;
 }
 
 /* ---------------------------------------------------------------------- */
@@ -1286,21 +1144,16 @@ tms470_info( struct flash_bank_s * bank,
  * [options...]
  */
 
-int 
-tms470_flash_bank_command( struct command_context_s *cmd_ctx, 
-                           char *cmd, 
-                           char **args, 
-                           int argc, 
-                           struct flash_bank_s *bank )
+int tms470_flash_bank_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct flash_bank_s *bank)
 {
-  bank->driver_priv = malloc( sizeof( tms470_flash_bank_t ) );
+       bank->driver_priv = malloc(sizeof(tms470_flash_bank_t));
 
-  if (!bank->driver_priv)
-    {
-      return ERROR_FLASH_OPERATION_FAILED;
-    }
+       if (!bank->driver_priv)
+       {
+               return ERROR_FLASH_OPERATION_FAILED;
+       }
 
-  (void) memset( bank->driver_priv, 0, sizeof( tms470_flash_bank_t ) );
+       (void)memset(bank->driver_priv, 0, sizeof(tms470_flash_bank_t));
 
-  return ERROR_OK;
+       return ERROR_OK;
 }
index c17a60fa450f41c645767691a585b1ab568c5801..18f5cccd0d6068f0ff70d23ca550761a438aa95b 100644 (file)
 
 typedef struct tms470_flash_bank_s
 {
-  unsigned ordinal;
+       unsigned ordinal;
 
-  /* device identification register */
-  u32 device_ident_reg;
-  u32 silicon_version;
-  u32 technology_family;
-  u32 rom_flash;
-  u32 part_number;
-  char * part_name;
-  
-  int probed;
+       /* device identification register */
+       u32 device_ident_reg;
+       u32 silicon_version;
+       u32 technology_family;
+       u32 rom_flash;
+       u32 part_number;
+       char * part_name;
+
+       int probed;
 } tms470_flash_bank_t;
 
 #endif /* TMS470_DOT_H */