Remove whitespace that occurs after '('.

- Replace '([ \t]*' with '('.

git-svn-id: svn://svn.berlios.de/openocd/trunk@2395 b42882b7-edfa-0310-969c-e2dbd0fdcd60

diff --git a/src/flash/at91sam3.c b/src/flash/at91sam3.c
index b9f7933a78be486185dc4f550e68d478d796aa5b..a09784d14c15c5c03c11a59d39b1c2447ae2f543 100644 (file)
--- a/src/flash/at91sam3.c
+++ b/src/flash/at91sam3.c
@@ -100,7 +100,7 @@
 
 
 static float
-_tomhz( uint32_t freq_hz )
+_tomhz(uint32_t freq_hz )
 {
        float f;
 
@@ -224,7 +224,7 @@ struct sam3_chip {
 
 struct sam3_reg_list {
        uint32_t address;  size_t struct_offset; const char *name;
-       void (*explain_func)( struct sam3_chip *pInfo );
+       void (*explain_func)(struct sam3_chip *pInfo );
 };
 
 
@@ -236,27 +236,27 @@ get_current_sam3(struct command_context_s *cmd_ctx )
        target_t *t;
        static struct sam3_chip *p;
 
-       t = get_current_target( cmd_ctx );
-       if ( !t ){
-               command_print( cmd_ctx, "No current target?");
+       t = get_current_target(cmd_ctx );
+       if (!t ){
+               command_print(cmd_ctx, "No current target?");
                return NULL;
        }
 
        p = all_sam3_chips;
-       if ( !p ){
+       if (!p ){
                // this should not happen
                // the command is not registered until the chip is created?
-               command_print( cmd_ctx, "No SAM3 chips exist?");
+               command_print(cmd_ctx, "No SAM3 chips exist?");
                return NULL;
        }
 
-       while ( p ){
-               if ( p->target == t ){
+       while (p ){
+               if (p->target == t ){
                        return p;
                }
                p = p->next;
        }
-       command_print( cmd_ctx, "Cannot find SAM3 chip?");
+       command_print(cmd_ctx, "Cannot find SAM3 chip?");
        return NULL;
 }
 
@@ -540,10 +540,10 @@ static const struct sam3_chip_details all_sam3_details[] = {
  * @param v        - result goes here
  */
 static int 
-EFC_GetStatus( struct sam3_bank_private *pPrivate, uint32_t *v )
+EFC_GetStatus(struct sam3_bank_private *pPrivate, uint32_t *v )
 {
        int r;
-       r = target_read_u32( pPrivate->pChip->target, pPrivate->controller_address + offset_EFC_FSR, v );
+       r = target_read_u32(pPrivate->pChip->target, pPrivate->controller_address + offset_EFC_FSR, v );
        LOG_DEBUG("Status: 0x%08x (lockerror: %d, cmderror: %d, ready: %d)", 
                          (unsigned int)(*v),
                          ((unsigned int)((*v >> 2) & 1)),
@@ -558,12 +558,12 @@ EFC_GetStatus( struct sam3_bank_private *pPrivate, uint32_t *v )
  * @param v        - result goes here
  */
 static int 
-EFC_GetResult( struct sam3_bank_private *pPrivate, uint32_t *v )
+EFC_GetResult(struct sam3_bank_private *pPrivate, uint32_t *v )
 {
        int r;
        uint32_t rv;
-       r = target_read_u32( pPrivate->pChip->target, pPrivate->controller_address + offset_EFC_FRR, &rv );
-       if ( v ){
+       r = target_read_u32(pPrivate->pChip->target, pPrivate->controller_address + offset_EFC_FRR, &rv );
+       if (v ){
                *v = rv;
        }
        LOG_DEBUG("Result: 0x%08x", ((unsigned int)(rv)));
@@ -593,14 +593,14 @@ EFC_StartCommand(struct sam3_bank_private *pPrivate,
        case AT91C_EFC_FCMD_SLB:
        case AT91C_EFC_FCMD_CLB:
                n = (pPrivate->size_bytes / pPrivate->page_size);
-               if ( argument >= n ){
+               if (argument >= n ){
                        LOG_ERROR("*BUG*: Embedded flash has only %u pages", (unsigned)(n));
                }
                break;
                
        case AT91C_EFC_FCMD_SFB:
        case AT91C_EFC_FCMD_CFB:
-               if ( argument >= pPrivate->pChip->details.n_gpnvms ){
+               if (argument >= pPrivate->pChip->details.n_gpnvms ){
                        LOG_ERROR("*BUG*: Embedded flash has only %d GPNVMs", 
                                          pPrivate->pChip->details.n_gpnvms );
                }
@@ -612,7 +612,7 @@ EFC_StartCommand(struct sam3_bank_private *pPrivate,
        case AT91C_EFC_FCMD_GFB:
        case AT91C_EFC_FCMD_STUI:
        case AT91C_EFC_FCMD_SPUI:
-               if ( argument != 0 ){
+               if (argument != 0 ){
                        LOG_ERROR("Argument is meaningless for cmd: %d", command );
                }
                break;
@@ -621,19 +621,19 @@ EFC_StartCommand(struct sam3_bank_private *pPrivate,
                break;
     }
 
-       if ( command == AT91C_EFC_FCMD_SPUI ){
+       if (command == AT91C_EFC_FCMD_SPUI ){
                // this is a very special situation.
                // Situation (1) - error/retry - see below
                //      And we are being called recursively
                // Situation (2) - normal, finished reading unique id
        } else {
                // it should be "ready"
-               EFC_GetStatus( pPrivate, &v );
-               if ( v & 1 ){
+               EFC_GetStatus(pPrivate, &v );
+               if (v & 1 ){
                        // then it is ready
                        // we go on
                } else {
-                       if ( retry ){
+                       if (retry ){
                                // we have done this before
                                // the controller is not responding.
                                LOG_ERROR("flash controller(%d) is not ready! Error", pPrivate->bank_number );
@@ -643,9 +643,9 @@ EFC_StartCommand(struct sam3_bank_private *pPrivate,
                                LOG_ERROR("Flash controller(%d) is not ready, attempting reset", 
                                                  pPrivate->bank_number );
                                // we do that by issuing the *STOP* command
-                               EFC_StartCommand( pPrivate, AT91C_EFC_FCMD_SPUI, 0 );
+                               EFC_StartCommand(pPrivate, AT91C_EFC_FCMD_SPUI, 0 );
                                // above is recursive, and further recursion is blocked by 
-                               // if ( command == AT91C_EFC_FCMD_SPUI ) above
+                               // if (command == AT91C_EFC_FCMD_SPUI ) above
                                goto do_retry;
                        }
                }
@@ -653,10 +653,10 @@ EFC_StartCommand(struct sam3_bank_private *pPrivate,
 
        v = (0x5A << 24) | (argument << 8) | command;
        LOG_DEBUG("Command: 0x%08x", ((unsigned int)(v)) );
-       r = target_write_u32( pPrivate->pBank->target, 
+       r = target_write_u32(pPrivate->pBank->target, 
                                                  pPrivate->controller_address + offset_EFC_FCR,
                                                  v);
-       if ( r != ERROR_OK ){
+       if (r != ERROR_OK ){
                LOG_DEBUG("Error Write failed");
        }
        return r;
@@ -670,7 +670,7 @@ EFC_StartCommand(struct sam3_bank_private *pPrivate,
  * @param status   - put command status bits here
  */
 static int 
-EFC_PerformCommand( struct sam3_bank_private *pPrivate, 
+EFC_PerformCommand(struct sam3_bank_private *pPrivate, 
                                        unsigned command, 
                                        unsigned argument, 
                                        uint32_t *status)
@@ -681,12 +681,12 @@ EFC_PerformCommand( struct sam3_bank_private *pPrivate,
        long long ms_now, ms_end;
 
        // default 
-       if ( status ){
+       if (status ){
                *status = 0;
        }
 
-       r = EFC_StartCommand( pPrivate, command, argument );
-       if ( r != ERROR_OK ){
+       r = EFC_StartCommand(pPrivate, command, argument );
+       if (r != ERROR_OK ){
                return r;
        }
 
@@ -694,22 +694,22 @@ EFC_PerformCommand( struct sam3_bank_private *pPrivate,
 
 
     do {
-               r = EFC_GetStatus( pPrivate, &v );
-               if ( r != ERROR_OK ){
+               r = EFC_GetStatus(pPrivate, &v );
+               if (r != ERROR_OK ){
                        return r;
                }
                ms_now = timeval_ms();
-               if ( ms_now > ms_end ){
+               if (ms_now > ms_end ){
                        // error
                        LOG_ERROR("Command timeout");
                        return ERROR_FAIL;
                }
     }
-    while ( (v & 1) == 0 )
+    while ((v & 1) == 0 )
                ;
 
        // error bits..
-       if ( status ){
+       if (status ){
                *status = (v & 0x6);
        }
        return ERROR_OK;
@@ -728,7 +728,7 @@ EFC_PerformCommand( struct sam3_bank_private *pPrivate,
  */
 
 static int
-FLASHD_ReadUniqueID ( struct sam3_bank_private *pPrivate )
+FLASHD_ReadUniqueID (struct sam3_bank_private *pPrivate )
 {
        int r;
        uint32_t v;
@@ -740,22 +740,22 @@ FLASHD_ReadUniqueID ( struct sam3_bank_private *pPrivate )
     pPrivate->pChip->cfg.unique_id[3] = 0;
 
        LOG_DEBUG("Begin");
-       r = EFC_StartCommand( pPrivate, AT91C_EFC_FCMD_STUI, 0);
-       if ( r < 0 ){
+       r = EFC_StartCommand(pPrivate, AT91C_EFC_FCMD_STUI, 0);
+       if (r < 0 ){
                return r;
        }
 
-       for ( x = 0 ; x < 4 ; x++ ){
-               r = target_read_u32( pPrivate->pChip->target, 
+       for (x = 0 ; x < 4 ; x++ ){
+               r = target_read_u32(pPrivate->pChip->target, 
                                                         pPrivate->pBank->base + (x * 4),
                                                         &v );
-               if ( r < 0 ){
+               if (r < 0 ){
                        return r;
                }
                pPrivate->pChip->cfg.unique_id[x] = v;
        }
 
-    r = EFC_PerformCommand( pPrivate, AT91C_EFC_FCMD_SPUI, 0, NULL );
+    r = EFC_PerformCommand(pPrivate, AT91C_EFC_FCMD_SPUI, 0, NULL );
        LOG_DEBUG("End: R=%d, id = 0x%08x, 0x%08x, 0x%08x, 0x%08x",
                          r, 
                          (unsigned int)(pPrivate->pChip->cfg.unique_id[0]),
@@ -770,10 +770,10 @@ FLASHD_ReadUniqueID ( struct sam3_bank_private *pPrivate )
  * @param pPrivate - the info about the bank.
  */
 static int
-FLASHD_EraseEntireBank( struct sam3_bank_private *pPrivate )
+FLASHD_EraseEntireBank(struct sam3_bank_private *pPrivate )
 {
        LOG_DEBUG("Here");
-       return EFC_PerformCommand( pPrivate, AT91C_EFC_FCMD_EA, 0, NULL );
+       return EFC_PerformCommand(pPrivate, AT91C_EFC_FCMD_EA, 0, NULL );
 }
 
 
@@ -786,33 +786,33 @@ FLASHD_EraseEntireBank( struct sam3_bank_private *pPrivate )
  */
 //------------------------------------------------------------------------------
 static int 
-FLASHD_GetGPNVM( struct sam3_bank_private *pPrivate, unsigned gpnvm, unsigned *puthere)
+FLASHD_GetGPNVM(struct sam3_bank_private *pPrivate, unsigned gpnvm, unsigned *puthere)
 {
        uint32_t v;
        int r;
 
        LOG_DEBUG("Here");
-       if ( pPrivate->bank_number != 0 ){
+       if (pPrivate->bank_number != 0 ){
                LOG_ERROR("GPNVM only works with Bank0");
                return ERROR_FAIL;
        }
 
-       if ( gpnvm >= pPrivate->pChip->details.n_gpnvms ){
+       if (gpnvm >= pPrivate->pChip->details.n_gpnvms ){
                LOG_ERROR("Invalid GPNVM %d, max: %d, ignored",
                                  gpnvm,pPrivate->pChip->details.n_gpnvms );
                return ERROR_FAIL;
        }
 
     // Get GPNVMs status
-       r = EFC_PerformCommand( pPrivate, AT91C_EFC_FCMD_GFB, 0, NULL );
-       if ( r != ERROR_OK ){
+       r = EFC_PerformCommand(pPrivate, AT91C_EFC_FCMD_GFB, 0, NULL );
+       if (r != ERROR_OK ){
                LOG_ERROR("Failed");
                return r;
        }
 
     r = EFC_GetResult(pPrivate, &v );
 
-       if ( puthere ){
+       if (puthere ){
                // Check if GPNVM is set
                // get the bit and make it a 0/1 
                *puthere = (v >> gpnvm) & 1;
@@ -830,29 +830,29 @@ FLASHD_GetGPNVM( struct sam3_bank_private *pPrivate, unsigned gpnvm, unsigned *p
  * Returns 0 if successful; otherwise returns an error code.
  */
 static int 
-FLASHD_ClrGPNVM( struct sam3_bank_private *pPrivate, unsigned gpnvm)
+FLASHD_ClrGPNVM(struct sam3_bank_private *pPrivate, unsigned gpnvm)
 {
        int r;
        unsigned v;
 
        LOG_DEBUG("Here");
-       if ( pPrivate->bank_number != 0 ){
+       if (pPrivate->bank_number != 0 ){
                LOG_ERROR("GPNVM only works with Bank0");
                return ERROR_FAIL;
        }
 
-       if ( gpnvm >= pPrivate->pChip->details.n_gpnvms ){
+       if (gpnvm >= pPrivate->pChip->details.n_gpnvms ){
                LOG_ERROR("Invalid GPNVM %d, max: %d, ignored",
                                  gpnvm,pPrivate->pChip->details.n_gpnvms );
                return ERROR_FAIL;
        }
 
-       r = FLASHD_GetGPNVM( pPrivate, gpnvm, &v );
-       if ( r != ERROR_OK ){
+       r = FLASHD_GetGPNVM(pPrivate, gpnvm, &v );
+       if (r != ERROR_OK ){
                LOG_DEBUG("Failed: %d",r);
                return r;
        }
-       r = EFC_PerformCommand( pPrivate, AT91C_EFC_FCMD_CFB, gpnvm, NULL);
+       r = EFC_PerformCommand(pPrivate, AT91C_EFC_FCMD_CFB, gpnvm, NULL);
        LOG_DEBUG("End: %d",r);
        return r;
 }
@@ -864,32 +864,32 @@ FLASHD_ClrGPNVM( struct sam3_bank_private *pPrivate, unsigned gpnvm)
  *
  */
 static int 
-FLASHD_SetGPNVM( struct sam3_bank_private *pPrivate, unsigned gpnvm)
+FLASHD_SetGPNVM(struct sam3_bank_private *pPrivate, unsigned gpnvm)
 {
        int r;
        unsigned v;
 
-       if ( pPrivate->bank_number != 0 ){
+       if (pPrivate->bank_number != 0 ){
                LOG_ERROR("GPNVM only works with Bank0");
                return ERROR_FAIL;
        }
 
-       if ( gpnvm >= pPrivate->pChip->details.n_gpnvms ){
+       if (gpnvm >= pPrivate->pChip->details.n_gpnvms ){
                LOG_ERROR("Invalid GPNVM %d, max: %d, ignored",
                                  gpnvm,pPrivate->pChip->details.n_gpnvms );
                return ERROR_FAIL;
        }
 
-       r = FLASHD_GetGPNVM( pPrivate, gpnvm, &v );
-       if ( r != ERROR_OK ){
+       r = FLASHD_GetGPNVM(pPrivate, gpnvm, &v );
+       if (r != ERROR_OK ){
                return r;
        }
-       if ( v ){
+       if (v ){
                // already set
                r = ERROR_OK;
        } else {
                // set it
-               r = EFC_PerformCommand( pPrivate, AT91C_EFC_FCMD_SFB, gpnvm, NULL );
+               r = EFC_PerformCommand(pPrivate, AT91C_EFC_FCMD_SFB, gpnvm, NULL );
        }
        return r;
 }
@@ -906,8 +906,8 @@ FLASHD_GetLockBits(struct sam3_bank_private *pPrivate, uint32_t *v )
 {
        int r;
        LOG_DEBUG("Here");
-    r = EFC_PerformCommand( pPrivate, AT91C_EFC_FCMD_GLB, 0, NULL);
-       if ( r == ERROR_OK ){
+    r = EFC_PerformCommand(pPrivate, AT91C_EFC_FCMD_GLB, 0, NULL);
+       if (r == ERROR_OK ){
                r = EFC_GetResult(pPrivate, v );
        }
        LOG_DEBUG("End: %d",r);
@@ -922,7 +922,7 @@ FLASHD_GetLockBits(struct sam3_bank_private *pPrivate, uint32_t *v )
  */
 
 static int
-FLASHD_Unlock( struct sam3_bank_private *pPrivate,
+FLASHD_Unlock(struct sam3_bank_private *pPrivate,
                           unsigned start_sector,
                           unsigned end_sector )
 {
@@ -956,7 +956,7 @@ FLASHD_Unlock( struct sam3_bank_private *pPrivate,
 
 
 static int 
-FLASHD_Lock( struct sam3_bank_private *pPrivate,
+FLASHD_Lock(struct sam3_bank_private *pPrivate,
                         unsigned start_sector,
                         unsigned end_sector )
 {
@@ -986,21 +986,21 @@ FLASHD_Lock( struct sam3_bank_private *pPrivate,
 /* begin helpful debug code */
 
 static void
-sam3_sprintf( struct sam3_chip *pChip , const char *fmt, ... )
+sam3_sprintf(struct sam3_chip *pChip , const char *fmt, ... )
 {
        va_list ap;
        va_start(ap,fmt);
-       if ( pChip->mbuf == NULL ){
+       if (pChip->mbuf == NULL ){
                return;
        }
 
-       membuf_vsprintf( pChip->mbuf, fmt, ap );
+       membuf_vsprintf(pChip->mbuf, fmt, ap );
        va_end(ap);
 }
 
 // print the fieldname, the field value, in dec & hex, and return field value
 static uint32_t 
-sam3_reg_fieldname( struct sam3_chip *pChip, 
+sam3_reg_fieldname(struct sam3_chip *pChip, 
                                        const char *regname,
                                        uint32_t value,
                                        unsigned shift,
@@ -1013,7 +1013,7 @@ sam3_reg_fieldname( struct sam3_chip *pChip,
        // extract the field
        v = value >> shift;
        v = v & ((1 << width)-1);
-       if ( width <= 16 ){
+       if (width <= 16 ){
                hwidth = 4;
                dwidth = 5;
        } else {
@@ -1022,7 +1022,7 @@ sam3_reg_fieldname( struct sam3_chip *pChip,
        }
        
        // show the basics
-       sam3_sprintf( pChip, "\t%*s: %*d [0x%0*x] ", 
+       sam3_sprintf(pChip, "\t%*s: %*d [0x%0*x] ", 
                                  REG_NAME_WIDTH, regname,
                                  dwidth, v,
                                  hwidth, v  );
@@ -1137,9 +1137,9 @@ static const char * const nvptype[] = {
        
 };
 
-static const char *_yes_or_no( uint32_t v )
+static const char *_yes_or_no(uint32_t v )
 {
-       if ( v ){
+       if (v ){
                return "YES";
        } else {
                return "NO";
@@ -1151,27 +1151,27 @@ static const char * const _rc_freq[] = {
 };
 
 static void 
-sam3_explain_ckgr_mor(  struct sam3_chip *pChip )
+sam3_explain_ckgr_mor(struct sam3_chip *pChip )
 {
        uint32_t v;
        uint32_t rcen;
 
        v = sam3_reg_fieldname(pChip, "MOSCXTEN", pChip->cfg.CKGR_MOR, 0, 1 );
-       sam3_sprintf( pChip, "(main xtal enabled: %s)\n",
+       sam3_sprintf(pChip, "(main xtal enabled: %s)\n",
                                  _yes_or_no(v) );
        v = sam3_reg_fieldname(pChip, "MOSCXTBY", pChip->cfg.CKGR_MOR, 1, 1 ); 
-       sam3_sprintf( pChip, "(main osc bypass: %s)\n",
+       sam3_sprintf(pChip, "(main osc bypass: %s)\n",
                                  _yes_or_no(v) );
        rcen = sam3_reg_fieldname(pChip, "MOSCRCEN", pChip->cfg.CKGR_MOR, 2, 1 );
-       sam3_sprintf( pChip, "(onchip RC-OSC enabled: %s)\n",
+       sam3_sprintf(pChip, "(onchip RC-OSC enabled: %s)\n",
                                  _yes_or_no(rcen) );
        v = sam3_reg_fieldname(pChip, "MOSCRCF", pChip->cfg.CKGR_MOR, 4, 3 );
-       sam3_sprintf( pChip, "(onchip RC-OSC freq: %s)\n",
+       sam3_sprintf(pChip, "(onchip RC-OSC freq: %s)\n",
                                  _rc_freq[v] );
 
        pChip->cfg.rc_freq = 0;
-       if ( rcen ){
-               switch ( v ){
+       if (rcen ){
+               switch (v ){
                default:
                        pChip->cfg.rc_freq = 0;
                case 0:
@@ -1187,10 +1187,10 @@ sam3_explain_ckgr_mor(  struct sam3_chip *pChip )
        }
 
        v = sam3_reg_fieldname(pChip,"MOSCXTST", pChip->cfg.CKGR_MOR, 8, 8 );
-       sam3_sprintf( pChip, "(startup clks, time= %f uSecs)\n",
+       sam3_sprintf(pChip, "(startup clks, time= %f uSecs)\n",
                                  ((float)(v * 1000000)) / ((float)(pChip->cfg.slow_freq)));
        v = sam3_reg_fieldname(pChip, "MOSCSEL", pChip->cfg.CKGR_MOR, 24, 1 );
-       sam3_sprintf( pChip, "(mainosc source: %s)\n",
+       sam3_sprintf(pChip, "(mainosc source: %s)\n",
                                  v ? "external xtal" : "internal RC");
        
        v = sam3_reg_fieldname(pChip,"CFDEN", pChip->cfg.CKGR_MOR, 25, 1 );
@@ -1201,97 +1201,97 @@ sam3_explain_ckgr_mor(  struct sam3_chip *pChip )
        
 
 static void 
-sam3_explain_chipid_cidr( struct sam3_chip *pChip )
+sam3_explain_chipid_cidr(struct sam3_chip *pChip )
 {
        int x;
        uint32_t v;
        const char *cp;
 
-       sam3_reg_fieldname( pChip, "Version", pChip->cfg.CHIPID_CIDR, 0, 5 );
+       sam3_reg_fieldname(pChip, "Version", pChip->cfg.CHIPID_CIDR, 0, 5 );
        sam3_sprintf(pChip,"\n");
 
-       v = sam3_reg_fieldname( pChip, "EPROC", pChip->cfg.CHIPID_CIDR, 5, 3 );
-       sam3_sprintf( pChip, "%s\n", eproc_names[v]);
+       v = sam3_reg_fieldname(pChip, "EPROC", pChip->cfg.CHIPID_CIDR, 5, 3 );
+       sam3_sprintf(pChip, "%s\n", eproc_names[v]);
        
-       v = sam3_reg_fieldname( pChip, "NVPSIZE", pChip->cfg.CHIPID_CIDR, 8, 4 );
-       sam3_sprintf( pChip, "%s\n", nvpsize[v]);
+       v = sam3_reg_fieldname(pChip, "NVPSIZE", pChip->cfg.CHIPID_CIDR, 8, 4 );
+       sam3_sprintf(pChip, "%s\n", nvpsize[v]);
 
-       v = sam3_reg_fieldname( pChip, "NVPSIZE2", pChip->cfg.CHIPID_CIDR, 12, 4 );
-       sam3_sprintf( pChip, "%s\n", nvpsize2[v]);
+       v = sam3_reg_fieldname(pChip, "NVPSIZE2", pChip->cfg.CHIPID_CIDR, 12, 4 );
+       sam3_sprintf(pChip, "%s\n", nvpsize2[v]);
 
-       v = sam3_reg_fieldname( pChip, "SRAMSIZE", pChip->cfg.CHIPID_CIDR, 16,4 );
-       sam3_sprintf( pChip, "%s\n", sramsize[ v ] );
+       v = sam3_reg_fieldname(pChip, "SRAMSIZE", pChip->cfg.CHIPID_CIDR, 16,4 );
+       sam3_sprintf(pChip, "%s\n", sramsize[ v ] );
 
-       v = sam3_reg_fieldname( pChip, "ARCH", pChip->cfg.CHIPID_CIDR, 20, 8 );
+       v = sam3_reg_fieldname(pChip, "ARCH", pChip->cfg.CHIPID_CIDR, 20, 8 );
        cp = _unknown;
-       for ( x = 0 ; archnames[x].name ; x++ ){
-               if ( v == archnames[x].value ){
+       for (x = 0 ; archnames[x].name ; x++ ){
+               if (v == archnames[x].value ){
                        cp = archnames[x].name;
                        break;
                }
        }
        
-       sam3_sprintf( pChip, "%s\n", cp );
+       sam3_sprintf(pChip, "%s\n", cp );
 
-       v = sam3_reg_fieldname( pChip, "NVPTYP", pChip->cfg.CHIPID_CIDR, 28, 3 );
-       sam3_sprintf( pChip, "%s\n", nvptype[ v ] );
+       v = sam3_reg_fieldname(pChip, "NVPTYP", pChip->cfg.CHIPID_CIDR, 28, 3 );
+       sam3_sprintf(pChip, "%s\n", nvptype[ v ] );
 
-       v = sam3_reg_fieldname( pChip, "EXTID", pChip->cfg.CHIPID_CIDR, 31, 1 );
+       v = sam3_reg_fieldname(pChip, "EXTID", pChip->cfg.CHIPID_CIDR, 31, 1 );
        sam3_sprintf(pChip, "(exists: %s)\n", _yes_or_no(v));
 }
 
 static void 
-sam3_explain_ckgr_mcfr( struct sam3_chip *pChip )
+sam3_explain_ckgr_mcfr(struct sam3_chip *pChip )
 {
        uint32_t v;
 
        
-       v = sam3_reg_fieldname( pChip, "MAINFRDY", pChip->cfg.CKGR_MCFR, 16, 1 );
-       sam3_sprintf( pChip, "(main ready: %s)\n", _yes_or_no( v ));
+       v = sam3_reg_fieldname(pChip, "MAINFRDY", pChip->cfg.CKGR_MCFR, 16, 1 );
+       sam3_sprintf(pChip, "(main ready: %s)\n", _yes_or_no(v ));
 
-       v = sam3_reg_fieldname( pChip, "MAINF", pChip->cfg.CKGR_MCFR, 0, 16 );
+       v = sam3_reg_fieldname(pChip, "MAINF", pChip->cfg.CKGR_MCFR, 0, 16 );
        
        v = (v * pChip->cfg.slow_freq) / 16;
        pChip->cfg.mainosc_freq = v;
 
        sam3_sprintf(pChip, "(%3.03f Mhz (%d.%03dkhz slowclk)\n",
-                                _tomhz( v ),
+                                _tomhz(v ),
                                 pChip->cfg.slow_freq / 1000,
                                 pChip->cfg.slow_freq % 1000);
 
 }
 
 static void
-sam3_explain_ckgr_plla( struct sam3_chip *pChip )
+sam3_explain_ckgr_plla(struct sam3_chip *pChip )
 {
        uint32_t mula,diva;
 
-       diva = sam3_reg_fieldname( pChip, "DIVA", pChip->cfg.CKGR_PLLAR, 0, 8 );
+       diva = sam3_reg_fieldname(pChip, "DIVA", pChip->cfg.CKGR_PLLAR, 0, 8 );
        sam3_sprintf(pChip,"\n");
-       mula = sam3_reg_fieldname( pChip, "MULA", pChip->cfg.CKGR_PLLAR, 16, 11 );
+       mula = sam3_reg_fieldname(pChip, "MULA", pChip->cfg.CKGR_PLLAR, 16, 11 );
        sam3_sprintf(pChip,"\n");
        pChip->cfg.plla_freq = 0;
-       if ( mula == 0 ){
+       if (mula == 0 ){
                sam3_sprintf(pChip,"\tPLLA Freq: (Disabled,mula = 0)\n");
-       } else if ( diva == 0 ){
+       } else if (diva == 0 ){
                sam3_sprintf(pChip,"\tPLLA Freq: (Disabled,diva = 0)\n");
-       } else if ( diva == 1 ){
+       } else if (diva == 1 ){
                pChip->cfg.plla_freq = (pChip->cfg.mainosc_freq * (mula + 1));
                sam3_sprintf(pChip,"\tPLLA Freq: %3.03f MHz\n",
-                                        _tomhz( pChip->cfg.plla_freq ));
+                                        _tomhz(pChip->cfg.plla_freq ));
        }
 }
                
                
 static void
-sam3_explain_mckr( struct sam3_chip *pChip )
+sam3_explain_mckr(struct sam3_chip *pChip )
 {
        uint32_t css, pres,fin;
        int pdiv;
        const char *cp;
 
-       css = sam3_reg_fieldname( pChip, "CSS", pChip->cfg.PMC_MCKR, 0, 2 );
-       switch ( css & 3 ){
+       css = sam3_reg_fieldname(pChip, "CSS", pChip->cfg.PMC_MCKR, 0, 2 );
+       switch (css & 3 ){
        case 0:
                fin = pChip->cfg.slow_freq;
                cp = "slowclk";
@@ -1305,7 +1305,7 @@ sam3_explain_mckr( struct sam3_chip *pChip )
                cp  = "plla";
                break;
        case 3:
-               if ( pChip->cfg.CKGR_UCKR & (1 << 16) ){
+               if (pChip->cfg.CKGR_UCKR & (1 << 16) ){
                        fin = 480 * 1000 * 1000;
                        cp = "upll";
                } else {
@@ -1315,11 +1315,11 @@ sam3_explain_mckr( struct sam3_chip *pChip )
                break;
        }
 
-       sam3_sprintf( pChip, "%s (%3.03f Mhz)\n",
+       sam3_sprintf(pChip, "%s (%3.03f Mhz)\n",
                                  cp, 
-                                 _tomhz( fin ) );
+                                 _tomhz(fin ) );
        pres = sam3_reg_fieldname(pChip, "PRES", pChip->cfg.PMC_MCKR, 4, 3 );
-       switch ( pres & 0x07 ){
+       switch (pres & 0x07 ){
        case 0:
                pdiv = 1;
                cp = "selected clock";
@@ -1352,20 +1352,20 @@ sam3_explain_mckr( struct sam3_chip *pChip )
                cp = "clock/6";
                break;
        }
-       sam3_sprintf( pChip, "(%s)\n", cp );
+       sam3_sprintf(pChip, "(%s)\n", cp );
        fin = fin / pdiv;
        // sam3 has a *SINGLE* clock - 
        // other at91 series parts have divisors for these.
        pChip->cfg.cpu_freq = fin;
        pChip->cfg.mclk_freq = fin;
        pChip->cfg.fclk_freq = fin;
-       sam3_sprintf( pChip, "\t\tResult CPU Freq: %3.03f\n",
-                                 _tomhz( fin ) );
+       sam3_sprintf(pChip, "\t\tResult CPU Freq: %3.03f\n",
+                                 _tomhz(fin ) );
 }
        
 #if 0
 static struct sam3_chip *
-target2sam3( target_t *pTarget )
+target2sam3(target_t *pTarget )
 {
        struct sam3_chip *pChip;
 
@@ -1386,7 +1386,7 @@ target2sam3( target_t *pTarget )
 #endif
 
 static uint32_t *
-sam3_get_reg_ptr( struct sam3_cfg *pCfg, const struct sam3_reg_list *pList )
+sam3_get_reg_ptr(struct sam3_cfg *pCfg, const struct sam3_reg_list *pList )
 {
        // this function exists to help 
        // keep funky offsetof() errors
@@ -1395,29 +1395,29 @@ sam3_get_reg_ptr( struct sam3_cfg *pCfg, const struct sam3_reg_list *pList )
        // By using prototypes - we can detect what would
        // be casting errors.
 
-       return ((uint32_t *)(  ((char *)(pCfg)) + pList->struct_offset ));
+       return ((uint32_t *)(((char *)(pCfg)) + pList->struct_offset ));
 }
        
        
-#define SAM3_ENTRY( NAME, FUNC )  { .address = SAM3_ ## NAME, .struct_offset = offsetof( struct sam3_cfg, NAME ), #NAME, FUNC }
+#define SAM3_ENTRY(NAME, FUNC )  { .address = SAM3_ ## NAME, .struct_offset = offsetof(struct sam3_cfg, NAME ), #NAME, FUNC }
 static const struct sam3_reg_list sam3_all_regs[] = {
-       SAM3_ENTRY( CKGR_MOR , sam3_explain_ckgr_mor ),
-       SAM3_ENTRY( CKGR_MCFR , sam3_explain_ckgr_mcfr ),
-       SAM3_ENTRY( CKGR_PLLAR , sam3_explain_ckgr_plla ),
-       SAM3_ENTRY( CKGR_UCKR , NULL ),
-       SAM3_ENTRY( PMC_FSMR , NULL ),
-       SAM3_ENTRY( PMC_FSPR , NULL ),
-       SAM3_ENTRY( PMC_IMR , NULL ),
-       SAM3_ENTRY( PMC_MCKR , sam3_explain_mckr ),
-       SAM3_ENTRY( PMC_PCK0 , NULL ),
-       SAM3_ENTRY( PMC_PCK1 , NULL ),
-       SAM3_ENTRY( PMC_PCK2 , NULL ),
-       SAM3_ENTRY( PMC_PCSR , NULL ),
-       SAM3_ENTRY( PMC_SCSR , NULL ),
-       SAM3_ENTRY( PMC_SR , NULL ),
-       SAM3_ENTRY( CHIPID_CIDR , sam3_explain_chipid_cidr ),
-       SAM3_ENTRY( CHIPID_EXID , NULL ),
-       SAM3_ENTRY( SUPC_CR, NULL ),
+       SAM3_ENTRY(CKGR_MOR , sam3_explain_ckgr_mor ),
+       SAM3_ENTRY(CKGR_MCFR , sam3_explain_ckgr_mcfr ),
+       SAM3_ENTRY(CKGR_PLLAR , sam3_explain_ckgr_plla ),
+       SAM3_ENTRY(CKGR_UCKR , NULL ),
+       SAM3_ENTRY(PMC_FSMR , NULL ),
+       SAM3_ENTRY(PMC_FSPR , NULL ),
+       SAM3_ENTRY(PMC_IMR , NULL ),
+       SAM3_ENTRY(PMC_MCKR , sam3_explain_mckr ),
+       SAM3_ENTRY(PMC_PCK0 , NULL ),
+       SAM3_ENTRY(PMC_PCK1 , NULL ),
+       SAM3_ENTRY(PMC_PCK2 , NULL ),
+       SAM3_ENTRY(PMC_PCSR , NULL ),
+       SAM3_ENTRY(PMC_SCSR , NULL ),
+       SAM3_ENTRY(PMC_SR , NULL ),
+       SAM3_ENTRY(CHIPID_CIDR , sam3_explain_chipid_cidr ),
+       SAM3_ENTRY(CHIPID_EXID , NULL ),
+       SAM3_ENTRY(SUPC_CR, NULL ),
 
        // TERMINATE THE LIST
        { .name = NULL }
@@ -1428,7 +1428,7 @@ static const struct sam3_reg_list sam3_all_regs[] = {
 
 
 static struct sam3_bank_private *
-get_sam3_bank_private( flash_bank_t *bank )
+get_sam3_bank_private(flash_bank_t *bank )
 {
        return (struct sam3_bank_private *)(bank->driver_priv);
 }
@@ -1438,7 +1438,7 @@ get_sam3_bank_private( flash_bank_t *bank )
  *    Determine the register name, address from the all registers table.
  */
 static const struct sam3_reg_list *
-sam3_GetReg( struct sam3_chip *pChip, uint32_t *goes_here )
+sam3_GetReg(struct sam3_chip *pChip, uint32_t *goes_here )
 {
        const struct sam3_reg_list *pReg;
 
@@ -1449,10 +1449,10 @@ sam3_GetReg( struct sam3_chip *pChip, uint32_t *goes_here )
                // calculate where this one go..
                // it is "possibly" this register.
                
-               pPossible = ((uint32_t *)( ((char *)(&(pChip->cfg))) + pReg->struct_offset ));
+               pPossible = ((uint32_t *)(((char *)(&(pChip->cfg))) + pReg->struct_offset ));
 
                // well? Is it this register
-               if ( pPossible == goes_here ){
+               if (pPossible == goes_here ){
                        // Jump for joy!
                        return pReg;
                }
@@ -1467,18 +1467,18 @@ sam3_GetReg( struct sam3_chip *pChip, uint32_t *goes_here )
 
 
 static int
-sam3_ReadThisReg( struct sam3_chip *pChip, uint32_t *goes_here )
+sam3_ReadThisReg(struct sam3_chip *pChip, uint32_t *goes_here )
 {
        const struct sam3_reg_list *pReg;
        int r;
 
-       pReg = sam3_GetReg( pChip, goes_here );
-       if ( !pReg ){
+       pReg = sam3_GetReg(pChip, goes_here );
+       if (!pReg ){
                return ERROR_FAIL;
        }
 
-       r = target_read_u32( pChip->target, pReg->address, goes_here );
-       if ( r != ERROR_OK ){
+       r = target_read_u32(pChip->target, pReg->address, goes_here );
+       if (r != ERROR_OK ){
                LOG_ERROR("Cannot read SAM3 register: %s @ 0x%08x, Err: %d\n",
                                  pReg->name, (unsigned)(pReg->address), r );
        }
@@ -1488,16 +1488,16 @@ sam3_ReadThisReg( struct sam3_chip *pChip, uint32_t *goes_here )
        
 
 static int
-sam3_ReadAllRegs( struct sam3_chip *pChip )
+sam3_ReadAllRegs(struct sam3_chip *pChip )
 {
        int r;
        const struct sam3_reg_list *pReg;
 
        pReg = &(sam3_all_regs[0]);
-       while ( pReg->name ){
-               r = sam3_ReadThisReg( pChip, 
-                                                                 sam3_get_reg_ptr( &(pChip->cfg), pReg ) );
-               if ( r != ERROR_OK ){
+       while (pReg->name ){
+               r = sam3_ReadThisReg(pChip, 
+                                                                 sam3_get_reg_ptr(&(pChip->cfg), pReg ) );
+               if (r != ERROR_OK ){
                        LOG_ERROR("Cannot read SAM3 registere: %s @ 0x%08x, Error: %d\n",
                                          pReg->name, ((unsigned)(pReg->address)), r );
                        return r;
@@ -1511,38 +1511,38 @@ sam3_ReadAllRegs( struct sam3_chip *pChip )
 
 
 static int
-sam3_GetInfo( struct sam3_chip *pChip )
+sam3_GetInfo(struct sam3_chip *pChip )
 {
        const struct sam3_reg_list *pReg;
        uint32_t regval;
 
-       membuf_reset( pChip->mbuf );
+       membuf_reset(pChip->mbuf );
 
 
        pReg = &(sam3_all_regs[0]);
        while (pReg->name){
                // display all regs
                LOG_DEBUG("Start: %s", pReg->name );
-               regval = *sam3_get_reg_ptr( &(pChip->cfg), pReg );
+               regval = *sam3_get_reg_ptr(&(pChip->cfg), pReg );
                sam3_sprintf(pChip, "%*s: [0x%08x] -> 0x%08x\n", 
                                         REG_NAME_WIDTH,
                                         pReg->name,
                                         pReg->address,
                                         regval );
-               if ( pReg->explain_func ){
-                       (*(pReg->explain_func))( pChip );
+               if (pReg->explain_func ){
+                       (*(pReg->explain_func))(pChip );
                }
                LOG_DEBUG("End: %s", pReg->name );
                pReg++;
        }
-       sam3_sprintf(pChip,"   rc-osc: %3.03f MHz\n", _tomhz( pChip->cfg.rc_freq                ));
-       sam3_sprintf(pChip,"  mainosc: %3.03f MHz\n", _tomhz( pChip->cfg.mainosc_freq   ));
-       sam3_sprintf(pChip,"     plla: %3.03f MHz\n", _tomhz( pChip->cfg.plla_freq              ));
-       sam3_sprintf(pChip," cpu-freq: %3.03f MHz\n", _tomhz( pChip->cfg.cpu_freq               ));
-       sam3_sprintf(pChip,"mclk-freq: %3.03f MHz\n", _tomhz( pChip->cfg.mclk_freq              ));     
+       sam3_sprintf(pChip,"   rc-osc: %3.03f MHz\n", _tomhz(pChip->cfg.rc_freq         ));
+       sam3_sprintf(pChip,"  mainosc: %3.03f MHz\n", _tomhz(pChip->cfg.mainosc_freq    ));
+       sam3_sprintf(pChip,"     plla: %3.03f MHz\n", _tomhz(pChip->cfg.plla_freq               ));
+       sam3_sprintf(pChip," cpu-freq: %3.03f MHz\n", _tomhz(pChip->cfg.cpu_freq                ));
+       sam3_sprintf(pChip,"mclk-freq: %3.03f MHz\n", _tomhz(pChip->cfg.mclk_freq               ));     
 
 
-       sam3_sprintf( pChip, " UniqueId: 0x%08x 0x%08x 0x%08x 0x%08x\n",
+       sam3_sprintf(pChip, " UniqueId: 0x%08x 0x%08x 0x%08x 0x%08x\n",
                                  pChip->cfg.unique_id[0],
                                  pChip->cfg.unique_id[1],
                                  pChip->cfg.unique_id[2],
@@ -1563,13 +1563,13 @@ sam3_erase_check(struct flash_bank_s *bank)
                LOG_ERROR("Target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
-       if ( 0 == bank->num_sectors ){
+       if (0 == bank->num_sectors ){
                LOG_ERROR("Target: not supported/not probed\n");
                return ERROR_FAIL;
        }
 
        LOG_INFO("sam3 - supports auto-erase, erase_check ignored");
-       for ( x = 0 ; x < bank->num_sectors ; x++ ){
+       for (x = 0 ; x < bank->num_sectors ; x++ ){
                bank->sectors[x].is_erased = 1;
        }
 
@@ -1592,21 +1592,21 @@ sam3_protect_check(struct flash_bank_s *bank)
        }
 
        pPrivate = get_sam3_bank_private(bank);
-       if ( !pPrivate ){
+       if (!pPrivate ){
                LOG_ERROR("no private for this bank?");
                return ERROR_FAIL;
        }
-       if ( !(pPrivate->probed) ){
+       if (!(pPrivate->probed) ){
                return ERROR_FLASH_BANK_NOT_PROBED;
        }
        
-       r = FLASHD_GetLockBits( pPrivate , &v );
-       if ( r != ERROR_OK ){
+       r = FLASHD_GetLockBits(pPrivate , &v );
+       if (r != ERROR_OK ){
                LOG_DEBUG("Failed: %d",r);
                return r;
        }
 
-       for ( x = 0 ; x < pPrivate->nsectors ; x++ ){
+       for (x = 0 ; x < pPrivate->nsectors ; x++ ){
                bank->sectors[x].is_protected = (!!(v & (1 << x)));
        }
        LOG_DEBUG("Done");
@@ -1626,16 +1626,16 @@ sam3_flash_bank_command(struct command_context_s *cmd_ctx,
        
        // is this an existing chip?
        while (pChip){
-               if ( pChip->target == bank->target ){
+               if (pChip->target == bank->target ){
                        break;
                }
                pChip = pChip->next;
        }
 
-       if ( !pChip ){
+       if (!pChip ){
                // this is a *NEW* chip
-               pChip = calloc( 1, sizeof(struct sam3_chip) );
-               if ( !pChip ){
+               pChip = calloc(1, sizeof(struct sam3_chip) );
+               if (!pChip ){
                        LOG_ERROR("NO RAM!");
                        return ERROR_FAIL;
                }
@@ -1648,13 +1648,13 @@ sam3_flash_bank_command(struct command_context_s *cmd_ctx,
                pChip->cfg.slow_freq = 32768;
                pChip->probed = 0;
                pChip->mbuf = membuf_new();
-               if ( !(pChip->mbuf) ){
+               if (!(pChip->mbuf) ){
                        LOG_ERROR("no memory");
                        return ERROR_FAIL;
                }
        }
        
-       switch ( bank->base ){
+       switch (bank->base ){
        default:
                LOG_ERROR("Address 0x%08x invalid bank address (try 0x%08x or 0x%08x)",
                                  ((unsigned int)(bank->base)),
@@ -1681,7 +1681,7 @@ sam3_flash_bank_command(struct command_context_s *cmd_ctx,
 }
 
 static int
-sam3_GetDetails( struct sam3_bank_private *pPrivate )
+sam3_GetDetails(struct sam3_bank_private *pPrivate )
 {
        const struct sam3_chip_details *pDetails;
        struct sam3_chip *pChip;
@@ -1693,26 +1693,26 @@ sam3_GetDetails( struct sam3_bank_private *pPrivate )
 
        LOG_DEBUG("Begin");
        pDetails = all_sam3_details;
-       while ( pDetails->name ){
-               if ( pDetails->chipid_cidr == pPrivate->pChip->cfg.CHIPID_CIDR ){
+       while (pDetails->name ){
+               if (pDetails->chipid_cidr == pPrivate->pChip->cfg.CHIPID_CIDR ){
                        break;
                } else {
                        pDetails++;
                }
        }
-       if ( pDetails->name == NULL ){
+       if (pDetails->name == NULL ){
                LOG_ERROR("SAM3 ChipID 0x%08x not found in table (perhaps you can this chip?)", 
                                  (unsigned int)(pPrivate->pChip->cfg.CHIPID_CIDR) );
                // Help the victim, print details about the chip
-               membuf_reset( pPrivate->pChip->mbuf );
-               membuf_sprintf( pPrivate->pChip->mbuf, 
+               membuf_reset(pPrivate->pChip->mbuf );
+               membuf_sprintf(pPrivate->pChip->mbuf, 
                                                "SAM3 CHIPID_CIDR: 0x%08x decodes as follows\n",
                                                pPrivate->pChip->cfg.CHIPID_CIDR );
-               sam3_explain_chipid_cidr( pPrivate->pChip );
-               cp = membuf_strtok( pPrivate->pChip->mbuf, "\n", &vp );
+               sam3_explain_chipid_cidr(pPrivate->pChip );
+               cp = membuf_strtok(pPrivate->pChip->mbuf, "\n", &vp );
                while (cp){
                        LOG_INFO("%s", cp );
-                       cp = membuf_strtok( NULL, "\n", &vp );
+                       cp = membuf_strtok(NULL, "\n", &vp );
                }
                return ERROR_FAIL;
        }
@@ -1730,17 +1730,17 @@ sam3_GetDetails( struct sam3_bank_private *pPrivate )
        //
 
        // save the "bank" pointers
-       for ( x = 0 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
+       for (x = 0 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
                saved_banks[ x ] = pChip->details.bank[x].pBank;
        }
 
        // Overwrite the "details" structure.
-       memcpy( &(pPrivate->pChip->details), 
+       memcpy(&(pPrivate->pChip->details), 
                        pDetails, 
                        sizeof(pPrivate->pChip->details));
 
        // now fix the ghosted pointers
-       for ( x = 0 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
+       for (x = 0 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
                pChip->details.bank[x].pChip = pChip;
                pChip->details.bank[x].pBank = saved_banks[x];
        }
@@ -1768,49 +1768,49 @@ _sam3_probe(struct flash_bank_s *bank, int noise)
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       pPrivate = get_sam3_bank_private( bank );
-       if ( !pPrivate ){
+       pPrivate = get_sam3_bank_private(bank );
+       if (!pPrivate ){
                LOG_ERROR("Invalid/unknown bank number\n");
                return ERROR_FAIL;
        }
 
        r = sam3_ReadAllRegs(pPrivate->pChip);
-       if ( r != ERROR_OK ){
+       if (r != ERROR_OK ){
                return r;
        }
 
        
        LOG_DEBUG("Here");
-       r = sam3_GetInfo( pPrivate->pChip );
-       if ( r != ERROR_OK ){
+       r = sam3_GetInfo(pPrivate->pChip );
+       if (r != ERROR_OK ){
                return r;
        }
-       if ( !(pPrivate->pChip->probed) ){
+       if (!(pPrivate->pChip->probed) ){
                pPrivate->pChip->probed = 1;
                LOG_DEBUG("Here");
-               r = sam3_GetDetails( pPrivate );
-               if ( r != ERROR_OK ){
+               r = sam3_GetDetails(pPrivate );
+               if (r != ERROR_OK ){
                        return r;
                }               
        }
 
        // update the flash bank size
-       for ( x = 0 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
-               if ( bank->base == pPrivate->pChip->details.bank[0].base_address ){
+       for (x = 0 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
+               if (bank->base == pPrivate->pChip->details.bank[0].base_address ){
                        bank->size =  pPrivate->pChip->details.bank[0].size_bytes;
                        break;
                }
        }
 
-       if ( bank->sectors == NULL ){
-               bank->sectors     = calloc(pPrivate->nsectors, (sizeof( (bank->sectors)[0] )));
-               if ( bank->sectors == NULL ){
+       if (bank->sectors == NULL ){
+               bank->sectors     = calloc(pPrivate->nsectors, (sizeof((bank->sectors)[0] )));
+               if (bank->sectors == NULL ){
                        LOG_ERROR("No memory!");
                        return ERROR_FAIL;
                }
                bank->num_sectors = pPrivate->nsectors;
                        
-               for ( x = 0 ; ((int)(x)) < bank->num_sectors ; x++ ){
+               for (x = 0 ; ((int)(x)) < bank->num_sectors ; x++ ){
                        bank->sectors[x].size         = pPrivate->sector_size;
                        bank->sectors[x].offset       = x * (pPrivate->sector_size);
                        // mark as unknown
@@ -1821,14 +1821,14 @@ _sam3_probe(struct flash_bank_s *bank, int noise)
 
        pPrivate->probed = 1;
 
-       r = sam3_protect_check( bank );
-       if ( r != ERROR_OK ){
+       r = sam3_protect_check(bank );
+       if (r != ERROR_OK ){
                return r;
        }
 
        LOG_DEBUG("Bank = %d, nbanks = %d",
                          pPrivate->bank_number , pPrivate->pChip->details.n_banks);
-       if ( (pPrivate->bank_number + 1) == pPrivate->pChip->details.n_banks ){
+       if ((pPrivate->bank_number + 1) == pPrivate->pChip->details.n_banks ){
                // read unique id, 
                // it appears to be associated with the *last* flash bank.
                FLASHD_ReadUniqueID(pPrivate);
@@ -1840,13 +1840,13 @@ _sam3_probe(struct flash_bank_s *bank, int noise)
 static int 
 sam3_probe(struct flash_bank_s *bank)
 {
-       return _sam3_probe( bank, 1 );
+       return _sam3_probe(bank, 1 );
 }
 
 static int 
 sam3_auto_probe(struct flash_bank_s *bank)
 {
-       return _sam3_probe( bank, 0 );
+       return _sam3_probe(bank, 0 );
 }
 
 
@@ -1863,21 +1863,21 @@ sam3_erase(struct flash_bank_s *bank, int first, int last)
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       r = sam3_auto_probe( bank );
-       if ( r != ERROR_OK ){
+       r = sam3_auto_probe(bank );
+       if (r != ERROR_OK ){
                LOG_DEBUG("Here,r=%d",r);
                return r;
        }
 
-       pPrivate = get_sam3_bank_private( bank );
-       if ( !(pPrivate->probed) ){
+       pPrivate = get_sam3_bank_private(bank );
+       if (!(pPrivate->probed) ){
                return ERROR_FLASH_BANK_NOT_PROBED;
        }
 
-       if ( (first == 0) && ((last + 1)== ((int)(pPrivate->nsectors))) ){
+       if ((first == 0) && ((last + 1)== ((int)(pPrivate->nsectors))) ){
                // whole chip
                LOG_DEBUG("Here");
-               return FLASHD_EraseEntireBank( pPrivate );
+               return FLASHD_EraseEntireBank(pPrivate );
        }
        LOG_INFO("sam3 auto-erases while programing (request ignored)");
        return ERROR_OK;
@@ -1895,15 +1895,15 @@ sam3_protect(struct flash_bank_s *bank, int set, int first, int last)
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       pPrivate = get_sam3_bank_private( bank );
-       if ( !(pPrivate->probed) ){
+       pPrivate = get_sam3_bank_private(bank );
+       if (!(pPrivate->probed) ){
                return ERROR_FLASH_BANK_NOT_PROBED;
        }
 
-       if ( set ){
-               r = FLASHD_Lock( pPrivate, (unsigned)(first), (unsigned)(last));
+       if (set ){
+               r = FLASHD_Lock(pPrivate, (unsigned)(first), (unsigned)(last));
        } else {
-               r = FLASHD_Unlock( pPrivate, (unsigned)(first), (unsigned)(last));
+               r = FLASHD_Unlock(pPrivate, (unsigned)(first), (unsigned)(last));
        }
        LOG_DEBUG("End: r=%d",r);
 
@@ -1913,7 +1913,7 @@ sam3_protect(struct flash_bank_s *bank, int set, int first, int last)
 
 
 static int
-sam3_info( flash_bank_t *bank, char *buf, int buf_size )
+sam3_info(flash_bank_t *bank, char *buf, int buf_size )
 {
        if (bank->target->state != TARGET_HALTED) {
                LOG_ERROR("Target not halted");
@@ -1924,7 +1924,7 @@ sam3_info( flash_bank_t *bank, char *buf, int buf_size )
 }
 
 static int
-sam3_page_read( struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *buf )
+sam3_page_read(struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *buf )
 {
        uint32_t adr;
        int r;
@@ -1932,12 +1932,12 @@ sam3_page_read( struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *b
        adr = pagenum * pPrivate->page_size;
        adr += adr + pPrivate->base_address;
 
-       r = target_read_memory( pPrivate->pChip->target, 
+       r = target_read_memory(pPrivate->pChip->target, 
                                                        adr,
                                                        4, /* THIS*MUST*BE* in 32bit values */
                                                        pPrivate->page_size / 4,
                                                        buf );
-       if ( r != ERROR_OK ){
+       if (r != ERROR_OK ){
                LOG_ERROR("SAM3: Flash program failed to read page phys address: 0x%08x", (unsigned int)(adr) );
        }
        return r;
@@ -1964,7 +1964,7 @@ sam3_page_read( struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *b
 // };
 //
 // 
-// uint32_t sam3_function( struct foo *p )
+// uint32_t sam3_function(struct foo *p )
 // {
 //   volatile uint32_t *v;
 //   uint32_t *d;
@@ -1978,7 +1978,7 @@ sam3_page_read( struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *b
 // 
 //   do {
 //     *d++ = *s++;
-//   } while ( --n )
+//   } while (--n )
 //     ;
 // 
 //   v = p->base;
@@ -1986,7 +1986,7 @@ sam3_page_read( struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *b
 //   v[ 1 ] = p->cmd;
 //   do {
 //     r = v[8/4];
-//   } while ( !(r&1) )
+//   } while (!(r&1) )
 //     ;
 //   return r;
 // }
@@ -2036,7 +2036,7 @@ sam3_page_write_opcodes[] = {
 
 
 static int
-sam3_page_write( struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *buf )
+sam3_page_write(struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *buf )
 {
        uint32_t adr;
        uint32_t status;
@@ -2046,30 +2046,30 @@ sam3_page_write( struct sam3_bank_private *pPrivate, unsigned pagenum, uint8_t *
        adr += (adr + pPrivate->base_address);
 
        LOG_DEBUG("Wr Page %u @ phys address: 0x%08x", pagenum, (unsigned int)(adr) );
-       r = target_write_memory( pPrivate->pChip->target, 
+       r = target_write_memory(pPrivate->pChip->target, 
                                                         adr,
                                                         4, /* THIS*MUST*BE* in 32bit values */
                                                         pPrivate->page_size / 4,
                                                         buf );
-       if ( r != ERROR_OK ){
+       if (r != ERROR_OK ){
                LOG_ERROR("SAM3: Failed to write (buffer) page at phys address 0x%08x", (unsigned int)(adr) );
                return r;
        }
 
-       r = EFC_PerformCommand( pPrivate,
+       r = EFC_PerformCommand(pPrivate,
                                                        // send Erase & Write Page
                                                        AT91C_EFC_FCMD_EWP,
                                                        pagenum,
                                                        &status );
 
-       if ( r != ERROR_OK ){
+       if (r != ERROR_OK ){
                LOG_ERROR("SAM3: Error performing Erase & Write page @ phys address 0x%08x", (unsigned int)(adr) );
        }
-       if ( status & (1 << 2) ){
+       if (status & (1 << 2) ){
                LOG_ERROR("SAM3: Page @ Phys address 0x%08x is locked", (unsigned int)(adr) );
                return ERROR_FAIL;
        }
-       if ( status & (1 << 1) ){
+       if (status & (1 << 1) ){
                LOG_ERROR("SAM3: Flash Command error @phys address 0x%08x", (unsigned int)(adr) );
                return ERROR_FAIL;
        }
@@ -2095,7 +2095,7 @@ sam3_write(struct flash_bank_s *bank,
        uint8_t *pagebuffer;
 
        // ignore dumb requests
-       if ( count == 0 ){
+       if (count == 0 ){
                return ERROR_OK;
        }
 
@@ -2105,12 +2105,12 @@ sam3_write(struct flash_bank_s *bank,
        }
 
        pPrivate = get_sam3_bank_private(bank);
-       if ( !(pPrivate->probed) ){
+       if (!(pPrivate->probed) ){
                return ERROR_FLASH_BANK_NOT_PROBED;
        }
 
 
-       if ( (offset + count) > pPrivate->size_bytes ){
+       if ((offset + count) > pPrivate->size_bytes ){
                LOG_ERROR("Flash write error - past end of bank");
                LOG_ERROR(" offset: 0x%08x, count 0x%08x, BankEnd: 0x%08x",
                                  (unsigned int)(offset), 
@@ -2119,7 +2119,7 @@ sam3_write(struct flash_bank_s *bank,
                return ERROR_FAIL;
        }
 
-       pagebuffer = alloca( pPrivate->page_size );
+       pagebuffer = alloca(pPrivate->page_size );
                
        // what page do we start & end in?
        page_cur = offset / pPrivate->page_size;
@@ -2136,20 +2136,20 @@ sam3_write(struct flash_bank_s *bank,
        //    (3) non-aligned end.
        
        // Handle special case - all one page.
-       if ( page_cur == page_end ){
+       if (page_cur == page_end ){
                LOG_DEBUG("Special case, all in one page");
-               r = sam3_page_read( pPrivate, page_cur, pagebuffer );
-               if ( r != ERROR_OK ){
+               r = sam3_page_read(pPrivate, page_cur, pagebuffer );
+               if (r != ERROR_OK ){
                        return r;
                }
                
                page_offset = (offset & (pPrivate->page_size-1));
-               memcpy( pagebuffer + page_offset,
+               memcpy(pagebuffer + page_offset,
                                buffer,
                                count );
                
-               r = sam3_page_write( pPrivate, page_cur, pagebuffer );
-               if ( r != ERROR_OK ){
+               r = sam3_page_write(pPrivate, page_cur, pagebuffer );
+               if (r != ERROR_OK ){
                        return r;
                }
                return ERROR_OK;
@@ -2157,22 +2157,22 @@ sam3_write(struct flash_bank_s *bank,
 
        // non-aligned start
        page_offset = offset & (pPrivate->page_size - 1);
-       if ( page_offset ){
+       if (page_offset ){
                LOG_DEBUG("Not-Aligned start");
                // read the partial
-               r = sam3_page_read( pPrivate, page_cur, pagebuffer );
-               if ( r != ERROR_OK ){
+               r = sam3_page_read(pPrivate, page_cur, pagebuffer );
+               if (r != ERROR_OK ){
                        return r;
                }
 
                // over-write with new data
                n = (pPrivate->page_size - page_offset );
-               memcpy( pagebuffer + page_offset,
+               memcpy(pagebuffer + page_offset,
                                buffer,
                                n );
        
-               r = sam3_page_write( pPrivate, page_cur, pagebuffer );
-               if ( r != ERROR_OK ){
+               r = sam3_page_write(pPrivate, page_cur, pagebuffer );
+               if (r != ERROR_OK ){
                        return r;
                }
 
@@ -2188,10 +2188,10 @@ sam3_write(struct flash_bank_s *bank,
        LOG_DEBUG("Full Page Loop: cur=%d, end=%d, count = 0x%08x", 
                          (int)page_cur, (int)page_end, (unsigned int)(count) );
 
-       while ( (page_cur < page_end) && 
+       while ((page_cur < page_end) && 
                   (count >= pPrivate->page_size) ){
-               r = sam3_page_write( pPrivate, page_cur, buffer );
-               if ( r != ERROR_OK ){
+               r = sam3_page_write(pPrivate, page_cur, buffer );
+               if (r != ERROR_OK ){
                        return r;
                }
                count    -= pPrivate->page_size;
@@ -2200,17 +2200,17 @@ sam3_write(struct flash_bank_s *bank,
        }
 
        // terminal partial page?
-       if ( count ){
+       if (count ){
                LOG_DEBUG("Terminal partial page, count = 0x%08x", (unsigned int)(count));
                // we have a partial page
-               r = sam3_page_read( pPrivate, page_cur, pagebuffer );
-               if ( r != ERROR_OK ){
+               r = sam3_page_read(pPrivate, page_cur, pagebuffer );
+               if (r != ERROR_OK ){
                        return r;
                }
                // data goes at start
-               memcpy( pagebuffer, buffer, count );
-               r = sam3_page_write( pPrivate, page_cur, pagebuffer );
-               if ( r != ERROR_OK ){
+               memcpy(pagebuffer, buffer, count );
+               r = sam3_page_write(pPrivate, page_cur, pagebuffer );
+               if (r != ERROR_OK ){
                        return r;
                }
                buffer += count;
@@ -2221,7 +2221,7 @@ sam3_write(struct flash_bank_s *bank,
 }
 
 static int
-sam3_handle_info_command(  struct command_context_s *cmd_ctx, char *cmd, char **argv, int argc )
+sam3_handle_info_command(struct command_context_s *cmd_ctx, char *cmd, char **argv, int argc )
 {
        struct sam3_chip *pChip;
        void *vp;
@@ -2230,17 +2230,17 @@ sam3_handle_info_command(  struct command_context_s *cmd_ctx, char *cmd, char **
        int r;
 
        pChip = get_current_sam3(cmd_ctx);
-       if ( !pChip ){
+       if (!pChip ){
                return ERROR_OK;
        }
 
        r = 0;
        
        // bank0 must exist before we can do anything
-       if ( pChip->details.bank[0].pBank == NULL ){
+       if (pChip->details.bank[0].pBank == NULL ){
                x = 0;
        need_define:
-               command_print( cmd_ctx, 
+               command_print(cmd_ctx, 
                                           "Please define bank %d via command: flash bank %s ... ", 
                                           x,
                                           at91sam3_flash.name );
@@ -2248,9 +2248,9 @@ sam3_handle_info_command(  struct command_context_s *cmd_ctx, char *cmd, char **
        }
 
        // if bank 0 is not probed, then probe it
-       if ( !(pChip->details.bank[0].probed) ){
-               r = sam3_auto_probe( pChip->details.bank[0].pBank );
-               if ( r != ERROR_OK ){
+       if (!(pChip->details.bank[0].probed) ){
+               r = sam3_auto_probe(pChip->details.bank[0].pBank );
+               if (r != ERROR_OK ){
                        return ERROR_FAIL;
                }
        }
@@ -2260,45 +2260,45 @@ sam3_handle_info_command(  struct command_context_s *cmd_ctx, char *cmd, char **
                
 
        // auto-probe other banks, 0 done above
-    for ( x = 1 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
+    for (x = 1 ; x < SAM3_MAX_FLASH_BANKS ; x++ ){
                // skip banks not present
-               if ( !(pChip->details.bank[x].present) ){
+               if (!(pChip->details.bank[x].present) ){
                        continue;
                }
                
-               if ( pChip->details.bank[x].pBank == NULL ){
+               if (pChip->details.bank[x].pBank == NULL ){
                        goto need_define;
                }
 
-               if ( pChip->details.bank[x].probed ){
+               if (pChip->details.bank[x].probed ){
                        continue;
                }
                
-               r = sam3_auto_probe( pChip->details.bank[x].pBank );
-               if ( r != ERROR_OK ){
+               r = sam3_auto_probe(pChip->details.bank[x].pBank );
+               if (r != ERROR_OK ){
                        return r;
                }
        }
                        
 
-       r = sam3_GetInfo( pChip );
-       if ( r != ERROR_OK ){
+       r = sam3_GetInfo(pChip );
+       if (r != ERROR_OK ){
                LOG_DEBUG("Sam3Info, Failed %d\n",r);
                return r;
        }
        
 
        // print results
-       cp = membuf_strtok( pChip->mbuf, "\n", &vp );
+       cp = membuf_strtok(pChip->mbuf, "\n", &vp );
        while (cp){
                command_print(cmd_ctx,"%s", cp );
-               cp = membuf_strtok( NULL, "\n", &vp );
+               cp = membuf_strtok(NULL, "\n", &vp );
        }
        return ERROR_OK;
 }
 
 static int
-sam3_handle_gpnvm_command( struct command_context_s *cmd_ctx, char *cmd, char **argv, int argc)
+sam3_handle_gpnvm_command(struct command_context_s *cmd_ctx, char *cmd, char **argv, int argc)
 {
        unsigned x,v;
        uint32_t v32;
@@ -2306,30 +2306,30 @@ sam3_handle_gpnvm_command( struct command_context_s *cmd_ctx, char *cmd, char **
        struct sam3_chip *pChip;
 
        pChip = get_current_sam3(cmd_ctx);
-       if ( !pChip ){
+       if (!pChip ){
                return ERROR_OK;
        }
 
-       if ( pChip->target->state != TARGET_HALTED ){
+       if (pChip->target->state != TARGET_HALTED ){
                LOG_ERROR("sam3 - target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
 
 
-       if ( pChip->details.bank[0].pBank == NULL ){
-               command_print( cmd_ctx, "Bank0 must be defined first via: flash bank %s ...",
+       if (pChip->details.bank[0].pBank == NULL ){
+               command_print(cmd_ctx, "Bank0 must be defined first via: flash bank %s ...",
                                           at91sam3_flash.name );
                return ERROR_FAIL;
        }
-       if ( !pChip->details.bank[0].probed ){
-               r = sam3_auto_probe( pChip->details.bank[0].pBank );
-               if ( r != ERROR_OK ){
+       if (!pChip->details.bank[0].probed ){
+               r = sam3_auto_probe(pChip->details.bank[0].pBank );
+               if (r != ERROR_OK ){
                        return r;
                }
        }
 
 
-       switch ( argc ){
+       switch (argc ){
        default:
                command_print(cmd_ctx,"Too many parameters\n");
                return ERROR_COMMAND_SYNTAX_ERROR;
@@ -2342,12 +2342,12 @@ sam3_handle_gpnvm_command( struct command_context_s *cmd_ctx, char *cmd, char **
                who = -1;
                break;
        case 2:
-               if ( (0 == strcmp( argv[0], "show" )) && (0 == strcmp( argv[1], "all" )) ){
+               if ((0 == strcmp(argv[0], "show" )) && (0 == strcmp(argv[1], "all" )) ){
                        who = -1;
                } else {
-                       r = parse_u32( argv[1], &v32 );
-                       if ( r != ERROR_OK ){
-                               command_print( cmd_ctx, "Not a number: %s", argv[1] );
+                       r = parse_u32(argv[1], &v32 );
+                       if (r != ERROR_OK ){
+                               command_print(cmd_ctx, "Not a number: %s", argv[1] );
                                return r;
                        }
                        who = v32;
@@ -2355,20 +2355,20 @@ sam3_handle_gpnvm_command( struct command_context_s *cmd_ctx, char *cmd, char **
                break;
        }
 
-       if ( 0 == strcmp( "show", argv[0] ) ){
-               if ( who == -1 ){
+       if (0 == strcmp("show", argv[0] ) ){
+               if (who == -1 ){
                showall:
-                       for ( x = 0 ; x < pChip->details.n_gpnvms ; x++ ){
-                               r = FLASHD_GetGPNVM( &(pChip->details.bank[0]), x, &v );
-                               if ( r != ERROR_OK ){
+                       for (x = 0 ; x < pChip->details.n_gpnvms ; x++ ){
+                               r = FLASHD_GetGPNVM(&(pChip->details.bank[0]), x, &v );
+                               if (r != ERROR_OK ){
                                        break;
                                }
                                command_print(cmd_ctx, "sam3-gpnvm%u: %u", x, v );
                        }
                        return r;
                }
-               if ( (who >= 0) && (((unsigned)(who)) < pChip->details.n_gpnvms) ){
-                       r = FLASHD_GetGPNVM( &(pChip->details.bank[0]), who, &v );
+               if ((who >= 0) && (((unsigned)(who)) < pChip->details.n_gpnvms) ){
+                       r = FLASHD_GetGPNVM(&(pChip->details.bank[0]), who, &v );
                        command_print(cmd_ctx, "sam3-gpnvm%u: %u", who, v );
                        return r;
                } else {
@@ -2377,25 +2377,25 @@ sam3_handle_gpnvm_command( struct command_context_s *cmd_ctx, char *cmd, char **
                }
        }
 
-       if ( who == -1 ){
-               command_print( cmd_ctx, "Missing GPNVM number");
+       if (who == -1 ){
+               command_print(cmd_ctx, "Missing GPNVM number");
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
        
-       if ( 0 == strcmp( "set", argv[0] ) ){
-               r = FLASHD_SetGPNVM( &(pChip->details.bank[0]), who );
-       } else if ( (0 == strcmp( "clr", argv[0] )) ||
-                          (0 == strcmp( "clear", argv[0])) ){ // quietly accept both
-               r = FLASHD_ClrGPNVM( &(pChip->details.bank[0]), who );
+       if (0 == strcmp("set", argv[0] ) ){
+               r = FLASHD_SetGPNVM(&(pChip->details.bank[0]), who );
+       } else if ((0 == strcmp("clr", argv[0] )) ||
+                          (0 == strcmp("clear", argv[0])) ){ // quietly accept both
+               r = FLASHD_ClrGPNVM(&(pChip->details.bank[0]), who );
        } else {
-               command_print( cmd_ctx, "Unkown command: %s", argv[0] );
+               command_print(cmd_ctx, "Unkown command: %s", argv[0] );
                r = ERROR_COMMAND_SYNTAX_ERROR;
        }
        return r;
 }
 
 static int 
-sam3_handle_slowclk_command( struct command_context_s *cmd_ctx, char *cmd, char **argv, int argc)
+sam3_handle_slowclk_command(struct command_context_s *cmd_ctx, char *cmd, char **argv, int argc)
 {
        uint32_t v;
        int r;
@@ -2403,19 +2403,19 @@ sam3_handle_slowclk_command( struct command_context_s *cmd_ctx, char *cmd, char
        struct sam3_chip *pChip;
 
        pChip = get_current_sam3(cmd_ctx);
-       if ( !pChip ){
+       if (!pChip ){
                return ERROR_OK;
        }
 
 
-       switch ( argc ){
+       switch (argc ){
        case 0:
                // show
                break;
        case 1:
                // set
-               r = parse_u32( argv[0], &v );
-               if ( v > 200000 ){
+               r = parse_u32(argv[0], &v );
+               if (v > 200000 ){
                        // absurd slow clock of 200Khz?
                        command_print(cmd_ctx,"Absurd/illegal slow clock freq: %d\n", (int)(v));
                        return ERROR_COMMAND_SYNTAX_ERROR;
@@ -2425,11 +2425,11 @@ sam3_handle_slowclk_command( struct command_context_s *cmd_ctx, char *cmd, char
                
        default:
                // error
-               command_print( cmd_ctx,"Too many parameters");
+               command_print(cmd_ctx,"Too many parameters");
                return ERROR_COMMAND_SYNTAX_ERROR;
                break;
        }
-       command_print( cmd_ctx, "Slowclk freq: %d.%03dkhz", 
+       command_print(cmd_ctx, "Slowclk freq: %d.%03dkhz", 
                                   (int)(pChip->cfg.slow_freq/ 1000),
                                   (int)(pChip->cfg.slow_freq% 1000));
        return ERROR_OK;
@@ -2438,26 +2438,26 @@ sam3_handle_slowclk_command( struct command_context_s *cmd_ctx, char *cmd, char
 
 static int sam3_registered;
 static int
-sam3_register_commands( struct command_context_s *cmd_ctx)
+sam3_register_commands(struct command_context_s *cmd_ctx)
 {
        command_t *pCmd;
 
        // only register once
-       if ( !sam3_registered ){
+       if (!sam3_registered ){
                sam3_registered++;
 
-               pCmd = register_command( cmd_ctx, NULL, "at91sam3", NULL, COMMAND_ANY, NULL );
-               register_command( cmd_ctx, pCmd,
+               pCmd = register_command(cmd_ctx, NULL, "at91sam3", NULL, COMMAND_ANY, NULL );
+               register_command(cmd_ctx, pCmd,
                                                  "gpnvm", 
                                                  sam3_handle_gpnvm_command, 
                                                  COMMAND_EXEC, 
                                                  "at91sam3 gpnvm [action [<BIT>], by default 'show', otherwise set | clear BIT");
-               register_command( cmd_ctx, pCmd,
+               register_command(cmd_ctx, pCmd,
                                                  "info",
                                                  sam3_handle_info_command,
                                                  COMMAND_EXEC,
                                                  "at91sam3 info - print information about the current sam3 chip");
-               register_command( cmd_ctx, pCmd,
+               register_command(cmd_ctx, pCmd,
                                                  "slowclk",
                                                  sam3_handle_slowclk_command,
                                                  COMMAND_EXEC,

diff --git a/src/helper/membuf.c b/src/helper/membuf.c
index e4280f9e0b665416882cc682ad948dcae0e1a757..0c1673fde16ff371fedf2af05fb4965c25f2cb48 100644 (file)
--- a/src/helper/membuf.c
+++ b/src/helper/membuf.c
@@ -37,32 +37,32 @@ struct membuf {
 };
 
 
-#define space_avail( pBuf )  (pBuf->maxlen - pBuf->curlen)
-#define dataend( pBuf )      ( ((char *)(pBuf->buf)) + pBuf->curlen )
+#define space_avail(pBuf )  (pBuf->maxlen - pBuf->curlen)
+#define dataend(pBuf )      (((char *)(pBuf->buf)) + pBuf->curlen )
 
 size_t 
-membuf_len( struct membuf *pBuf )
+membuf_len(struct membuf *pBuf )
 {
     return pBuf->curlen;
 }
 
 const void *
-membuf_datapointer( struct membuf *pBuf )
+membuf_datapointer(struct membuf *pBuf )
 {
     return ((void *)(pBuf->buf));
 }
 
 const char *
-membuf_strtok( struct membuf *pBuf, const char *sep, void **pLast )
+membuf_strtok(struct membuf *pBuf, const char *sep, void **pLast )
 {
-    if ( pBuf ){
+    if (pBuf ){
        pBuf->_strtoklast = NULL;
        *pLast = pBuf;
-       return strtok_r( ((char *)(pBuf->buf)), sep, &(pBuf->_strtoklast) );
+       return strtok_r(((char *)(pBuf->buf)), sep, &(pBuf->_strtoklast) );
     } else {
        // recover our pBuf
        pBuf = *((struct membuf **)(pLast));
-       return strtok_r( NULL, sep, &(pBuf->_strtoklast) );
+       return strtok_r(NULL, sep, &(pBuf->_strtoklast) );
     }
 }
        
@@ -74,11 +74,11 @@ membuf_new(void)
     // by default - parameters are zero.
     struct membuf *pBuf;
 
-    pBuf = calloc( 1, sizeof(*pBuf) );
-    if ( pBuf ){
+    pBuf = calloc(1, sizeof(*pBuf) );
+    if (pBuf ){
        // we *ALWAYS* allocate +1 for null terminator.
-       pBuf->buf = calloc( DEFAULT_BUFSIZE + 1, sizeof(char));
-       if ( pBuf->buf == NULL ){
+       pBuf->buf = calloc(DEFAULT_BUFSIZE + 1, sizeof(char));
+       if (pBuf->buf == NULL ){
            free(pBuf);
            pBuf = NULL;
        } else {
@@ -90,7 +90,7 @@ membuf_new(void)
 
 
 struct membuf *
-membuf_grow( struct membuf *pBuf, int n )
+membuf_grow(struct membuf *pBuf, int n )
 {
     void *vp;
     signed int newsize;
@@ -99,13 +99,13 @@ membuf_grow( struct membuf *pBuf, int n )
     newsize = ((int)(pBuf->maxlen)) + n;
 
     // do not go negative, or too small
-    if ( newsize < DEFAULT_BUFSIZE ){
+    if (newsize < DEFAULT_BUFSIZE ){
        newsize = DEFAULT_BUFSIZE;
     }
 
     // always alloc +1 for the null terminator
-    vp = realloc( pBuf->buf, newsize + 1 );
-    if ( vp ){
+    vp = realloc(pBuf->buf, newsize + 1 );
+    if (vp ){
        pBuf->buf    = vp;
        pBuf->maxlen = newsize;
        return pBuf;
@@ -115,16 +115,16 @@ membuf_grow( struct membuf *pBuf, int n )
 }
 
 
-void membuf_reset( struct membuf *pBuf )
+void membuf_reset(struct membuf *pBuf )
 {
     pBuf->curlen = 0;
 }
 
 
-void membuf_delete( struct membuf *pBuf )
+void membuf_delete(struct membuf *pBuf )
 {
-    if ( pBuf ){
-       if ( pBuf->buf){
+    if (pBuf ){
+       if (pBuf->buf){
            // wack data so it cannot be reused
            memset(pBuf->buf,0,pBuf->maxlen);
            free(pBuf->buf);
@@ -136,18 +136,18 @@ void membuf_delete( struct membuf *pBuf )
 }
 
 int
-membuf_sprintf( struct membuf *pBuf , const char *fmt, ... )
+membuf_sprintf(struct membuf *pBuf , const char *fmt, ... )
 {
     int r;
     va_list ap;
-    va_start( ap, fmt );
-    r = membuf_vsprintf( pBuf, fmt, ap );
+    va_start(ap, fmt );
+    r = membuf_vsprintf(pBuf, fmt, ap );
     va_end(ap);
     return r;
 }
 
 int
-membuf_vsprintf( struct membuf *pBuf, const char *fmt, va_list ap )
+membuf_vsprintf(struct membuf *pBuf, const char *fmt, va_list ap )
 {
     int r;
     size_t sa;
@@ -159,11 +159,11 @@ membuf_vsprintf( struct membuf *pBuf, const char *fmt, va_list ap )
        sa = space_avail(pBuf);
 
        // do work
-       r = vsnprintf( dataend( pBuf ),
+       r = vsnprintf(dataend(pBuf ),
                       sa,
                       fmt, 
                       ap );
-       if ( (r > 0) && (((size_t)(r)) < sa) ){
+       if ((r > 0) && (((size_t)(r)) < sa) ){
            // Success!
            pBuf->curlen += ((size_t)(r));
            // remember: We always alloc'ed +1
@@ -174,7 +174,7 @@ membuf_vsprintf( struct membuf *pBuf, const char *fmt, va_list ap )
        }
 
        // failure
-       if ( r < 0 ){
+       if (r < 0 ){
            // Option(A) format error
            // Option(B) glibc2.0 bug
            // assume (B).
@@ -182,12 +182,12 @@ membuf_vsprintf( struct membuf *pBuf, const char *fmt, va_list ap )
        }
 
        // don't do this again
-       if ( grew ){
+       if (grew ){
            r = -1;
            break;
        }
        grew = 1;
-       pBuf = membuf_grow( pBuf, r );
+       pBuf = membuf_grow(pBuf, r );
        if (pBuf == NULL){
            // grow failed
            r = -1;
@@ -198,33 +198,33 @@ membuf_vsprintf( struct membuf *pBuf, const char *fmt, va_list ap )
 }
 
 struct membuf *
-membuf_strcat( struct membuf *pBuf, const char *pStr )
+membuf_strcat(struct membuf *pBuf, const char *pStr )
 {
-    return membuf_append( pBuf, pStr, strlen( pStr ) );
+    return membuf_append(pBuf, pStr, strlen(pStr ) );
 }
 
 struct membuf *
-membuf_append( struct membuf *pBuf, const void *pData, size_t len )
+membuf_append(struct membuf *pBuf, const void *pData, size_t len )
 {
     size_t sa;
     int r;
 
     // how much room is there?
-    sa = space_avail( pBuf );
+    sa = space_avail(pBuf );
 
     // will it fit?
-    if ( sa < len ){
+    if (sa < len ){
        // if not, how much do we need?
        r = ((int)(sa - len));
        // do the grow.
-       pBuf = membuf_grow( pBuf, r );
+       pBuf = membuf_grow(pBuf, r );
        // failed?
        if (pBuf == NULL){
            return pBuf;
        }
     }
     // append
-    memcpy( dataend(pBuf),
+    memcpy(dataend(pBuf),
            pData,
            len );
     pBuf->curlen += len;

diff --git a/src/helper/membuf.h b/src/helper/membuf.h
index c1af3ae4eb33d5c51c26da834a78bea6d9d00e1c..0e9d81afcf0fdd4ba13573d95d94bde1af509295 100644 (file)
--- a/src/helper/membuf.h
+++ b/src/helper/membuf.h
@@ -30,7 +30,7 @@ struct membuf *membuf_new(void);
 /** delete (destroy) the mem buffer
  * @param pBuf - buffer to release
  */
-void membuf_delete( struct membuf *pBuf );
+void membuf_delete(struct membuf *pBuf );
 
 
 /** grow/shrink a membuf by specified amount.
@@ -39,14 +39,14 @@ void membuf_delete( struct membuf *pBuf );
  *
  * Symantics of 'realloc()' return NULL on failure
  */
-struct membuf *membuf_grow( struct membuf *pBuf, int amount );
+struct membuf *membuf_grow(struct membuf *pBuf, int amount );
 
 /** how long is this buffer (memlen(), strlen())
  * @param pBuf - the buffer
  * 
  * @returns: length of current buffer.
  */
-size_t membuf_len( struct membuf *pBuf );
+size_t membuf_len(struct membuf *pBuf );
 
 
 /** reset an membuf to zero length.
@@ -54,7 +54,7 @@ size_t membuf_len( struct membuf *pBuf );
  *
  * Note this does not 'release' the memory buffer
  */
-void membuf_reset( struct membuf *pBuf );
+void membuf_reset(struct membuf *pBuf );
 
 
 /** sprintf() to the string buffer
@@ -64,7 +64,7 @@ void membuf_reset( struct membuf *pBuf );
  * Returns 0 on success
  * Returns non-zero on failure
  */
-int membuf_sprintf( struct membuf *pBuf , const char *fmt, ... );
+int membuf_sprintf(struct membuf *pBuf , const char *fmt, ... );
 
 /** vsprintf() to the string buffer
  * @param pBuf - buffer to capture sprintf() data into
@@ -74,7 +74,7 @@ int membuf_sprintf( struct membuf *pBuf , const char *fmt, ... );
  * Returns 0 on success
  * Returns non-zero on failure
  */
-int membuf_vsprintf( struct membuf *pBuf , const char *fmt, va_list ap);
+int membuf_vsprintf(struct membuf *pBuf , const char *fmt, va_list ap);
 
 /** Tokenize lines using strtok() 
  * @param pBuf - buffer to tokenize
@@ -84,14 +84,14 @@ int membuf_vsprintf( struct membuf *pBuf , const char *fmt, va_list ap);
  *
  * NOTE: This call is <b > destructive</b> to the buffer.
  */
-const char *membuf_strtok( struct membuf *pBuf, const char *delim, void **pSave );
+const char *membuf_strtok(struct membuf *pBuf, const char *delim, void **pSave );
 
 /** Return pointer to the memory in the buffer
  * @param pBuf - buffer
  *
  * NOTE: Thou shall not modify this pointer, it is <b > CONST</b>
  */
-const void *membuf_datapointer( struct membuf *pBuf );
+const void *membuf_datapointer(struct membuf *pBuf );
 
 
 /** Append data to the buffer
@@ -102,7 +102,7 @@ const void *membuf_datapointer( struct membuf *pBuf );
  * Modified symantics of "memcpy()".  On memory allocation failure
  * returns NULL.  On success, returns pointer to orginal membuf.
  */
-struct membuf *membuf_append( struct membuf *pBuf, const void *pData, size_t len );
+struct membuf *membuf_append(struct membuf *pBuf, const void *pData, size_t len );
 
 
 /** Append string to the buffer
@@ -112,7 +112,7 @@ struct membuf *membuf_append( struct membuf *pBuf, const void *pData, size_t len
  * Modified symantics of "strcat()".  On memory allocation failure
  * returns NULL.  On success, returns pointer to orginal membuf.
  */
-struct membuf *membuf_strcat( struct membuf *pBuf, const char *s );
+struct membuf *membuf_strcat(struct membuf *pBuf, const char *s );
 
 
 #endif