Re: [patch] document target sim args & arm options

[Matt Rice <ratmice@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 483 bytes --]

On Fri, Mar 26, 2010 at 7:45 AM, Eli Zaretskii <eliz@gnu.org> wrote:
>> Date: Fri, 26 Mar 2010 07:10:07 -0700
>> From: Matt Rice <ratmice@gmail.com>
>>
>> +@item target sim @r{[}@var{simargs}@r{]} @r{...}
>
> This will look better if you use @dots{} instead of @r{...}.
>
> Okay with this change.

Thanks,

attached is an updated patch, also changing another occurance of @r{...}
forgot to mention i don't have commit access, there is a request in the queue,
so whatever is easiest.

[-- Attachment #2: simargs.diff --]
[-- Type: application/octet-stream, Size: 2105 bytes --]

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 4bef3f0..56dbe5d 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -3872,7 +3872,7 @@ A call to @code{exec}.  This is currently only available for HP-UX
 and @sc{gnu}/Linux.
 
 @item syscall
-@itemx syscall @r{[}@var{name} @r{|} @var{number}@r{]} @r{...}
+@itemx syscall @r{[}@var{name} @r{|} @var{number}@r{]} @dots{} 
 @cindex break on a system call.
 A call to or return from a system call, a.k.a.@: @dfn{syscall}.  A
 syscall is a mechanism for application programs to request a service
@@ -14899,7 +14899,7 @@ useful if you have some other way of getting the stub to the target
 system, and you can put it somewhere in memory where it won't get
 clobbered by the download.
 
-@item target sim
+@item target sim @r{[}@var{simargs}@r{]} @dots{}
 @cindex built-in simulator target
 Builtin CPU simulator.  @value{GDBN} includes simulators for most architectures.
 In general,
@@ -17295,6 +17295,25 @@ well as the Angel monitor.
 Show the setting of RDI heartbeat packets.
 @end table
 
+@table @code
+@item target sim @r{[}@var{simargs}@r{]} @dots{} 
+The @value{GDBN} ARM simulator accepts the following optional arguments.
+
+@table @code
+@item --swi-support=@var{type}
+Tell the simulator which SWI interfaces to support.
+@var{type} may be a comma separated list of the following values.
+The default value is @code{all}.
+
+@table @code
+@item none
+@item demon
+@item angel
+@item redboot
+@item all
+@end table
+@end table
+@end table
 
 @node M32R/D
 @subsection Renesas M32R/D and M32R/SDI
diff --git a/gdb/remote-sim.c b/gdb/remote-sim.c
index 9f1adc7..96b45de 100644
--- a/gdb/remote-sim.c
+++ b/gdb/remote-sim.c
@@ -889,7 +889,7 @@ init_gdbsim_ops (void)
 {
   gdbsim_ops.to_shortname = "sim";
   gdbsim_ops.to_longname = "simulator";
-  gdbsim_ops.to_doc = "Use the compiled-in simulator.";
+  gdbsim_ops.to_doc = "Use the compiled-in simulator.  Any arguments will be passed on to the simulator.";
   gdbsim_ops.to_open = gdbsim_open;
   gdbsim_ops.to_close = gdbsim_close;
   gdbsim_ops.to_detach = gdbsim_detach;