public inbox for linux-gpio@vger.kernel.org
 help / color / mirror / Atom feed
From: andy.shevchenko@gmail.com
To: Bartosz Golaszewski <brgl@bgdev.pl>
Cc: "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
	Linus Walleij <linus.walleij@linaro.org>,
	Kent Gibson <warthog618@gmail.com>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Ferry Toth <fntoth@gmail.com>
Subject: Re: [ANNOUNCE] libgpiod v2.0-rc3 released
Date: Fri, 24 Feb 2023 14:28:51 +0200	[thread overview]
Message-ID: <Y/itg/cmrPCGa5qf@surfacebook> (raw)
In-Reply-To: <CAMRc=Mc+cAPZnxFXPPp0RzhUqFRBYBkf1c9=wTozY15gTyi5aQ@mail.gmail.com>

Thu, Feb 23, 2023 at 08:25:02PM +0100, Bartosz Golaszewski kirjoitti:
> I decided to introduce a late change to the C++ bindings - mark all
> public types as final since they already don't have virtual
> destructors and inheriting from them makes little sense anyway. This
> has little impact on the programming interface but I still think it's
> worth another RC and I also have a gut feeling that makes me want to
> sit down over the weekend and inspect the entire API once more before
> carving it in stone for the foreseeable future.
> 
> The tarball and git tree are in their usual places[1][2].

Thank you for the update!

A bit of an offtopic here (but related a bit as well), but since all parties
are in Cc list I dare to ask.

I have got one bug report internally and, while thinking over it (it has nothing
to do with the library, but with the flow on how we change line states during
requests and releases), realised that we probably have no knob to tell GPIO driver
to which state pin should be left after release.

This at least allows several things to achieve:
1) emulation of the sysfs behaviour (to some extent) without a necessity of
   the context (yes, I know that this is still error prone, but why not);
2) allow the possibility to grab a GPIO line and set it to the particular
   state and leave the process off (makes sense in some setups where it's
   guaranted that no other process will touch the line);
3) something else I forgot or not even thinking of.

That said, would be nice to have an additional flag (during request?)
to tell kernel what it should do with the line after releasing the
handle from user space.

Thoughts?

P.S. Sorry if I missed any discussion related to this in the past.
In such case, please share the links.

> [1] https://mirrors.edge.kernel.org/pub/software/libs/libgpiod/
> [2] git://git.kernel.org/pub/scm/libs/libgpiod/libgpiod.git

-- 
With Best Regards,
Andy Shevchenko



  reply	other threads:[~2023-02-24 12:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-23 19:25 [ANNOUNCE] libgpiod v2.0-rc3 released Bartosz Golaszewski
2023-02-24 12:28 ` andy.shevchenko [this message]
2023-02-27 10:36   ` Linus Walleij
2023-02-27 11:41     ` Andy Shevchenko
2023-03-01 13:25   ` Bartosz Golaszewski

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=Y/itg/cmrPCGa5qf@surfacebook \
    --to=andy.shevchenko@gmail.com \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=brgl@bgdev.pl \
    --cc=fntoth@gmail.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=warthog618@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