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

Bartosz Golaszewski posted 5 patches 1 month, 3 weeks ago
There is a newer version of this series
drivers/gpio/gpiolib-cdev.c |  88 +++++++++++++++++------------------
drivers/gpio/gpiolib.c      | 110 ++++++++++++++++++++++++++++++++++++--------
drivers/gpio/gpiolib.h      |   9 +++-
3 files changed, 141 insertions(+), 66 deletions(-)
[PATCH 0/5] gpio: notify user-space about config changes in the kernel
Posted by Bartosz Golaszewski 1 month, 3 weeks ago
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,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Re: [PATCH 0/5] gpio: notify user-space about config changes in the kernel
Posted by Kent Gibson 1 month, 3 weeks ago
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>
>
Re: (subset) [PATCH 0/5] gpio: notify user-space about config changes in the kernel
Posted by Bartosz Golaszewski 1 month, 2 weeks ago
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,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>