[Qemu-commits] [qemu/qemu] 8d2b06: target/sh4: Fix code style for checkp

qemu-commits
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 8d2b06: target/sh4: Fix code style for checkp

From:	Peter Maydell
Subject:	[Qemu-commits] [qemu/qemu] 8d2b06: target/sh4: Fix code style for checkpatch.pl
Date:	Mon, 08 Mar 2021 07:51:28 -0800
  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 8d2b06fbc2967b0e53d86e24e572fb58dcb59ad7
      
https://github.com/qemu/qemu/commit/8d2b06fbc2967b0e53d86e24e572fb58dcb59ad7
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M target/sh4/helper.c

  Log Message:
  -----------
  target/sh4: Fix code style for checkpatch.pl

We are going to move this code, fix its style first.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20210127232151.3523581-2-f4bug@amsat.org>


  Commit: 53e047c2db0aba27e873c99bc789120157548203
      
https://github.com/qemu/qemu/commit/53e047c2db0aba27e873c99bc789120157548203
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M target/sh4/helper.c

  Log Message:
  -----------
  target/sh4: Replace magic value by MMUAccessType definitions

Replace the 0/1/2 magic values by the corresponding MMUAccessType.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20210127232151.3523581-3-f4bug@amsat.org>


  Commit: 31ffda71338348915f54e997edc12d9e30425438
      
https://github.com/qemu/qemu/commit/31ffda71338348915f54e997edc12d9e30425438
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M target/sh4/helper.c

  Log Message:
  -----------
  target/sh4: Remove unused 'int access_type' argument

get_mmu_address() and get_physical_address() don't use their
'int access_type' argument: remove it along with ACCESS_INT
in superh_cpu_tlb_fill().

Suggested-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: be617b44fef9747f0529234ad4d1dfbc88971e30
      
https://github.com/qemu/qemu/commit/be617b44fef9747f0529234ad4d1dfbc88971e30
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M target/sh4/helper.c

  Log Message:
  -----------
  target/sh4: Let get_physical_address() use MMUAccessType access_type

superh_cpu_tlb_fill() already provides a access_type variable of
type MMUAccessType, and it is passed along, but casted as integer
and renamed 'rw'.
Simply replace 'int rw' by 'MMUAccessType access_type'.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20210127232151.3523581-5-f4bug@amsat.org>


  Commit: f32393ac0abe33525d9496737fdb37e9016256ba
      
https://github.com/qemu/qemu/commit/f32393ac0abe33525d9496737fdb37e9016256ba
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M target/sh4/cpu.h

  Log Message:
  -----------
  target/sh4: Remove unused definitions

Remove these confusing and unused definitions.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20210127232151.3523581-6-f4bug@amsat.org>


  Commit: d044adc21ac6088a414bc6a32a598d9ad89f8016
      
https://github.com/qemu/qemu/commit/d044adc21ac6088a414bc6a32a598d9ad89f8016
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M hw/block/tc58128.c
    M include/hw/sh4/sh.h

  Log Message:
  -----------
  hw/sh4: Add missing license

This code was introduced in commit 27c7ca7e775,
("SHIX board emulation (Samuel Tardieu)"). Use
the same license.

Cc: Samuel Tardieu <sam@rfc1149.net>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-2-f4bug@amsat.org>


  Commit: c3c7153f95ed48c0491a70d3be1f11990752fd04
      
https://github.com/qemu/qemu/commit/c3c7153f95ed48c0491a70d3be1f11990752fd04
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M hw/sh4/Kconfig

  Log Message:
  -----------
  hw/sh4: Add missing Kconfig dependency on SH7750 for the R2D board

r2d_init() calls sh7750_init() so depends on SH7750.Harmless
at the moment because nothing actually uses CONFIG_SH7750
(hw/sh4/meson.build always compiles sh7750.c and sh7750_regnames.c
unconditionally).

Fixes: 7ab58d4c841 ("sh4-softmmu.mak: express dependencies with Kconfig")
Reported-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-3-f4bug@amsat.org>


  Commit: 5d07a6985a80b20d5857825c6e5804ba5a2542d1
      
https://github.com/qemu/qemu/commit/5d07a6985a80b20d5857825c6e5804ba5a2542d1
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M hw/intc/Kconfig
    M hw/intc/meson.build
    M hw/sh4/Kconfig

  Log Message:
  -----------
  hw/intc: Introduce SH_INTC Kconfig entry

We want to be able to use the 'SH4' config for architecture
specific features. Add more fine-grained selection by adding
a CONFIG_SH_INTC selector for the SH4 interrupt controller.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-4-f4bug@amsat.org>


  Commit: 7ae5c8bb6e57b08927a0f9ecae610abda0484ec6
      
https://github.com/qemu/qemu/commit/7ae5c8bb6e57b08927a0f9ecae610abda0484ec6
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M MAINTAINERS
    M hw/char/Kconfig
    M hw/char/meson.build
    M hw/sh4/Kconfig

  Log Message:
  -----------
  hw/char: Introduce SH_SCI Kconfig entry

We want to be able to use the 'SH4' config for architecture
specific features. Add more fine-grained selection by adding
a CONFIG_SH_SCI selector for the SH4 serial controller.

Add the missing MAINTAINERS entries.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-5-f4bug@amsat.org>


  Commit: 25e79527a882eebdbbe8b7408d17f81a29de7d4b
      
https://github.com/qemu/qemu/commit/25e79527a882eebdbbe8b7408d17f81a29de7d4b
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M MAINTAINERS
    M hw/sh4/Kconfig
    M hw/timer/Kconfig
    M hw/timer/meson.build

  Log Message:
  -----------
  hw/timer: Introduce SH_TIMER Kconfig entry

We want to be able to use the 'SH4' config for architecture
specific features. Add more fine-grained selection by adding
a CONFIG_SH_TIMER selector for the SH4 timer control unit.

Add the missing MAINTAINERS entries.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-6-f4bug@amsat.org>


  Commit: 475e70bde6cbde2d38d612b19107229a4ff42f11
      
https://github.com/qemu/qemu/commit/475e70bde6cbde2d38d612b19107229a4ff42f11
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M MAINTAINERS
    M hw/block/Kconfig
    M hw/block/meson.build
    M hw/sh4/Kconfig

  Log Message:
  -----------
  hw/block: Introduce TC58128 eeprom Kconfig entry

Add more fine-grained selection by adding a CONFIG_TC58128
selector for the TC58128 eeprom.

As this device is only used by the Shix machine, add an entry
to the proper section in MAINTAINERS.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-7-f4bug@amsat.org>


  Commit: c64bd101fe1c47d550d02ee19543c9e400af9258
      
https://github.com/qemu/qemu/commit/c64bd101fe1c47d550d02ee19543c9e400af9258
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M MAINTAINERS
    M hw/pci-host/Kconfig
    M hw/pci-host/meson.build
    A hw/pci-host/sh_pci.c
    M hw/sh4/Kconfig
    M hw/sh4/meson.build
    R hw/sh4/sh_pci.c

  Log Message:
  -----------
  hw/pci-host: Introduce SH_PCI Kconfig entry

We want to be able to use the 'SH4' config for architecture
specific features. Add more fine-grained selection by adding
a CONFIG_SH_PCI selector for the SH4 PCI controller.
Move the file with the other PCI host devices in hw/pci-host
and add its missing MAINTAINERS entries.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-8-f4bug@amsat.org>


  Commit: c64dea94bd8597697b722994cbbb91a6bd0a9f8e
      
https://github.com/qemu/qemu/commit/c64dea94bd8597697b722994cbbb91a6bd0a9f8e
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M hw/sh4/Kconfig

  Log Message:
  -----------
  hw/sh4: Remove now unused CONFIG_SH4 from Kconfig

As replaced the generic CONFIG_SH4 by more fine-grained
selectors, we can remove this now unused config variable.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210222141514.2646278-9-f4bug@amsat.org>


  Commit: ef95ca038295bdf6749cbce426b281c21a08971e
      
https://github.com/qemu/qemu/commit/ef95ca038295bdf6749cbce426b281c21a08971e
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-03-06 (Sat, 06 Mar 2021)

  Changed paths:
    M hw/sh4/sh7750_regs.h

  Log Message:
  -----------
  hw/sh4/sh7750_regs: Replace link to license by its full content

This file is borrowed from the RTEMS source code, which comes
with a GPL-2.0-or-later license with a header exception.

Expand the GPL-2.0-or-later license in place to not be dependent
on a 3rd party website. This also fix the misleading comment "The
license and distribution terms for this file may be found in the
file LICENSE in this distribution" referring to the RTEMS distribution
and not to the QEMU one.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20210222185605.2714192-1-f4bug@amsat.org>


  Commit: 229a834518b950d56fd1bc94923276504d0ee9d4
      
https://github.com/qemu/qemu/commit/229a834518b950d56fd1bc94923276504d0ee9d4
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-03-08 (Mon, 08 Mar 2021)

  Changed paths:
    M MAINTAINERS
    M hw/block/Kconfig
    M hw/block/meson.build
    M hw/block/tc58128.c
    M hw/char/Kconfig
    M hw/char/meson.build
    M hw/intc/Kconfig
    M hw/intc/meson.build
    M hw/pci-host/Kconfig
    M hw/pci-host/meson.build
    A hw/pci-host/sh_pci.c
    M hw/sh4/Kconfig
    M hw/sh4/meson.build
    M hw/sh4/sh7750_regs.h
    R hw/sh4/sh_pci.c
    M hw/timer/Kconfig
    M hw/timer/meson.build
    M include/hw/sh4/sh.h
    M target/sh4/cpu.h
    M target/sh4/helper.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/philmd-gitlab/tags/renesas-20210306' 
into staging

Renesas patches queue

- MMU prototype cleanups
- Clarify licenses
- Fine-grained Kconfig entries for SH-4 devices

# gpg: Signature made Sat 06 Mar 2021 15:30:46 GMT
# gpg:                using RSA key FAABE75E12917221DCFD6BB2E3E32C2CDEADC0DE
# gpg: Good signature from "Philippe Mathieu-Daudé (F4BUG) <f4bug@amsat.org>" 
[full]
# Primary key fingerprint: FAAB E75E 1291 7221 DCFD  6BB2 E3E3 2C2C DEAD C0DE

* remotes/philmd-gitlab/tags/renesas-20210306:
  hw/sh4/sh7750_regs: Replace link to license by its full content
  hw/sh4: Remove now unused CONFIG_SH4 from Kconfig
  hw/pci-host: Introduce SH_PCI Kconfig entry
  hw/block: Introduce TC58128 eeprom Kconfig entry
  hw/timer: Introduce SH_TIMER Kconfig entry
  hw/char: Introduce SH_SCI Kconfig entry
  hw/intc: Introduce SH_INTC Kconfig entry
  hw/sh4: Add missing Kconfig dependency on SH7750 for the R2D board
  hw/sh4: Add missing license
  target/sh4: Remove unused definitions
  target/sh4: Let get_physical_address() use MMUAccessType access_type
  target/sh4: Remove unused 'int access_type' argument
  target/sh4: Replace magic value by MMUAccessType definitions
  target/sh4: Fix code style for checkpatch.pl

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


Compare: https://github.com/qemu/qemu/compare/0436c55edf6b...229a834518b9
[Prev in Thread]
Current Thread
[Next in Thread]
[Qemu-commits] [qemu/qemu] 8d2b06: target/sh4: Fix code style for checkpatch.pl, Peter Maydell <=
- [Qemu-commits] [qemu/qemu] 8d2b06: target/sh4: Fix code style for checkpatch.pl, Peter Maydell, 2021/03/08
Prev by Date: [Qemu-commits] [qemu/qemu] c6986f: KVM: x86: do not fail if software breakpoint has a...
Next by Date: [Qemu-commits] [qemu/qemu] 8d2b06: target/sh4: Fix code style for checkpatch.pl
Previous by thread: [Qemu-commits] [qemu/qemu] c6986f: KVM: x86: do not fail if software breakpoint has a...
Next by thread: [Qemu-commits] [qemu/qemu] 8d2b06: target/sh4: Fix code style for checkpatch.pl
Index(es):
- Date
- Thread