Patchwork arm: translate: comment typo - s/middel/middle/

login
register
mail settings
Submitter Peter A. G. Crosthwaite
Date Aug. 6, 2012, 7:05 a.m.
Message ID <1344236756-9660-1-git-send-email-peter.crosthwaite@petalogix.com>
Download mbox | patch
Permalink /patch/175277/
State New
Headers show

Comments

Peter A. G. Crosthwaite - Aug. 6, 2012, 7:05 a.m.
Signed-off-by: Peter A. G. Crosthwaite <peter.crosthwaite@petalogix.com>
---
 target-arm/translate.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Andreas Färber - Aug. 6, 2012, 2:12 p.m.
Am 06.08.2012 09:05, schrieb Peter A. G. Crosthwaite:
> Signed-off-by: Peter A. G. Crosthwaite <peter.crosthwaite@petalogix.com>
> ---
>  target-arm/translate.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/target-arm/translate.c b/target-arm/translate.c
> index 29008a4..494c682 100644
> --- a/target-arm/translate.c
> +++ b/target-arm/translate.c
> @@ -9892,7 +9892,7 @@ static inline void gen_intermediate_code_internal(CPUARMState *env,
>      } else {
>          /* While branches must always occur at the end of an IT block,
>             there are a few other things that can cause us to terminate
> -           the TB in the middel of an IT block:
> +           the TB in the middle of an IT block:
>              - Exception generating instructions (bkpt, swi, undefined).
>              - Page boundaries.
>              - Hardware watchpoints.

FWIW

Reviewed-by: Andreas Färber <afaerber@suse.de>

/-F
Peter Maydell - Aug. 6, 2012, 2:17 p.m.
On 6 August 2012 08:05, Peter A. G. Crosthwaite
<peter.crosthwaite@petalogix.com> wrote:
> Signed-off-by: Peter A. G. Crosthwaite <peter.crosthwaite@petalogix.com>
> ---
>  target-arm/translate.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/target-arm/translate.c b/target-arm/translate.c
> index 29008a4..494c682 100644
> --- a/target-arm/translate.c
> +++ b/target-arm/translate.c
> @@ -9892,7 +9892,7 @@ static inline void gen_intermediate_code_internal(CPUARMState *env,
>      } else {
>          /* While branches must always occur at the end of an IT block,
>             there are a few other things that can cause us to terminate
> -           the TB in the middel of an IT block:
> +           the TB in the middle of an IT block:
>              - Exception generating instructions (bkpt, swi, undefined).
>              - Page boundaries.
>              - Hardware watchpoints.
> --
> 1.7.0.4
>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Happy for this to go via the trivial queue.

-- PMM
Stefan Hajnoczi - Aug. 10, 2012, 1:35 p.m.
On Mon, Aug 06, 2012 at 05:05:56PM +1000, Peter A. G. Crosthwaite wrote:
> Signed-off-by: Peter A. G. Crosthwaite <peter.crosthwaite@petalogix.com>
> ---
>  target-arm/translate.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Also congrats on getting two Reviewed-bys.  That's a rate of 1
Reviewed-by per changed character and the most thorough review a QEMU
patch has ever had! :)

Stefan

Patch

diff --git a/target-arm/translate.c b/target-arm/translate.c
index 29008a4..494c682 100644
--- a/target-arm/translate.c
+++ b/target-arm/translate.c
@@ -9892,7 +9892,7 @@  static inline void gen_intermediate_code_internal(CPUARMState *env,
     } else {
         /* While branches must always occur at the end of an IT block,
            there are a few other things that can cause us to terminate
-           the TB in the middel of an IT block:
+           the TB in the middle of an IT block:
             - Exception generating instructions (bkpt, swi, undefined).
             - Page boundaries.
             - Hardware watchpoints.