[0/6] target/arm: general cleanups

Message

Fabiano Rosas Dec. 13, 2022, 7:05 p.m. UTC

Hi folks,

I'm splitting this big series from Claudio from last year into more
manageable chunks (at least for me):

https://lore.kernel.org/r/[e-mail]

This is the first chunk with only the most trivial patches that make
sense even without the kvm/tcg and sysemu/user splits.

Claudio Fontana (1):
  target/arm: cleanup cpu includes

Fabiano Rosas (5):
  target/arm: Fix checkpatch comment style warnings in helper.c
  target/arm: Fix checkpatch space errors in helper.c
  target/arm: Fix checkpatch brace errors in helper.c
  target/arm: Remove unused includes from m_helper.c
  target/arm: Remove unused includes from helper.c

 target/arm/cpu.c      |   1 -
 target/arm/cpu64.c    |   6 -
 target/arm/helper.c   | 439 ++++++++++++++++++++++++++----------------
 target/arm/m_helper.c |  16 --
 4 files changed, 278 insertions(+), 184 deletions(-)

Comments

Claudio Fontana Dec. 14, 2022, 11:48 a.m. UTC | #1

On 12/13/22 20:05, Fabiano Rosas wrote:
> Hi folks,
> 
> I'm splitting this big series from Claudio from last year into more
> manageable chunks (at least for me):
> 
> https://lore.kernel.org/r/[e-mail]
> 
> This is the first chunk with only the most trivial patches that make
> sense even without the kvm/tcg and sysemu/user splits.
> 
> Claudio Fontana (1):
>   target/arm: cleanup cpu includes
> 
> Fabiano Rosas (5):
>   target/arm: Fix checkpatch comment style warnings in helper.c
>   target/arm: Fix checkpatch space errors in helper.c
>   target/arm: Fix checkpatch brace errors in helper.c
>   target/arm: Remove unused includes from m_helper.c
>   target/arm: Remove unused includes from helper.c
> 
>  target/arm/cpu.c      |   1 -
>  target/arm/cpu64.c    |   6 -
>  target/arm/helper.c   | 439 ++++++++++++++++++++++++++----------------
>  target/arm/m_helper.c |  16 --
>  4 files changed, 278 insertions(+), 184 deletions(-)
> 

For the series:

Reviewed-by: Claudio Fontana <[e-mail]>

Cornelia Huck Dec. 14, 2022, 2:30 p.m. UTC | #2

On Tue, Dec 13 2022, Fabiano Rosas <[e-mail]> wrote:

> Hi folks,
>
> I'm splitting this big series from Claudio from last year into more
> manageable chunks (at least for me):
>
> https://lore.kernel.org/r/[e-mail]
>
> This is the first chunk with only the most trivial patches that make
> sense even without the kvm/tcg and sysemu/user splits.
>
> Claudio Fontana (1):
>   target/arm: cleanup cpu includes
>
> Fabiano Rosas (5):
>   target/arm: Fix checkpatch comment style warnings in helper.c
>   target/arm: Fix checkpatch space errors in helper.c
>   target/arm: Fix checkpatch brace errors in helper.c
>   target/arm: Remove unused includes from m_helper.c
>   target/arm: Remove unused includes from helper.c
>
>  target/arm/cpu.c      |   1 -
>  target/arm/cpu64.c    |   6 -
>  target/arm/helper.c   | 439 ++++++++++++++++++++++++++----------------
>  target/arm/m_helper.c |  16 --
>  4 files changed, 278 insertions(+), 184 deletions(-)

For the series

Reviewed-by: Cornelia Huck <[e-mail]>

Peter Maydell Jan. 5, 2023, 12:29 p.m. UTC | #3

On Tue, 13 Dec 2022 at 19:08, Fabiano Rosas <[e-mail]> wrote:
>
> Hi folks,
>
> I'm splitting this big series from Claudio from last year into more
> manageable chunks (at least for me):
>
> https://lore.kernel.org/r/[e-mail]
>
> This is the first chunk with only the most trivial patches that make
> sense even without the kvm/tcg and sysemu/user splits.



Applied to target-arm.next, thanks.

-- PMM