diff options
author | marcel <marcel@FreeBSD.org> | 2011-02-07 05:36:32 +0000 |
---|---|---|
committer | marcel <marcel@FreeBSD.org> | 2011-02-07 05:36:32 +0000 |
commit | afb715ad585ad96def5d54a43aeb9f83a2cf6476 (patch) | |
tree | 4c3f6a55e0b71968e657f8d0f059b3e7c8b84e88 /sys/arm | |
parent | 4054909b914e5c1b9387a1d42335f1052313e242 (diff) | |
download | FreeBSD-src-afb715ad585ad96def5d54a43aeb9f83a2cf6476.zip FreeBSD-src-afb715ad585ad96def5d54a43aeb9f83a2cf6476.tar.gz |
Remove use_high from the softc and simply check the number of GPIO
pins to determine whether there's a high register set or not. This
allows platform_gpio_init() to work without duplicating the work
done in the attach method.
Diffstat (limited to 'sys/arm')
-rw-r--r-- | sys/arm/mv/gpio.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/sys/arm/mv/gpio.c b/sys/arm/mv/gpio.c index 566c147..4053aba 100644 --- a/sys/arm/mv/gpio.c +++ b/sys/arm/mv/gpio.c @@ -64,7 +64,6 @@ struct mv_gpio_softc { bus_space_handle_t bsh; uint8_t pin_num; /* number of GPIO pins */ uint8_t irq_num; /* number of real IRQs occupied by GPIO controller */ - uint8_t use_high; }; extern struct resource_spec mv_gpio_res[]; @@ -141,7 +140,6 @@ mv_gpio_attach(device_t dev) uint32_t dev_id, rev_id; sc = (struct mv_gpio_softc *)device_get_softc(dev); - if (sc == NULL) return (ENXIO); @@ -156,12 +154,10 @@ mv_gpio_attach(device_t dev) dev_id == MV_DEV_MV78100_Z0 ) { sc->pin_num = 32; sc->irq_num = 4; - sc->use_high = 0; } else if (dev_id == MV_DEV_88F6281) { sc->pin_num = 50; sc->irq_num = 7; - sc->use_high = 1; } else { device_printf(dev, "unknown chip id=0x%x\n", dev_id); @@ -182,7 +178,7 @@ mv_gpio_attach(device_t dev) bus_space_write_4(sc->bst, sc->bsh, GPIO_INT_LEV_MASK, 0); bus_space_write_4(sc->bst, sc->bsh, GPIO_INT_CAUSE, 0); - if (sc->use_high) { + if (sc->pin_num > GPIO_PINS_PER_REG) { bus_space_write_4(sc->bst, sc->bsh, GPIO_HI_INT_EDGE_MASK, 0); bus_space_write_4(sc->bst, sc->bsh, @@ -217,7 +213,7 @@ mv_gpio_intr(void *arg) int_cause = mv_gpio_reg_read(GPIO_INT_CAUSE); gpio_val = mv_gpio_reg_read(GPIO_DATA_IN); gpio_val &= int_cause; - if (mv_gpio_softc->use_high) { + if (mv_gpio_softc->pin_num > GPIO_PINS_PER_REG) { int_cause_hi = mv_gpio_reg_read(GPIO_HI_INT_CAUSE); gpio_val_hi = mv_gpio_reg_read(GPIO_HI_DATA_IN); gpio_val_hi &= int_cause_hi; @@ -231,7 +227,7 @@ mv_gpio_intr(void *arg) i++; } - if (mv_gpio_softc->use_high) { + if (mv_gpio_softc->pin_num > GPIO_PINS_PER_REG) { i = 0; while (gpio_val_hi != 0) { if (gpio_val_hi & 1) |