iio: dummy: evgen: use irq_sim

Switch to using the recently added interrupt simulator for dummy irqs.

---
bloat-o-meter output:
add/remove: 0/3 grow/shrink: 1/5 up/down: 12/-540 (-528)

function                                     old     new   delta
iio_dummy_evgen_get_irq                      136     148     +12
iio_evgen_release                             40      36      -4
iio_dummy_evgen_get_regs                      32      28      -4
iio_dummy_work_handler                        20       -     -20
iio_dummy_event_irqunmask                     32       -     -32
iio_dummy_event_irqmask                       32       -     -32
iio_evgen_poke                               140     100     -40
init_module                                  376     172    -204
iio_dummy_evgen_init                         376     172    -204

Total: Before=2804, After=2276, chg -18.83%

v1 -> v2:
- added a missing kfree()

 drivers/iio/dummy/Kconfig           |  2 +-
 drivers/iio/dummy/iio_dummy_evgen.c | 89 ++++++++-----------------------------
 2 files changed, 20 insertions(+), 71 deletions(-)

index aa5824d96a43..5a29fbd3c531 100644
@@ -5,7 +5,7 @@ menu "IIO dummy driver"
 	depends on IIO
 
 config IIO_DUMMY_EVGEN
-	select IRQ_WORK
+	select IRQ_SIM
 	tristate
 
 config IIO_SIMPLE_DUMMY

2.13.2

I've backported irq_sim to v4.9 for testing and got compile errors due to a missing slab.h include. In upstream this include comes implicit through irq.h (coincidentally added by you), I wonder if we should still add it to irq_sim.c since it is explicitly using the allocation API.

2017-09-29 15:15 GMT+02:00 Lars-Peter Clausen <[email protected]>:

Thanks for the hint, I'll send it together with other updates (e.g. the reverse irq lookup).

Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it.

Thanks,

Jonathan