Thanks Utkarsh,

I have comments below.

On Sun, Mar 22, 2020 at 4:53 AM utkarsh.ra...@gmail.com
<utkarsh.ra...@gmail.com> wrote:
>
> ---
>  user/start/tools.rst | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
> diff --git a/user/start/tools.rst b/user/start/tools.rst
> index c3f039b..8a1f393 100644
> --- a/user/start/tools.rst
> +++ b/user/start/tools.rst
> @@ -76,3 +76,28 @@ source code used.
>
>
>  Add ``--verbose`` to the GCC command for the the verbose version details.
> +
> +RTEMS cross-compiler over GCC cross-compiler
It is still a gcc cross compiler. I don't know that I like the heading
here. Maybe "Need for RTEMS-Specific Cross-Compiler" is better?

> +--------------------------------------------
> +    The new users are often confused as to why they can't use their 
> distributions cross-compiler for their target on rtems, e.g.,
Line length. Please limit to 80 characters. This is a general comment
that I won't repeat below.

Remove "The", start with "New users"
distributions -> distribution's

> +    the riscv64-linux-gnu or the arm-none-eabi-gcc.Below mentioned are some 
> the reasons for using the RTEMS cross-compiler.
add space in "gcc.Below"
"some the" -> "some of the"

> +
> + ``Correct configuration of newlib`` -
> +    Newlib is a C standard library implementation intended
Rebalance line lengths. You can probably set up your editor to do this
automatically. That is the best choice when working with narrative
text. (With code, you should deal with line breaks manually.)

> +    for use on embedded systems. Most of the POSIX and libc support for 
> RTEMS is derived from newlib.
> +    The RTEMS cross-compiler configures newlib correctly for RTEMS.
> +
> + ``Threading in GCC supported libraries`` -
supported -> support

> +    GCC supported threading libraries such as ``pthread.h``
ditto

> +    provide threading support to an application,these libraries are tailored 
> according to RTEMS using
add space after comma
Replace comma with period or semicolon

> +    the RTEMS cross-compiler.
> +
> + ``Provide preprocessor define _rtems_`` -
Two underscores __rtems__ here.

> +    The ``_rtems_`` preprocessor define is genarally used to include rtems
typo: generally

> +    specific libraries.This preprocessor define is provided by the RTEMS 
> cross-compiler at the compile time.

It is used to provide conditional code compilation in source files
that are shared with other projects, e.g., in newlib or imported code
from freebsd.


> +
> + ``Multilib variants to match the BSP``-
> +    RTEMS configures GCC to create separate runtime libraries for
> +    each   supported instruction set, floating point unit, vector unit, word 
> size (e.g. 32-bit and 64-bit), endianess,
too  many spaces after "each"
typo: endianness has two 'n'

> +    code model, ABI, processor errata workarounds, and so on in the 
> architecture. This is termed multilib.
what is "code model"?

"This is term multilib." -> "These libraries are termed multilib variants."

> +    Multilibs variants to match the BSP are set by selecting a specific set 
> of machine options using the RTEMS cross-compiler.
> --
> 2.17.1
>
> _______________________________________________
> devel mailing list
> devel@rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to