linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kent Gibson <warthog618@gmail.com>
To: Seamus de Mora <seamusdemora@gmail.com>
Cc: linux-gpio@vger.kernel.org
Subject: Re: [libgpiod] Some thoughts following a brief test of libgpiod ver 2.1
Date: Wed, 3 Jan 2024 17:49:31 +0800	[thread overview]
Message-ID: <20240103094931.GA117646@rigel> (raw)
In-Reply-To: <CAJ8C1XOP+h9iptC2gB0-6+Loaxr3Ow2MXYUjAxqr_t2EgBmDew@mail.gmail.com>

On Wed, Jan 03, 2024 at 01:51:53AM -0600, Seamus de Mora wrote:
> On Thu, Dec 28, 2023 at 3:29 AM Kent Gibson <warthog618@gmail.com> wrote:
> >
> > On Wed, Dec 27, 2023 at 07:19:54PM -0600, Seamus de Mora wrote:
> > > Hello,
> > >
> > > I've done some testing/evaluation of the 'libgpiod ver 2.1', and I'd
> > > like to share a few thoughts from that experience.
> > > <snip>
>
> > Then you might want to update your kernel - the kernel device driver was
> > changed to support peristing [1].
> >
> > I get this on my Pi4 running bookworm:
> >
> > $ gpioset -t0 GPIO23=0
> > $ gpioinfo GPIO23
> > gpiochip0 23    "GPIO23"                output
> > $ gpioget -a GPIO23
> > "GPIO23"=inactive
> > $ gpioinfo GPIO23
> > gpiochip0 23    "GPIO23"                output
> > $ gpioset -t0 GPIO23=1
> > $ gpioget -a GPIO23
> > "GPIO23"=active
>
> Yes - the device driver on my bulleye is current; that change was
> committed back in 1Q 2023 IIRC...
>
> I hope I've not already asked this, but:
> In ver 1.6.X of libgpiod, gpioset exits immediately, and returns to
> the bash prompt. The GPIO line remains set at the value designated
> after gpioset exits. AIUI, the driver change from 1Q 2023 was
> responsible for this.
>
> In ver 2.1 of libgpiod, gpioset (without options) does not exit. This
> means there is no return to the bash prompt. The GPIO line still
> remains set at the designated value, so there is no change in the
> behavior of the GPIO line between ver 1.6.X and 2.1.
>
> My question is why does the un-optioned gpioset ver 2.1 not exit - as
> it did in ver 1.6.X?
>

You did, and I answered on SE.
But to reiterate; gpioset v1 exited immediately and that caused
confusion when the driver would revert the line to its default state.
That made it look like gpoioset wasn't doing anything or was generating
a glitch.
That results in "gpioset doesn't work" bug reports, and we got tired of
that.
The decision was to make it block by default to make it clearer that you
lose control over the line when it exits.

In short, we changed it because people complained about it, either
explicitly or implicitly.

The -t0 option can be used to emulate the v1 behaviour.

Cheers,
Kent.

  reply	other threads:[~2024-01-03  9:49 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-28  1:19 Some thoughts following a brief test of libgpiod ver 2.1 Seamus de Mora
2023-12-28  9:29 ` [libgpiod] " Kent Gibson
2023-12-29  1:01   ` Seamus de Mora
2023-12-29  1:32     ` Kent Gibson
2023-12-29  1:41       ` Seamus de Mora
2023-12-29  2:04         ` Kent Gibson
2024-01-03  7:51   ` Seamus de Mora
2024-01-03  9:49     ` Kent Gibson [this message]
2024-01-03 19:47       ` Seamus de Mora
2024-01-03 23:25         ` Kent Gibson
2024-01-03 23:46           ` Seamus de Mora
2024-01-03 19:30   ` Stefan Wahren
2024-01-03 10:47 ` Bartosz Golaszewski
2024-01-03 17:52   ` Seamus de Mora
2024-01-03 18:35     ` Bartosz Golaszewski
2024-01-03 22:09       ` Seamus de Mora
2024-01-04  0:15         ` Kent Gibson
2024-01-04  3:22           ` Seamus de Mora
2024-01-04  4:43             ` Kent Gibson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240103094931.GA117646@rigel \
    --to=warthog618@gmail.com \
    --cc=linux-gpio@vger.kernel.org \
    --cc=seamusdemora@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).