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

2018-12-17 Thread Bartosz Golaszewski
pon., 17 gru 2018 o 13:59 Uwe Kleine-König napisał(a): > > On Mon, Dec 17, 2018 at 11:32:45AM +0100, Bartosz Golaszewski wrote: > > śr., 5 gru 2018 o 13:38 Bartosz Golaszewski > > napisał(a): > > > > > > śr., 5 gru 2018 o 13:20 Linus Walleij > > > napisał(a): > > > > > > > > On Mon, Dec 3,

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

2018-12-17 Thread Uwe Kleine-König
On Mon, Dec 17, 2018 at 11:32:45AM +0100, Bartosz Golaszewski wrote: > śr., 5 gru 2018 o 13:38 Bartosz Golaszewski > napisał(a): > > > > śr., 5 gru 2018 o 13:20 Linus Walleij napisał(a): > > > > > > On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König > > > wrote: > > > > On Mon, Dec 03, 2018 at

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

2018-12-17 Thread Uwe Kleine-König
On Fri, Dec 14, 2018 at 03:07:37PM +0100, Linus Walleij wrote: > On Wed, Dec 5, 2018 at 2:20 PM Uwe Kleine-König > wrote: > > On Wed, Dec 05, 2018 at 01:19:54PM +0100, Linus Walleij wrote: > > > > > The iio testing driver only needs the trigger and relies on an irq that > > > > then calls the

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

2018-12-17 Thread Bartosz Golaszewski
śr., 5 gru 2018 o 13:38 Bartosz Golaszewski napisał(a): > > śr., 5 gru 2018 o 13:20 Linus Walleij napisał(a): > > > > On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König > > wrote: > > > On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > > > > > > It used to live in the

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

2018-12-14 Thread Linus Walleij
On Wed, Dec 5, 2018 at 2:20 PM Uwe Kleine-König wrote: > On Wed, Dec 05, 2018 at 01:19:54PM +0100, Linus Walleij wrote: > > > 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

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

2018-12-05 Thread Uwe Kleine-König
On Wed, Dec 05, 2018 at 01:19:54PM +0100, Linus Walleij wrote: > On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König > wrote: > > On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > > > > It used to live in the gpio-mockup driver and I generalized it > > > precisely because there

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

2018-12-05 Thread Uwe Kleine-König
On Wed, Dec 05, 2018 at 01:19:54PM +0100, Linus Walleij wrote: > On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König > wrote: > > On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > > > > It used to live in the gpio-mockup driver and I generalized it > > > precisely because there

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

2018-12-05 Thread Linus Walleij
On Wed, Dec 5, 2018 at 1:38 PM Bartosz Golaszewski wrote: > All that would be great, but at this point I just want to fix broken > tests in user-space. After that we can think about how to > improve/approach simulating interrupts all we want. That is true. The other Linus is known to steamroll

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

2018-12-05 Thread Linus Walleij
On Wed, Dec 5, 2018 at 1:38 PM Bartosz Golaszewski wrote: > All that would be great, but at this point I just want to fix broken > tests in user-space. After that we can think about how to > improve/approach simulating interrupts all we want. That is true. The other Linus is known to steamroll

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

2018-12-05 Thread Bartosz Golaszewski
śr., 5 gru 2018 o 13:20 Linus Walleij napisał(a): > > On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König > wrote: > > On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > > > > It used to live in the gpio-mockup driver and I generalized it > > > precisely because there was

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

2018-12-05 Thread Bartosz Golaszewski
śr., 5 gru 2018 o 13:20 Linus Walleij napisał(a): > > On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König > wrote: > > On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > > > > It used to live in the gpio-mockup driver and I generalized it > > > precisely because there was

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

2018-12-05 Thread Linus Walleij
On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König wrote: > On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > > 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

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

2018-12-05 Thread Linus Walleij
On Mon, Dec 3, 2018 at 12:06 PM Uwe Kleine-König wrote: > On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > > 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

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

2018-12-03 Thread Uwe Kleine-König
On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > pon., 3 gru 2018 o 11:49 Uwe Kleine-König > napisał(a): > > > > On Mon, Dec 03, 2018 at 11:23:38AM +0100, Bartosz Golaszewski wrote: > > > niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > > > > > > > niedz., 2

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

2018-12-03 Thread Uwe Kleine-König
On Mon, Dec 03, 2018 at 11:57:26AM +0100, Bartosz Golaszewski wrote: > pon., 3 gru 2018 o 11:49 Uwe Kleine-König > napisał(a): > > > > On Mon, Dec 03, 2018 at 11:23:38AM +0100, Bartosz Golaszewski wrote: > > > niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > > > > > > > niedz., 2

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

2018-12-03 Thread Bartosz Golaszewski
pon., 3 gru 2018 o 11:49 Uwe Kleine-König napisał(a): > > On Mon, Dec 03, 2018 at 11:23:38AM +0100, Bartosz Golaszewski wrote: > > niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > > > > > niedz., 2 gru 2018 o 22:56 Uwe Kleine-König > > > napisał(a): > > > > > > > > Hello, > > > >

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

2018-12-03 Thread Bartosz Golaszewski
pon., 3 gru 2018 o 11:49 Uwe Kleine-König napisał(a): > > On Mon, Dec 03, 2018 at 11:23:38AM +0100, Bartosz Golaszewski wrote: > > niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > > > > > niedz., 2 gru 2018 o 22:56 Uwe Kleine-König > > > napisał(a): > > > > > > > > Hello, > > > >

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

2018-12-03 Thread Uwe Kleine-König
On Mon, Dec 03, 2018 at 11:23:38AM +0100, Bartosz Golaszewski wrote: > niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > > > niedz., 2 gru 2018 o 22:56 Uwe Kleine-König > > napisał(a): > > > > > > Hello, > > > > > > On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski

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

2018-12-03 Thread Uwe Kleine-König
On Mon, Dec 03, 2018 at 11:23:38AM +0100, Bartosz Golaszewski wrote: > niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > > > niedz., 2 gru 2018 o 22:56 Uwe Kleine-König > > napisał(a): > > > > > > Hello, > > > > > > On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski

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

2018-12-03 Thread Bartosz Golaszewski
niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > niedz., 2 gru 2018 o 22:56 Uwe Kleine-König > napisał(a): > > > > Hello, > > > > On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski wrote: > > > We're getting too much into details of how to handle simulated > > >

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

2018-12-03 Thread Bartosz Golaszewski
niedz., 2 gru 2018 o 23:20 Bartosz Golaszewski napisał(a): > > niedz., 2 gru 2018 o 22:56 Uwe Kleine-König > napisał(a): > > > > Hello, > > > > On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski wrote: > > > We're getting too much into details of how to handle simulated > > >

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

2018-12-02 Thread Bartosz Golaszewski
niedz., 2 gru 2018 o 22:56 Uwe Kleine-König napisał(a): > > Hello, > > On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski wrote: > > We're getting too much into details of how to handle simulated > > interrupts and we can continue discussing it, but meanwhile I'd like > > to address a

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

2018-12-02 Thread Bartosz Golaszewski
niedz., 2 gru 2018 o 22:56 Uwe Kleine-König napisał(a): > > Hello, > > On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski wrote: > > We're getting too much into details of how to handle simulated > > interrupts and we can continue discussing it, but meanwhile I'd like > > to address a

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

2018-12-02 Thread Uwe Kleine-König
Hello, On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski wrote: > We're getting too much into details of how to handle simulated > interrupts and we can continue discussing it, but meanwhile I'd like > to address a different thing: > > Thomas, Linus: after commit fa38869b0161

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

2018-12-02 Thread Uwe Kleine-König
Hello, On Thu, Nov 29, 2018 at 07:14:45PM +0100, Bartosz Golaszewski wrote: > We're getting too much into details of how to handle simulated > interrupts and we can continue discussing it, but meanwhile I'd like > to address a different thing: > > Thomas, Linus: after commit fa38869b0161

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

2018-12-02 Thread Bartosz Golaszewski
niedz., 2 gru 2018 o 13:29 Marc Zyngier napisał(a): > > On Fri, 30 Nov 2018 23:26:25 +0100 > Linus Walleij wrote: > > > On Thu, Nov 29, 2018 at 7:14 PM Bartosz Golaszewski wrote: > > > > > Thomas, Linus: after commit fa38869b0161 ("gpiolib: Don't support irq > > > sharing for userspace") some

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

2018-12-02 Thread Bartosz Golaszewski
niedz., 2 gru 2018 o 13:29 Marc Zyngier napisał(a): > > On Fri, 30 Nov 2018 23:26:25 +0100 > Linus Walleij wrote: > > > On Thu, Nov 29, 2018 at 7:14 PM Bartosz Golaszewski wrote: > > > > > Thomas, Linus: after commit fa38869b0161 ("gpiolib: Don't support irq > > > sharing for userspace") some

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

2018-12-02 Thread Marc Zyngier
On Fri, 30 Nov 2018 23:26:25 +0100 Linus Walleij wrote: > On Thu, Nov 29, 2018 at 7:14 PM Bartosz Golaszewski wrote: > > > Thomas, Linus: after commit fa38869b0161 ("gpiolib: Don't support irq > > sharing for userspace") some libgpiod tests are failing because we can > > no longer depend on

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

2018-12-02 Thread Marc Zyngier
On Fri, 30 Nov 2018 23:26:25 +0100 Linus Walleij wrote: > On Thu, Nov 29, 2018 at 7:14 PM Bartosz Golaszewski wrote: > > > Thomas, Linus: after commit fa38869b0161 ("gpiolib: Don't support irq > > sharing for userspace") some libgpiod tests are failing because we can > > no longer depend on

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

2018-11-30 Thread Linus Walleij
On Thu, Nov 29, 2018 at 7:14 PM Bartosz Golaszewski wrote: > Thomas, Linus: after commit fa38869b0161 ("gpiolib: Don't support irq > sharing for userspace") some libgpiod tests are failing because we can > no longer depend on reading the value of a dummy GPIO after detecting > an interrupt to

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

2018-11-30 Thread Linus Walleij
On Thu, Nov 29, 2018 at 7:14 PM Bartosz Golaszewski wrote: > Thomas, Linus: after commit fa38869b0161 ("gpiolib: Don't support irq > sharing for userspace") some libgpiod tests are failing because we can > no longer depend on reading the value of a dummy GPIO after detecting > an interrupt to

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

2018-11-29 Thread Bartosz Golaszewski
niedz., 25 lis 2018 o 22:18 Uwe Kleine-König napisał(a): > > On Fri, Nov 23, 2018 at 04:59:46PM +0100, Bartosz Golaszewski wrote: > > śr., 21 lis 2018 o 20:15 Uwe Kleine-König > > napisał(a): > > > > > > Hello Bartosz, > > > > > > On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski

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

2018-11-29 Thread Bartosz Golaszewski
niedz., 25 lis 2018 o 22:18 Uwe Kleine-König napisał(a): > > On Fri, Nov 23, 2018 at 04:59:46PM +0100, Bartosz Golaszewski wrote: > > śr., 21 lis 2018 o 20:15 Uwe Kleine-König > > napisał(a): > > > > > > Hello Bartosz, > > > > > > On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski

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

2018-11-25 Thread Uwe Kleine-König
On Fri, Nov 23, 2018 at 04:59:46PM +0100, Bartosz Golaszewski wrote: > śr., 21 lis 2018 o 20:15 Uwe Kleine-König > napisał(a): > > > > Hello Bartosz, > > > > On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski wrote: > > > wt., 20 lis 2018 o 18:17 Uwe Kleine-König > > > napisał(a): > >

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

2018-11-25 Thread Uwe Kleine-König
On Fri, Nov 23, 2018 at 04:59:46PM +0100, Bartosz Golaszewski wrote: > śr., 21 lis 2018 o 20:15 Uwe Kleine-König > napisał(a): > > > > Hello Bartosz, > > > > On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski wrote: > > > wt., 20 lis 2018 o 18:17 Uwe Kleine-König > > > napisał(a): > >

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

2018-11-23 Thread Bartosz Golaszewski
śr., 21 lis 2018 o 20:15 Uwe Kleine-König napisał(a): > > Hello Bartosz, > > On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski wrote: > > wt., 20 lis 2018 o 18:17 Uwe Kleine-König > > napisał(a): > > > > > > On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > > > >

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

2018-11-23 Thread Bartosz Golaszewski
śr., 21 lis 2018 o 20:15 Uwe Kleine-König napisał(a): > > Hello Bartosz, > > On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski wrote: > > wt., 20 lis 2018 o 18:17 Uwe Kleine-König > > napisał(a): > > > > > > On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > > > >

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

2018-11-21 Thread Uwe Kleine-König
Hello Bartosz, On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski wrote: > wt., 20 lis 2018 o 18:17 Uwe Kleine-König > napisał(a): > > > > On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > > > From: Bartosz Golaszewski > > > > > > The irq_sim irqchip doesn't

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

2018-11-21 Thread Uwe Kleine-König
Hello Bartosz, On Wed, Nov 21, 2018 at 05:34:32PM +0100, Bartosz Golaszewski wrote: > wt., 20 lis 2018 o 18:17 Uwe Kleine-König > napisał(a): > > > > On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > > > From: Bartosz Golaszewski > > > > > > The irq_sim irqchip doesn't

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

2018-11-21 Thread Bartosz Golaszewski
wt., 20 lis 2018 o 18:17 Uwe Kleine-König napisał(a): > > On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski > > > > The irq_sim irqchip doesn't allow to configure the sensitivity so every > > call to irq_sim_fire() fires a dummy interrupt. This

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

2018-11-21 Thread Bartosz Golaszewski
wt., 20 lis 2018 o 18:17 Uwe Kleine-König napisał(a): > > On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski > > > > The irq_sim irqchip doesn't allow to configure the sensitivity so every > > call to irq_sim_fire() fires a dummy interrupt. This

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

2018-11-20 Thread Uwe Kleine-König
On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > The irq_sim irqchip doesn't allow to configure the sensitivity so every > call to irq_sim_fire() fires a dummy interrupt. This used to not matter > for gpio-mockup (one of the irq_sim users)

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

2018-11-20 Thread Uwe Kleine-König
On Tue, Nov 20, 2018 at 02:40:31PM +0100, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > The irq_sim irqchip doesn't allow to configure the sensitivity so every > call to irq_sim_fire() fires a dummy interrupt. This used to not matter > for gpio-mockup (one of the irq_sim users)

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

2018-11-20 Thread Bartosz Golaszewski
From: Bartosz Golaszewski The irq_sim irqchip doesn't allow to configure the sensitivity so every call to irq_sim_fire() fires a dummy interrupt. This used to not matter for gpio-mockup (one of the irq_sim users) until commit fa38869b0161 ("gpiolib: Don't support irq sharing for userspace")

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

2018-11-20 Thread Bartosz Golaszewski
From: Bartosz Golaszewski The irq_sim irqchip doesn't allow to configure the sensitivity so every call to irq_sim_fire() fires a dummy interrupt. This used to not matter for gpio-mockup (one of the irq_sim users) until commit fa38869b0161 ("gpiolib: Don't support irq sharing for userspace")