From: "Alastair D'Silva" <alastair@au1.ibm.com>
To: qemu-arm@nongnu.org
Cc: qemu-devel@nongnu.org, "Peter Maydell" <peter.maydell@linaro.org>,
"Cédric Le Goater" <clg@kaod.org>,
"Andrew Jeffery" <andrew@aj.id.au>,
"Joel Stanley" <joel@jms.id.au>,
"Alastair D'Silva" <alastair@d-silva.org>
Subject: [Qemu-devel] [PATCH v4 0/8] Add support for the Epson RX8900 RTC to the aspeed board
Date: Thu, 15 Dec 2016 16:48:04 +1100 [thread overview]
Message-ID: <20161215054812.12602-1-alastair@au1.ibm.com> (raw)
From: Alastair D'Silva <alastair@d-silva.org>
This patch series adds support for the Epson RX8900 RTC to the
Aspeed board.
The tests use the existing imx25 infrastructure, and some minor
changes have been made to uniquely identify the I2C buses.
Support for named interrupts has been implemented in qtest to
facilitate testing.
Changelog:
v4:
- Split overloaded patches
- Fix spelling of Celsius
- Remove degrees sign from messages
- Don't force float precision
- Reformat named interrupt qtest protocol for consistency
- Minor formatting changes
- Don't connect the RX8900 INT line to NVIC:22, it should be
connected to GPIOJ10 on the ASPEED SOC
v3:
- Add range checks for time components
- Shorten trace messages
- Use qemu generated i2c bus names
- Address style comments from Alexey for rx8900 driver
- Fix long lines in qtest harness
- Remove trailing whitespace in hw/arm/pxa
- Remove stray headers from rx8900 driver
- Support low voltage detection in rx8900 driver
v2:
- Avoid using error_report
- Fix TIE behaviour
- Move temperature init out of reset, it should not change
on a soft reset
- Fix IRQ in test failures
- Add a NULL check for i2c slave init callbacks, so that we
no longer need to implement empty init functions.
- Refactor to use the existing qemu trace infrastructure
- Move the RX8900 definition into the aspeed machine init
- It it possible for check_update_interrupt_seconds in the
rx8900 test to return 6, depending on the alignment with
the wall clock.
- Support low voltage detection
Alastair D'Silva (8):
arm: Uniquely name imx25 I2C buses.
qtest: Support named interrupts
qtest: Support setting named GPIOs
qtest: Fix whitespace
hw/i2c: Tidy up NULL check for i2c slave init callbacks
hw/timer: Add Epson RX8900 RTC support
tests: Test all implemented RX8900 functionality
arm: Add an RX8900 RTC to the ASpeed board
default-configs/arm-softmmu.mak | 1 +
hw/arm/aspeed.c | 29 ++
hw/arm/imx25_pdk.c | 2 +-
hw/core/irq.c | 18 +-
hw/i2c/core.c | 4 +-
hw/i2c/imx_i2c.c | 2 +-
hw/timer/Makefile.objs | 2 +
hw/timer/rx8900.c | 912 ++++++++++++++++++++++++++++++++++++++++
hw/timer/rx8900_regs.h | 141 +++++++
hw/timer/trace-events | 31 ++
include/hw/irq.h | 13 +-
qtest.c | 103 +++--
tests/Makefile.include | 2 +
tests/libqtest.c | 85 +++-
tests/libqtest.h | 59 +++
tests/rx8900-test.c | 882 ++++++++++++++++++++++++++++++++++++++
16 files changed, 2243 insertions(+), 43 deletions(-)
create mode 100644 hw/timer/rx8900.c
create mode 100644 hw/timer/rx8900_regs.h
create mode 100644 tests/rx8900-test.c
--
2.9.3
next reply other threads:[~2016-12-15 5:49 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-15 5:48 Alastair D'Silva [this message]
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 1/8] arm: Uniquely name imx25 I2C buses Alastair D'Silva
2016-12-16 13:03 ` Peter Maydell
2016-12-16 20:17 ` Jean-Christophe DUBOIS
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 2/8] qtest: Support named interrupts Alastair D'Silva
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 3/8] qtest: Support setting named GPIOs Alastair D'Silva
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 4/8] qtest: Fix whitespace Alastair D'Silva
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 5/8] hw/i2c: Tidy up NULL check for i2c slave init callbacks Alastair D'Silva
2016-12-16 12:56 ` Peter Maydell
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 6/8] hw/timer: Add Epson RX8900 RTC support Alastair D'Silva
2017-01-04 4:59 ` Andrew Jeffery
2017-01-04 5:34 ` Alastair D'Silva
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 7/8] tests: Test all implemented RX8900 functionality Alastair D'Silva
2017-01-04 6:14 ` Andrew Jeffery
2017-01-04 23:30 ` Alastair D'Silva
2016-12-15 5:48 ` [Qemu-devel] [PATCH v4 8/8] arm: Add an RX8900 RTC to the ASpeed board Alastair D'Silva
2017-01-04 6:19 ` Andrew Jeffery
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161215054812.12602-1-alastair@au1.ibm.com \
--to=alastair@au1.ibm.com \
--cc=alastair@d-silva.org \
--cc=andrew@aj.id.au \
--cc=clg@kaod.org \
--cc=joel@jms.id.au \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.