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] |
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] |