This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[RFC] New port: CR16: gdb


Hi,
Please find attached an updated patch for the gdb port of the CR16 target.
This has been updated from my previous attempts of the same,
http://sourceware.org/ml/gdb-patches/2013-06/msg00489.html

This patch incorporates changes suggested by Pedro,
http://sourceware.org/ml/gdb-patches/2013-06/msg00734.html

> isn't exposing r0_orig to gDB necessary for syscall restarting,
Added this register using separate target description feature,
added in features/cr16-linux.xml. This will generate the cr16-linux.dat
in gdb/regformats which will be used by the gdbserver.

>> with the current simulator port which already supports them.
> Are these always present in all versions of CR16 silicon?  IOW, are we
> safe with adding them to the core register set (*)?  Otherwise, you
> should really split them to a separate target description feature.
Added new target description which supports these additional debug registers
under features/cr16-dbg.xml The tdep files have been updated to lookup these
features and update their register sets accordingly.

>> * configure.ac : Add support for cr16-*-*
> No space before ':'.
> Full stop and double space.
>> +/* The breakpoint instruction used by uClinux target  */
> Ditto.  "by the uClinux".
>> "on _the_ CR16 target".
>> = goes on the next line.
>> +  if (readbuf)
>  if (readbuf != NULL)
>> +  if (writebuf)
> if (writebuf != NULL)
> Nope, this is not the GNU/Linux file.  Also spurious double space

Fixed the above typos and formatting.

I will also be posting an updated gdbserver port which will compliment
this port.

There are changes to configure.ac here. I will copy these to
gcc-patches, once approved.
Kindly review and let me know if OK.

Regards,
Kaushik

2013-09-04 Kaushik Phatak  <kaushik.phatak@kpitcummins.com>
    gdb/Changelog
    * configure.tgt: Handle cr16*-*-*linux and cr16*-*-*.
    * cr16-linux-tdep.c: New file.
    * cr16-tdep.c: New file.
    * cr16-tdep.h: New file.
    * configure.ac: Add support for cr16-*-*
    * configure: Regenerate.
    * features/cr16-linux-core.xml: New.
    * features/cr16-linux.xml: New.
    * features/cr16-core.xml: New.
    * features/cr16-dbg.xml: New.
    * features/cr16-linux.c: New (autogenerated).
    * features/cr16-dbg.c: New (autogenerated).

Attachment: cr16_gdb.diff
Description: Binary data


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]