This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v6 2/5] gdb: Add OpenRISC or1k and or1knd target support
- From: Stafford Horne <shorne at gmail dot com>
- To: Yao Qi <qiyaoltc at gmail dot com>
- Cc: GDB patches <gdb-patches at sourceware dot org>, Openrisc <openrisc at lists dot librecores dot org>, Franck Jullien <franck dot jullien at gmail dot com>
- Date: Tue, 16 May 2017 20:18:28 +0900
- Subject: Re: [PATCH v6 2/5] gdb: Add OpenRISC or1k and or1knd target support
- Authentication-results: sourceware.org; auth=none
- References: <cover.1493038197.git.shorne@gmail.com> <cover.1493038197.git.shorne@gmail.com> <ae4e2c7e3e9d364bf73adba1eedaeb3f4691d907.1493038197.git.shorne@gmail.com> <864lx38h2i.fsf@gmail.com> <20170502155305.GI2724@lianli.shorne-pla.net> <86shket8tx.fsf@gmail.com>
On Tue, May 09, 2017 at 03:15:38PM +0100, Yao Qi wrote:
> Stafford Horne <shorne@gmail.com> writes:
>
> > I understand this is nothing new, but this tells to the users some extra
> > details about using target remote for OpenRISC (example which platforms
> > support it). Also, this is consistent with some other targets like
> > Microblaze.
> >
>
> All the targets listed in "Embedded Processors" do have "target
> remote". We don't have to document it for every one. IMO, we need to
> remove "target remote" from Microblaze part too.
Sure I will remove. Also I am adding 'set debug or1k' and 'show debug
or1k'.
> >> > +
> >> > +@kindex target sim
> >> > +@item target sim
> >> > +
> >> > +Runs the builtin CPU simulator which can run very basic
> >> > +programs but does not support most hardware functions like MMU.
> >> > +For more complex use cases the user is advised to run an external
> >> > +target, and connect using @samp{target remote}.
> >> > +
> >> > +Example: @code{target sim}
> >> > +
> >> > +@end table
> >> > +
> >> > @node PowerPC Embedded
> >> > @subsection PowerPC Embedded
> >> >
> >> > @@ -41088,6 +41131,7 @@ registers using the capitalization used in
> >> > the description.
> >> > * M68K Features::
> >> > * NDS32 Features::
> >> > * Nios II Features::
> >> > +* OpenRISC 1000 Features::
> >> > * PowerPC Features::
> >> > * S/390 and System z Features::
> >> > * Sparc Features::
> >> > @@ -41374,6 +41418,32 @@ targets. It should contain the 32 core
> >> > registers (@samp{zero},
> >> > @samp{pc}, and the 16 control registers (@samp{status} through
> >> > @samp{mpuacc}).
> >> >
> >> > +@node OpenRISC 1000 Features
> >> > +@subsection Openrisc 1000 Features
> >> > +@cindex target descriptions, OpenRISC 1000 features
> >> > +
> >> > +The @samp{org.gnu.gdb.or1k.group0} feature is required for OpenRISC 1000
> >> > +targets. It should contain the 32 general purpose registers (@samp{r0}
> >> > +through @samp{r31}), @samp{ppc}, @samp{npc} and @samp{sr}.
> >> > +
> >> > +Along with the default reggroups like @samp{system} and @samp{general}
> >> > +provided by @value{GDBN}, OpenRISC targets can use the following reggroups
> >> > +to group their many registers:
> >> > +
> >> > +@smallexample
> >> > + Group Type
> >> > + immu user
> >> > + dmmu user
> >> > + icache user
> >> > + dcache user
> >> > + pic user
> >> > + timer user
> >> > + power user
> >> > + perf user
> >> > + mac user
> >> > + debug user
> >> > +@end smallexample
> >> > +
> >>
> >> Why do you need to document the reggroups?
> >
> > These register groups can be used by the target description features. If
> > not documented one would have to look into the code. In general arbitrary
> > groups are not allowed by features. This is also related to patch 1/5.
>
> "maintenance print reggroups" can tell the reggroups, so don't need to
> document them.
Sure, I will remove.
> --
> Yao (齐尧)