2012/12/13 Spencer Oliver <[email protected]>:
> 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.
>

Ok thanks.

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

Yes I can handle this. I keep you posted.

> 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

Reply via email to