Re: [PATCH v5 2/4] gdb: provide openrisc target description XML files.

[Stafford Horne <shorne@gmail.com>]

On Tue, Apr 04, 2017 at 10:25:51PM +0100, Yao Qi wrote:
> Stafford Horne <shorne@gmail.com> writes:
> 
> > diff --git a/gdb/features/or1k.xml b/gdb/features/or1k.xml
> > new file mode 100644
> > index 0000000..688b340
> > --- /dev/null
> > +++ b/gdb/features/or1k.xml
> > @@ -0,0 +1,2274 @@
> > +<?xml version="1.0"?>
> > +<!-- Copyright (C) 2016 Free Software Foundation, Inc.
> > +
> > +     Copying and distribution of this file, with or without modification,
> > +     are permitted in any medium without royalty provided the copyright
> > +     notice and this notice are preserved.  -->
> > +
> > +<!DOCTYPE target SYSTEM "gdb-target.dtd">
> > +<target>
> > +  <architecture>or1k</architecture>
> > +  <feature name="org.gnu.gdb.or1k.group0">
> 
> 11 features are defined, but your patch 1/4 only checks feature group0.
> Does it indicate that only group0 is a required feature?  If so, we need
> to document it here,
> https://sourceware.org/gdb/current/onlinedocs/gdb/Standard-Target-Features.html
> 
> Of course, the debugging stub,like OpenOCD, can provide more features
> than GDB required.  In GDB, we need only to define required features.

I have removed these 11 features and just left the 1.  I think its
better that the targets provide all the extras.

Also, I documented it.  I will send the patches.

> -- 
> Yao (齐尧)