[PATCH v1 2/3] doc: develop: Fix typos and wording in gdb.rst

Lothar Rubusch l.rubusch at gmail.com
Wed Nov 20 11:25:10 CET 2024


Hi, thank you all! This is outstanding, such a fast feedback! As I
also just ran a tool over the texts, adjusted formatting and
reverted/accepted some changes manually to my knowledge (I'm not a
native speaker).

On Tue, Nov 19, 2024 at 11:48 PM Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> Am 19. November 2024 23:28:36 MEZ schrieb Lothar Rubusch <l.rubusch at gmail.com>:
> >Fix some typos and duplicate words in gdb.rst.
[...]
> >     CONFIG_CC_OPTIMIZE_FOR_DEBUG=y
> >@@ -24,8 +24,8 @@ Otherwise build, install, and run U-Boot as usual.
> > Using OpenOCD as GDB server
> > ---------------------------
> >
> >-`OpenOCD <https://openocd.org/>`_ is an open source tool supporting hardware
> >-debug probes, and providing a GDB server. It is readily available in major Linux
> >+`OpenOCD <https://openocd.org/>`_ is an open-source tool supporting hardware
> >+debug probes and provide a GDB server. It is readily available in major Linux
>
> Thanks for the patch.
>
> 'tool' is singular. You cannot use 'provide' here.

Ok.

> > distributions or you can build it from source.
> >
> > Here is example of starting OpenOCD on Debian using a J-Link adapter and a
> >@@ -124,7 +124,7 @@ with the command *bdinfo*:
> >     Early malloc usage: cd8 / 2000
> >
> > Look out for the line starting with *relocaddr* which has the address
> >-you need, ``0x27f7a000`` in this case.
> >+You need, ``0x27f7a000`` in this case.
>
> You is only capitalized at the start of sentences.

Ok.

> >
> > On most architectures (not sandbox, x86, Xtensa) the global data pointer is
> > stored in a fixed register:
> >@@ -144,7 +144,7 @@ riscv        gp
> > sh           r13
> > ============ ========
> >
> >-On these architecture the relocation address cat be determined by
> >+On this architecture the relocation address can be determined by
>
> On these architectures ...

Ok.


More information about the U-Boot mailing list