diff options
| author | Peter Maydell | 2020-07-03 17:59:45 +0200 |
|---|---|---|
| committer | Peter Maydell | 2020-07-03 17:59:45 +0200 |
| commit | 871f82722ccddf8886acf19989289072bc73e873 (patch) | |
| tree | e04d54a6eae6243ee3eccb365cd63393e07d6848 /hw/arm/spitz.c | |
| parent | hw/arm/spitz: Use max111x properties to set initial values (diff) | |
| download | qemu-871f82722ccddf8886acf19989289072bc73e873.tar.gz qemu-871f82722ccddf8886acf19989289072bc73e873.tar.xz qemu-871f82722ccddf8886acf19989289072bc73e873.zip | |
hw/misc/max111x: Use GPIO lines rather than max111x_set_input()
The max111x ADC device model allows other code to set the level on
the 8 ADC inputs using the max111x_set_input() function. Replace
this with generic qdev GPIO inputs, which also allow inputs to be set
to arbitrary values.
Using GPIO lines will make it easier for board code to wire things
up, so that if device A wants to set the ADC input it doesn't need to
have a direct pointer to the max111x but can just set that value on
its output GPIO, which is then wired up by the board to the
appropriate max111x input.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-id: 20200628142429.17111-11-peter.maydell@linaro.org
Diffstat (limited to 'hw/arm/spitz.c')
| -rw-r--r-- | hw/arm/spitz.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/hw/arm/spitz.c b/hw/arm/spitz.c index 93a25edcb5..fa592aad6d 100644 --- a/hw/arm/spitz.c +++ b/hw/arm/spitz.c @@ -696,13 +696,14 @@ static void corgi_ssp_gpio_cs(void *opaque, int line, int level) static void spitz_adc_temp_on(void *opaque, int line, int level) { + int batt_temp; + if (!max1111) return; - if (level) - max111x_set_input(max1111, MAX1111_BATT_TEMP, SPITZ_BATTERY_TEMP); - else - max111x_set_input(max1111, MAX1111_BATT_TEMP, 0); + batt_temp = level ? SPITZ_BATTERY_TEMP : 0; + + qemu_set_irq(qdev_get_gpio_in(max1111, MAX1111_BATT_TEMP), batt_temp); } static void corgi_ssp_realize(SSISlave *d, Error **errp) |