forked from mirrors/linux
		
	gpiolib: add a per-gpio_device line state notification workqueue
In order to prepare the line state notification mechanism for working in atomic context as well, add a dedicated, high-priority, ordered workqueue to GPIO device which will be used to queue the events fron any context for them to be emitted always in process context. Reviewed-by: Kent Gibson <warthog618@gmail.com> Link: https://lore.kernel.org/r/20241018-gpio-notify-in-kernel-events-v5-5-c79135e58a1c@linaro.org Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
This commit is contained in:
		
							parent
							
								
									8c44447bd7
								
							
						
					
					
						commit
						7b9b77a8bb
					
				
					 2 changed files with 10 additions and 0 deletions
				
			
		|  | @ -2749,6 +2749,11 @@ int gpiolib_cdev_register(struct gpio_device *gdev, dev_t devt) | |||
| 	gdev->chrdev.owner = THIS_MODULE; | ||||
| 	gdev->dev.devt = MKDEV(MAJOR(devt), gdev->id); | ||||
| 
 | ||||
| 	gdev->line_state_wq = alloc_ordered_workqueue(dev_name(&gdev->dev), | ||||
| 						      WQ_HIGHPRI); | ||||
| 	if (!gdev->line_state_wq) | ||||
| 		return -ENOMEM; | ||||
| 
 | ||||
| 	ret = cdev_device_add(&gdev->chrdev, &gdev->dev); | ||||
| 	if (ret) | ||||
| 		return ret; | ||||
|  | @ -2765,6 +2770,7 @@ int gpiolib_cdev_register(struct gpio_device *gdev, dev_t devt) | |||
| 
 | ||||
| void gpiolib_cdev_unregister(struct gpio_device *gdev) | ||||
| { | ||||
| 	destroy_workqueue(gdev->line_state_wq); | ||||
| 	cdev_device_del(&gdev->chrdev, &gdev->dev); | ||||
| 	blocking_notifier_call_chain(&gdev->device_notifier, 0, NULL); | ||||
| } | ||||
|  |  | |||
|  | @ -17,6 +17,7 @@ | |||
| #include <linux/module.h> | ||||
| #include <linux/notifier.h> | ||||
| #include <linux/srcu.h> | ||||
| #include <linux/workqueue.h> | ||||
| 
 | ||||
| #define GPIOCHIP_NAME	"gpiochip" | ||||
| 
 | ||||
|  | @ -44,6 +45,8 @@ | |||
|  * @list: links gpio_device:s together for traversal | ||||
|  * @line_state_notifier: used to notify subscribers about lines being | ||||
|  *                       requested, released or reconfigured | ||||
|  * @line_state_wq: used to emit line state events from a separate thread in | ||||
|  *                 process context | ||||
|  * @device_notifier: used to notify character device wait queues about the GPIO | ||||
|  *                   device being unregistered | ||||
|  * @srcu: protects the pointer to the underlying GPIO chip | ||||
|  | @ -70,6 +73,7 @@ struct gpio_device { | |||
| 	void			*data; | ||||
| 	struct list_head        list; | ||||
| 	struct blocking_notifier_head line_state_notifier; | ||||
| 	struct workqueue_struct	*line_state_wq; | ||||
| 	struct blocking_notifier_head device_notifier; | ||||
| 	struct srcu_struct	srcu; | ||||
| 
 | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Bartosz Golaszewski
						Bartosz Golaszewski