qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH V3 08/10] disas/: fix some comment spelling errors


From: Alex Bennée
Subject: Re: [PATCH V3 08/10] disas/: fix some comment spelling errors
Date: Thu, 17 Sep 2020 09:44:07 +0100
User-agent: mu4e 1.5.5; emacs 28.0.50

zhaolichang <zhaolichang@huawei.com> writes:

> I found that there are many spelling errors in the comments of qemu,
> so I used the spellcheck tool to check the spelling errors
> and finally found some spelling errors in the disas folder.
>
> Signed-off-by: zhaolichang <zhaolichang@huawei.com>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

I suggest on your next posting you CC qemu-trivial@nongnu.org so the
patches can get pulled in via the trivial tree.

> ---
>  disas/hppa.c | 2 +-
>  disas/m68k.c | 8 ++++----
>  disas/ppc.c  | 2 +-
>  3 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/disas/hppa.c b/disas/hppa.c
> index 2dbd1fc445..dcf9a47f34 100644
> --- a/disas/hppa.c
> +++ b/disas/hppa.c
> @@ -2021,7 +2021,7 @@ print_insn_hppa (bfd_vma memaddr, disassemble_info 
> *info)
>                       fput_fp_reg (GET_FIELD (insn, 6, 10), info);
>                     break;
>  
> -                   /* 'fA' will not generate a space before the regsiter
> +                   /* 'fA' will not generate a space before the register
>                        name.  Normally that is fine.  Except that it
>                        causes problems with xmpyu which has no FP format
>                        completer.  */
> diff --git a/disas/m68k.c b/disas/m68k.c
> index 863409c67c..aefaecfbd6 100644
> --- a/disas/m68k.c
> +++ b/disas/m68k.c
> @@ -70,7 +70,7 @@ struct floatformat
>    unsigned int exp_start;
>    unsigned int exp_len;
>    /* Bias added to a "true" exponent to form the biased exponent.  It
> -     is intentionally signed as, otherwize, -exp_bias can turn into a
> +     is intentionally signed as, otherwise, -exp_bias can turn into a
>       very large number (e.g., given the exp_bias of 0x3fff and a 64
>       bit long, the equation (long)(1 - exp_bias) evaluates to
>       4294950914) instead of -16382).  */
> @@ -479,7 +479,7 @@ struct m68k_opcode_alias
>        and remaining 3 bits of register shifted 9 bits in first word.
>        Indicate upper/lower in 1 bit shifted 7 bits in second word.
>        Use with `R' or `u' format.
> -   n  `m' withouth upper/lower indication. (For M[S]ACx; 4 bits split
> +   n  `m' without upper/lower indication. (For M[S]ACx; 4 bits split
>        with MSB shifted 6 bits in first word and remaining 3 bits of
>        register shifted 9 bits in first word.  No upper/lower
>        indication is done.)  Use with `R' or `u' format.
> @@ -854,7 +854,7 @@ fetch_arg (unsigned char *buffer,
>  
>  /* Check if an EA is valid for a particular code.  This is required
>     for the EMAC instructions since the type of source address determines
> -   if it is a EMAC-load instruciton if the EA is mode 2-5, otherwise it
> +   if it is a EMAC-load instruction if the EA is mode 2-5, otherwise it
>     is a non-load EMAC instruction and the bits mean register Ry.
>     A similar case exists for the movem instructions where the register
>     mask is interpreted differently for different EAs.  */
> @@ -1080,7 +1080,7 @@ print_indexed (int basereg,
>  
>  /* Returns number of bytes "eaten" by the operand, or
>     return -1 if an invalid operand was found, or -2 if
> -   an opcode tabe error was found.
> +   an opcode table error was found.
>     ADDR is the pc for this arg to be relative to.  */
>  
>  static int
> diff --git a/disas/ppc.c b/disas/ppc.c
> index 63e97cfe1d..02be878198 100644
> --- a/disas/ppc.c
> +++ b/disas/ppc.c
> @@ -5226,7 +5226,7 @@ operand_value_powerpc (const struct powerpc_operand 
> *operand,
>        if ((operand->flags & PPC_OPERAND_SIGNED) != 0)
>       {
>         /* BITM is always some number of zeros followed by some
> -          number of ones, followed by some numer of zeros.  */
> +          number of ones, followed by some number of zeros.  */
>         unsigned long top = operand->bitm;
>         /* top & -top gives the rightmost 1 bit, so this
>            fills in any trailing zeros.  */


-- 
Alex Bennée



reply via email to

[Prev in Thread] Current Thread [Next in Thread]