mbox series

[0/5] gpio: notify user-space about config changes in the kernel

Message ID 20241004-gpio-notify-in-kernel-events-v1-0-8ac29e1df4fe@linaro.org
Headers show
Series gpio: notify user-space about config changes in the kernel | expand

Message

Bartosz Golaszewski Oct. 4, 2024, 2:43 p.m. UTC
We currently only emit events on changed line config to user-space on
changes made from user-space. Users have no way of getting notified
about in-kernel changes. This series improves the situation and also
contains a couple other related improvements.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
---
Bartosz Golaszewski (5):
      gpiolib: use v2 defines for line state change events
      gpiolib: unify two loops initializing GPIO descriptors
      gpio: cdev: update flags at once when reconfiguring from user-space
      gpiolib: simplify notifying user-space about line requests
      gpiolib: notify user-space about in-kernel line state changes

 drivers/gpio/gpiolib-cdev.c |  88 +++++++++++++++++------------------
 drivers/gpio/gpiolib.c      | 110 ++++++++++++++++++++++++++++++++++++--------
 drivers/gpio/gpiolib.h      |   9 +++-
 3 files changed, 141 insertions(+), 66 deletions(-)
---
base-commit: 58ca61c1a866bfdaa5e19fb19a2416764f847d75
change-id: 20240924-gpio-notify-in-kernel-events-07cd912153e8

Best regards,

Comments

Kent Gibson Oct. 6, 2024, 1:29 p.m. UTC | #1
On Fri, Oct 04, 2024 at 04:43:21PM +0200, Bartosz Golaszewski wrote:
> We currently only emit events on changed line config to user-space on
> changes made from user-space. Users have no way of getting notified
> about in-kernel changes. This series improves the situation and also
> contains a couple other related improvements.
>
> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

Patches 1-3 look good to me.

Reviewed-by: Kent Gibson <warthog618@gmail.com>

for those three.

> ---
> Bartosz Golaszewski (5):
>       gpiolib: use v2 defines for line state change events
>       gpiolib: unify two loops initializing GPIO descriptors
>       gpio: cdev: update flags at once when reconfiguring from user-space
>       gpiolib: simplify notifying user-space about line requests
>       gpiolib: notify user-space about in-kernel line state changes
>
>  drivers/gpio/gpiolib-cdev.c |  88 +++++++++++++++++------------------
>  drivers/gpio/gpiolib.c      | 110 ++++++++++++++++++++++++++++++++++++--------
>  drivers/gpio/gpiolib.h      |   9 +++-
>  3 files changed, 141 insertions(+), 66 deletions(-)
> ---
> base-commit: 58ca61c1a866bfdaa5e19fb19a2416764f847d75
> change-id: 20240924-gpio-notify-in-kernel-events-07cd912153e8
>
> Best regards,
> --
> Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
>
Bartosz Golaszewski Oct. 8, 2024, 8:22 a.m. UTC | #2
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>


On Fri, 04 Oct 2024 16:43:21 +0200, Bartosz Golaszewski wrote:
> We currently only emit events on changed line config to user-space on
> changes made from user-space. Users have no way of getting notified
> about in-kernel changes. This series improves the situation and also
> contains a couple other related improvements.
> 
> 

Applied, thanks!

[1/5] gpiolib: use v2 defines for line state change events
      commit: ee194b12bf9a2f2ed57b9c5bc7a5f221f7f4a06f
[2/5] gpiolib: unify two loops initializing GPIO descriptors
      commit: fa17f749ee5bc6afdaa9e0ddbe6a816b490dad7d
[3/5] gpio: cdev: update flags at once when reconfiguring from user-space
      commit: b7adfb6076ff0c1ebbde56d1903daa3d07db92c5

Best regards,