Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread David Brownell 
On Tuesday 23 June 2009, Duane Ellis wrote:
  Author: duane
  Date: 2009-06-24 04:01:14 +0200 (Wed, 24 Jun 2009)
  New Revision: 2383

I get all kinds of build errors on Ubuntu 9.04/x86_32 where the
chip details banks get initialized.  The errors made no sense to
me, and they went away when I changed the

.bank[0] = { ... },
.bank[1] = { ... },

to be

.bank = {{ ... }, { ... }},

Also, that sam3 flash file has lots of whitespace issues; Zach,
maybe you could run your scripts on it?  My quickie fixes gave

 src/flash/at91sam3.c | 1139 -
 1 file changed, 564 insertions(+), 575 deletions(-)

totalling about 70 KB of fixups on that file.

- Dave

___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread David Brownell 
On Wednesday 24 June 2009, David Brownell wrote:
 I get all kinds of build errors on Ubuntu 9.04/x86_32 where the
 chip details banks get initialized.  The errors made no sense to
 me, and they went away when I changed the
 
 .bank[0] = { ... },
 .bank[1] = { ... },
 
 to be
 
 .bank = {{ ... }, { ... }},
 

Ditto 8.04.2/x86_64 ...

at91sam3.c:316: warning: initialized field overwritten
at91sam3.c:316: warning: (near initialization for ‘all_sam3_details[0].bank’)
at91sam3.c:353: warning: initialized field overwritten
at91sam3.c:353: warning: (near initialization for ‘all_sam3_details[1].bank’)
at91sam3.c:391: warning: initialized field overwritten
at91sam3.c:391: warning: (near initialization for ‘all_sam3_details[2].bank’)
at91sam3.c:443: warning: initialized field overwritten
at91sam3.c:443: warning: (near initialization for ‘all_sam3_details[3].bank’)
at91sam3.c:480: warning: initialized field overwritten
at91sam3.c:480: warning: (near initialization for ‘all_sam3_details[4].bank’)
at91sam3.c:518: warning: initialized field overwritten
at91sam3.c:518: warning: (near initialization for ‘all_sam3_details[5].bank’)
make[3]: *** [at91sam3.lo] Error 1

___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread Duane Ellis 
Zach Welch wrote:
 Fixed.  There were a few could be used uninitialized warnings too.
 These seem like they should have been covered by --enable-werror.

 Duane, are you building with that option before committing changes?
   

I normally build with optimization disabled so that I can *debug* the 
code - with the optimizer disabled, the uninitialized errors will not occur.

-Duane.

___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread Duane Ellis 
David Brownell wrote:
 I get all kinds of build errors on Ubuntu 9.04/x86_32 where the
 chip details banks get initialized.  The errors made no sense to
 me, and they went away when I changed the

   .bank[0] = { ... },
   .bank[1] = { ... },

 to be

   .bank = {{ ... }, { ... }},

   
i thought the bank[0] = {  } was valid C99 initialization.

I've been using cygwin, by default uses:

/usr/lib/gcc/i686-pc-cygwin/3.4.4/specs

Perhaps something has changed I am unaware of .

-Duane.

___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread Zach Welch 
On Wed, 2009-06-24 at 09:09 -0400, Duane Ellis wrote:
 Zach Welch wrote:
  Fixed.  There were a few could be used uninitialized warnings too.
  These seem like they should have been covered by --enable-werror.
 
  Duane, are you building with that option before committing changes?

 
 I normally build with optimization disabled so that I can *debug* the 
 code - with the optimizer disabled, the uninitialized errors will not occur.

Thank you!  That explains it completely.  I was a bit confused.

--Z
___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread Zach Welch 
On Wed, 2009-06-24 at 09:13 -0400, Duane Ellis wrote:
 David Brownell wrote:
  I get all kinds of build errors on Ubuntu 9.04/x86_32 where the
  chip details banks get initialized.  The errors made no sense to
  me, and they went away when I changed the
 
  .bank[0] = { ... },
  .bank[1] = { ... },
 
  to be
 
  .bank = {{ ... }, { ... }},
 

 i thought the bank[0] = {  } was valid C99 initialization.
 
 I've been using cygwin, by default uses:
 
 /usr/lib/gcc/i686-pc-cygwin/3.4.4/specs
 
 Perhaps something has changed I am unaware of .

I just realized that I left out the '.bank = ' bits.

Sorry for that minor mess, but it compiled.  :)

--Z
___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread Duane Ellis 
David Brownell wrote:
  I get all kinds of build errors on Ubuntu 9.04/x86_32 where the
  [snip]
  .bank[0] = { ... },

duane  i thought the bank[0] = {  } was valid C99 initialization.

zach   I just realized that I left out the '.bank = ' bits.
zach  Sorry for that minor mess, but it compiled. :)

Huh? What do you mean? Confused.

-Duane.

___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread Zach Welch 
On Wed, 2009-06-24 at 09:56 -0400, Duane Ellis wrote:
 David Brownell wrote:
   I get all kinds of build errors on Ubuntu 9.04/x86_32 where the
   [snip]
   .bank[0] = { ... },
 
 duane  i thought the bank[0] = {  } was valid C99 initialization.
 
 zach   I just realized that I left out the '.bank = ' bits.
 zach  Sorry for that minor mess, but it compiled. :)
 
 Huh? What do you mean? Confused.

Well, I changed 

 .bank[0] = { ... },
 .bank[1] = { ... },

to 

___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-24 Thread Zach Welch 
On Wed, 2009-06-24 at 07:28 -0700, Zach Welch wrote:
 On Wed, 2009-06-24 at 09:56 -0400, Duane Ellis wrote:
  David Brownell wrote:
I get all kinds of build errors on Ubuntu 9.04/x86_32 where the
[snip]
.bank[0] = { ... },
  
  duane  i thought the bank[0] = {  } was valid C99 initialization.
  
  zach   I just realized that I left out the '.bank = ' bits.
  zach  Sorry for that minor mess, but it compiled. :)
  
  Huh? What do you mean? Confused.
 
 Well, I changed 
 
  .bank[0] = { ... },
  .bank[1] = { ... },
 
 to 
 

{ { ... }, { ... } }.

but it should have been:

.bank = { { ... }, { ... } }

See?  I missed the '.bank =' part  and I hit send accidentally
before finishing this message.   Sorry.

Cheers,

Zach
___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

[Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-23 Thread Duane Ellis 
SVN Commit -

 Author: duane
 Date: 2009-06-24 04:01:14 +0200 (Wed, 24 Jun 2009)
 New Revision: 2383


OpenOCD now supports the AT91SAM3 - new CortexM3 product from Atmel.

Adds support for the FLASH component, includes DOCs for the new target 
in the OpenOCD.texi file.

The openocd.cfg file I used is/was:

#debug_level 3
source [find interface/olimex-jtag-tiny.cfg]

jtag_speed 6
telnet_port 
gdb_port 

source [find board/atmel_sam3u_ek.cfg]
$_TARGETNAME configure -event gdb-flash-erase-start {
halt
}

init
halt


___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] New Flash Target: Atmel AT91SAM3 - Cortex M3

2009-06-23 Thread David Brownell 
On Tuesday 23 June 2009, Duane Ellis wrote:
 SVN Commit -
 
  Author: duane
  Date: 2009-06-24 04:01:14 +0200 (Wed, 24 Jun 2009)
  New Revision: 2383
 
 
 OpenOCD now supports the AT91SAM3 - new CortexM3 product from Atmel.

Great!  Now, where can we get these boards?

A Very Nice  thing here:  high speed USB support.

Just perfect for a smart high speed JTAG adapter.  ;)

- dave
___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development