On 13 December 2012 14:27, Franck Jullien <[email protected]> wrote: > Hi, > > I could find a way to post my reply on gerrit....
To reply to comments inline just goto the file and click the comment to reply to. http://openocd.zylin.com/#/c/1051/1/src/server/gdb_server.c To add a comment double click on the line. > > 2012/12/13 Spencer Oliver (Code Review) <[email protected]>: >> Spencer Oliver has posted comments on this change. >> >> Change subject: gdb_server: Add gdb_tdesc_path cmd and qXfer:features:read >> support v2 >> ...................................................................... >> >> >> Patch Set 1: (2 inline comments) >> >> Like Mathias patch this only solves half the problem, do you have plans to >> sort the other bit, eg. setting the correct target registers depending on >> arch. >> >> Until that is done i do not understand the value in this patch, please >> correct me if i am missing something. Do we not get the same behaviour by >> using "set tdesc" in gdb. >> > > Yes it's the same thing... But, as it is supported by gdb, why don't > implement it in openocd ? > > However, your idea is very good. > It would be a very good thing to have a dynamically created tdesc file > built from the arch registers list (is that what you meant ?). I suppose we have two possible ways, create the target registers from the gdb xml or create the gdb xml from the target registers. Probably easier todo the latter. > I can work on it. > This is something i have been meaning todo for a while, but would be great if you can find the time. Cheers Spen ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
