Re: gdb support for Atmel AVR

[Daniel Jacobowitz <drow@mvista.com>]

On Fri, Feb 08, 2002 at 10:59:58AM -0700, Theodore A. Roth wrote:
> :)
> :)The one thing you'll notice isn't on the check list is test results.  On 
> :)that count, far as I'm conceerned, as soon as your target is showing 
> :)reasonable signs of life it is ready for acceptance.
> 
> Some people are already using it for real work so I think it's got a heart 
> beat already. ;)
> 
> I'm especially interested in someone reviewing the changes I had to make 
> outside of avr-tdep.c and config/avr/*. These changes may affect other 
> targets. I've tried really hard to keep those changes to a minimum though.

I really do not think that TARGET_REMOTE_ADDR_BIT should be
necessary... in what way was TARGET_ADDR_BIT/TARGET_POINTER_BIT
inadequate?  Do you have different sized code and data pointers?


Oh, reading further down the patch I see that AVR is a Harvard
architecture.  There is support for this in GDB, with
CODE_SPACE/DATA_SPACE that were recently introduced (for the d10v, I
think).  You may have some problems if they are of different size, I
suppose.



Also, I think (.avrgdbinit aside...) that you should not have a
tm-avr.h at all.  You can set multi-arch from configure.tgt.

-- 
Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer