Re: [PATCH v2] gdbserver: bfin: new port

Mike Frysinger
2010-12-14T17:48:02Z

[-- Attachment #1: Type: Text/Plain, Size: 1452 bytes --]

On Tuesday, December 14, 2010 12:39:27 Pedro Alves wrote:
> On Tuesday 14 December 2010 16:57:48, Mike Frysinger wrote:
> > thinking about it more, the gdb port atm doesnt support FDPIC. so if the
> > only contentious point of the gdbserver code is the FDPIC handling, then
> > i can simply drop that and revisit when the Blackfin/FDPIC core code is
> > merged.
>
> I think that's the best approach. Would it be easy to split
> the linux bits out of gdb/bfin-tdep.c into gdb/bfin-linux-tdep.c?
> You could even push the bare metal (bfin-elf?) bits first without
> any linux bits as a first step.

i imagine for someone who is a gdb expert, this would be trivial. but it
would require a bit of research on my side, so i'd prefer to not do this
initially. the code Joel is OK with supports both Blackfin bare metal and
Blackfin Linux/FLAT.

i'll open a tracker item to get this stuff eventually split out, but i dont
see it being done in a time frame that is before the gdb-7.3 release which i
really want to be a part of.

> Please also get rid of gdb/config/bfin/bfin.mt -- .mt files have
> long since been eliminated from gdb, the code that would go there you
> already have in configure.tgt.

i couldnt find documentation on this file (or mts in general) so i was
hesitant to touch it. the only thing we have in there is to set the TDEPFILES
variable, but i guess that too is no longer used anywhere ?
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]