[PATCH 0/6] Additional NPCM7xx features, devices and tests

Havard Skinnemoen via posted 6 patches 5 years, 1 month ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201008232154.94221-1-hskinnemoen@google.com
Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Gerd Hoffmann <kraxel@redhat.com>, Havard Skinnemoen <hskinnemoen@google.com>, Tyrone Ting <kfting@nuvoton.com>, Laurent Vivier <lvivier@redhat.com>, Thomas Huth <thuth@redhat.com>
There is a newer version of this series
docs/system/arm/nuvoton.rst               |   6 +-
hw/usb/hcd-ehci.h                         |   1 +
include/hw/arm/npcm7xx.h                  |   8 +
include/hw/gpio/npcm7xx_gpio.h            |  55 +++
include/hw/misc/npcm7xx_clk.h             |   9 +
include/hw/misc/npcm7xx_rng.h             |  34 ++
include/hw/timer/npcm7xx_timer.h          |  43 +-
hw/arm/npcm7xx.c                          | 125 ++++-
hw/gpio/npcm7xx_gpio.c                    | 409 ++++++++++++++++
hw/misc/npcm7xx_clk.c                     |  20 +
hw/misc/npcm7xx_rng.c                     | 179 +++++++
hw/timer/npcm7xx_timer.c                  | 279 +++++++++--
hw/usb/hcd-ehci-sysbus.c                  |  19 +
tests/qtest/npcm7xx_gpio-test.c           | 385 +++++++++++++++
tests/qtest/npcm7xx_rng-test.c            | 278 +++++++++++
tests/qtest/npcm7xx_timer-test.c          | 562 ++++++++++++++++++++++
tests/qtest/npcm7xx_watchdog_timer-test.c | 313 ++++++++++++
MAINTAINERS                               |   1 +
hw/gpio/meson.build                       |   1 +
hw/gpio/trace-events                      |   7 +
hw/misc/meson.build                       |   1 +
hw/misc/trace-events                      |   4 +
tests/qtest/meson.build                   |   4 +
23 files changed, 2682 insertions(+), 61 deletions(-)
create mode 100644 include/hw/gpio/npcm7xx_gpio.h
create mode 100644 include/hw/misc/npcm7xx_rng.h
create mode 100644 hw/gpio/npcm7xx_gpio.c
create mode 100644 hw/misc/npcm7xx_rng.c
create mode 100644 tests/qtest/npcm7xx_gpio-test.c
create mode 100644 tests/qtest/npcm7xx_rng-test.c
create mode 100644 tests/qtest/npcm7xx_timer-test.c
create mode 100644 tests/qtest/npcm7xx_watchdog_timer-test.c
[PATCH 0/6] Additional NPCM7xx features, devices and tests
Posted by Havard Skinnemoen via 5 years, 1 month ago
This is an update to the initial NPCM7xx patch series adding

  - A timer test that found several issues that were fixed in the final version
    of the series (see
    https://www.mail-archive.com/qemu-devel@nongnu.org/msg739516.html).
  - Watchdog timer support. This makes the reboot command work.
  - Random Number Generator device.
  - USB Host Controllers.
  - GPIO Controllers.

The watchdog was implemented by my new teammate Hao Wu. Expect to see more
patches from him in the near future.

This series has also been pushed to the npcm7xx-5.2-update branch of my github
repository at

  https://github.com/hskinnemoen/qemu

Again, thanks a lot for reviewing!

Havard

Hao Wu (1):
  hw/timer: Adding watchdog for NPCM7XX Timer.

Havard Skinnemoen (5):
  tests/qtest: Add npcm7xx timer test
  Move npcm7xx_timer_reached_zero call out of npcm7xx_timer_pause
  hw/misc: Add npcm7xx random number generator
  hw/arm/npcm7xx: Add EHCI and OHCI controllers
  hw/gpio: Add GPIO model for Nuvoton NPCM7xx

 docs/system/arm/nuvoton.rst               |   6 +-
 hw/usb/hcd-ehci.h                         |   1 +
 include/hw/arm/npcm7xx.h                  |   8 +
 include/hw/gpio/npcm7xx_gpio.h            |  55 +++
 include/hw/misc/npcm7xx_clk.h             |   9 +
 include/hw/misc/npcm7xx_rng.h             |  34 ++
 include/hw/timer/npcm7xx_timer.h          |  43 +-
 hw/arm/npcm7xx.c                          | 125 ++++-
 hw/gpio/npcm7xx_gpio.c                    | 409 ++++++++++++++++
 hw/misc/npcm7xx_clk.c                     |  20 +
 hw/misc/npcm7xx_rng.c                     | 179 +++++++
 hw/timer/npcm7xx_timer.c                  | 279 +++++++++--
 hw/usb/hcd-ehci-sysbus.c                  |  19 +
 tests/qtest/npcm7xx_gpio-test.c           | 385 +++++++++++++++
 tests/qtest/npcm7xx_rng-test.c            | 278 +++++++++++
 tests/qtest/npcm7xx_timer-test.c          | 562 ++++++++++++++++++++++
 tests/qtest/npcm7xx_watchdog_timer-test.c | 313 ++++++++++++
 MAINTAINERS                               |   1 +
 hw/gpio/meson.build                       |   1 +
 hw/gpio/trace-events                      |   7 +
 hw/misc/meson.build                       |   1 +
 hw/misc/trace-events                      |   4 +
 tests/qtest/meson.build                   |   4 +
 23 files changed, 2682 insertions(+), 61 deletions(-)
 create mode 100644 include/hw/gpio/npcm7xx_gpio.h
 create mode 100644 include/hw/misc/npcm7xx_rng.h
 create mode 100644 hw/gpio/npcm7xx_gpio.c
 create mode 100644 hw/misc/npcm7xx_rng.c
 create mode 100644 tests/qtest/npcm7xx_gpio-test.c
 create mode 100644 tests/qtest/npcm7xx_rng-test.c
 create mode 100644 tests/qtest/npcm7xx_timer-test.c
 create mode 100644 tests/qtest/npcm7xx_watchdog_timer-test.c

-- 
2.28.0.1011.ga647a8990f-goog


Re: [PATCH 0/6] Additional NPCM7xx features, devices and tests
Posted by Peter Maydell 5 years ago
On Fri, 9 Oct 2020 at 00:22, Havard Skinnemoen <hskinnemoen@google.com> wrote:
>
> This is an update to the initial NPCM7xx patch series adding
>
>   - A timer test that found several issues that were fixed in the final version
>     of the series (see
>     https://www.mail-archive.com/qemu-devel@nongnu.org/msg739516.html).
>   - Watchdog timer support. This makes the reboot command work.
>   - Random Number Generator device.
>   - USB Host Controllers.
>   - GPIO Controllers.
>
> The watchdog was implemented by my new teammate Hao Wu. Expect to see more
> patches from him in the near future.
>
> This series has also been pushed to the npcm7xx-5.2-update branch of my github
> repository at
>
>   https://github.com/hskinnemoen/qemu
>
> Again, thanks a lot for reviewing!

I'm going to take patch 1 (the timer test case) into my target-arm.next
queue; I've sent some review comments on the other parts of the series.

thanks
-- PMM