Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-03-02 Thread Linus Walleij
On Wed, Feb 24, 2021 at 3:34 AM William Breathitt Gray wrote: > On Tue, Feb 23, 2021 at 06:45:16PM +0100, Oleksij Rempel wrote: > > > to make this possible, i would need hack gpiolib framework and add > > > name/label exporter. But after endless rounds of pingponging me for > > > renaming the dri

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-25 Thread William Breathitt Gray
On Fri, Feb 26, 2021 at 07:46:01AM +0100, Oleksij Rempel wrote: > On Wed, Feb 24, 2021 at 05:20:21PM +0900, William Breathitt Gray wrote: > > On Wed, Feb 24, 2021 at 05:11:03PM +0900, William Breathitt Gray wrote: > > > On Wed, Feb 24, 2021 at 08:35:06AM +0100, Oleksij Rempel wrote: > > > > On Wed,

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-25 Thread Oleksij Rempel
On Wed, Feb 24, 2021 at 05:20:21PM +0900, William Breathitt Gray wrote: > On Wed, Feb 24, 2021 at 05:11:03PM +0900, William Breathitt Gray wrote: > > On Wed, Feb 24, 2021 at 08:35:06AM +0100, Oleksij Rempel wrote: > > > On Wed, Feb 24, 2021 at 11:34:06AM +0900, William Breathitt Gray wrote: > > > >

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-24 Thread William Breathitt Gray
On Wed, Feb 24, 2021 at 05:11:03PM +0900, William Breathitt Gray wrote: > On Wed, Feb 24, 2021 at 08:35:06AM +0100, Oleksij Rempel wrote: > > On Wed, Feb 24, 2021 at 11:34:06AM +0900, William Breathitt Gray wrote: > > > Alternatively, we can take a more generic approach: ignore the GPIO > > > names

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-24 Thread William Breathitt Gray
On Wed, Feb 24, 2021 at 08:35:06AM +0100, Oleksij Rempel wrote: > On Wed, Feb 24, 2021 at 11:34:06AM +0900, William Breathitt Gray wrote: > > On Tue, Feb 23, 2021 at 06:45:16PM +0100, Oleksij Rempel wrote: > > > On Tue, Feb 23, 2021 at 11:06:56AM +0100, Oleksij Rempel wrote: > > > > On Mon, Feb 22,

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-23 Thread Oleksij Rempel
Hello William, On Wed, Feb 24, 2021 at 11:34:06AM +0900, William Breathitt Gray wrote: > On Tue, Feb 23, 2021 at 06:45:16PM +0100, Oleksij Rempel wrote: > > Hello William, > > > > Here is cooled down technical answer. Excuse me for over reacting. > > Hello Oleksij, > > Let me apologize too if I

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-23 Thread William Breathitt Gray
On Tue, Feb 23, 2021 at 06:45:16PM +0100, Oleksij Rempel wrote: > Hello William, > > Here is cooled down technical answer. Excuse me for over reacting. Hello Oleksij, Let me apologize too if I offended you in some way in with previous response, I assure you that was not my intention. I truly do

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-23 Thread Oleksij Rempel
Hello William, Here is cooled down technical answer. Excuse me for over reacting. On Tue, Feb 23, 2021 at 11:06:56AM +0100, Oleksij Rempel wrote: > On Mon, Feb 22, 2021 at 10:43:00AM +0900, William Breathitt Gray wrote: > > On Mon, Feb 15, 2021 at 10:17:37AM +0100, Oleksij Rempel wrote: > > > > >

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-23 Thread Oleksij Rempel
On Mon, Feb 22, 2021 at 10:43:00AM +0900, William Breathitt Gray wrote: > On Mon, Feb 15, 2021 at 10:17:37AM +0100, Oleksij Rempel wrote: > > > > +static irqreturn_t event_cnt_isr(int irq, void *dev_id) > > > > +{ > > > > + struct event_cnt_priv *priv = dev_id; > > > > + > > > > + atomi

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-21 Thread William Breathitt Gray
On Mon, Feb 15, 2021 at 10:17:37AM +0100, Oleksij Rempel wrote: > > > +static irqreturn_t event_cnt_isr(int irq, void *dev_id) > > > +{ > > > + struct event_cnt_priv *priv = dev_id; > > > + > > > + atomic_inc(&priv->count); > > > > This is just used to count the number of interrupts right? I wonde

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-15 Thread Oleksij Rempel
Hi William, On Sun, Feb 14, 2021 at 05:54:22PM +0900, William Breathitt Gray wrote: > On Mon, Feb 08, 2021 at 02:53:47PM +0100, Oleksij Rempel wrote: > > Add simple IRQ or GPIO base event counter. This device is used to measure > > rotation speed of some agricultural devices, so no high frequency

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-15 Thread Oleksij Rempel
On Fri, Feb 12, 2021 at 10:26:39AM +0100, Linus Walleij wrote: > On Mon, Feb 8, 2021 at 2:53 PM Oleksij Rempel wrote: > > > Add simple IRQ or GPIO base event counter. This device is used to measure > > rotation speed of some agricultural devices, so no high frequency on the > > counter pin is exp

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-14 Thread William Breathitt Gray
On Mon, Feb 08, 2021 at 02:53:47PM +0100, Oleksij Rempel wrote: > Add simple IRQ or GPIO base event counter. This device is used to measure > rotation speed of some agricultural devices, so no high frequency on the > counter pin is expected. > > The maximal measurement frequency depends on the CPU

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-12 Thread Linus Walleij
On Mon, Feb 8, 2021 at 2:53 PM Oleksij Rempel wrote: > Add simple IRQ or GPIO base event counter. This device is used to measure > rotation speed of some agricultural devices, so no high frequency on the > counter pin is expected. > > The maximal measurement frequency depends on the CPU and syste

Re: [PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-08 Thread William Breathitt Gray
On Mon, Feb 08, 2021 at 02:53:47PM +0100, Oleksij Rempel wrote: > Add simple IRQ or GPIO base event counter. This device is used to measure > rotation speed of some agricultural devices, so no high frequency on the > counter pin is expected. > > The maximal measurement frequency depends on the CPU

[PATCH v5 2/2] counter: add IRQ or GPIO based event counter

2021-02-08 Thread Oleksij Rempel
Add simple IRQ or GPIO base event counter. This device is used to measure rotation speed of some agricultural devices, so no high frequency on the counter pin is expected. The maximal measurement frequency depends on the CPU and system load. On the idle iMX6S I was able to measure up to 20kHz with