Thread (30 messages) 30 messages, 5 authors, 2018-12-17

Re: [PATCH 1/2] irq/irq_sim: provide irq_sim_fire_edge()

From: Linus Walleij <hidden>
Date: 2018-12-05 12:20:15
Also in: lkml

On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König
[off-list ref] wrote:
On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote:
quoted
It used to live in the gpio-mockup driver and I generalized it
precisely because there was another driver - iio evgen - which was
doing basically the same thing. While I don't know if there'll be more
users (I'd guess it would be useful for testing purposes of other
subsystems) having the same functionality implemented once is better
than twice.
The iio testing driver only needs the trigger and relies on an irq that
then calls the registerd handler. The iio driver doesn't need to tune
the edge sensitivity though and if your mockup driver just only calls
the fire routine if the configured sensitivity justifies that,
everything should work as expected.
Simulating edges in the generic IRQ simulator codes seems
generally useful to me, even if there is just one user now.

Certainly for any kind of IRQ testing, it could be interesting to
throw several low-to-high and high-to-low transitions
on a driver and see how it reacts.

But it is up to the irqchip maintainers to state whether they
agree.

Yours,
Linus Walleij
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help