Loading...
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 | This is a place for planning the ongoing long-term work in the GPIO subsystem. GPIO descriptors Starting with commit 79a9becda894 the GPIO subsystem embarked on a journey to move away from the global GPIO numberspace and toward a decriptor-based approach. This means that GPIO consumers, drivers and machine descriptions ideally have no use or idea of the global GPIO numberspace that has/was used in the inception of the GPIO subsystem. Work items: - Convert all GPIO device drivers to only #include <linux/gpio/driver.h> - Convert all consumer drivers to only #include <linux/gpio/consumer.h> - Convert all machine descriptors in "boardfiles" to only #include <linux/gpio/machine.h>, the other option being to convert it to a machine description such as device tree, ACPI or fwnode that implicitly does not use global GPIO numbers. - When this work is complete (will require some of the items in the following ongoing work as well) we can delete the old global numberspace accessors from <linux/gpio.h> and eventually delete <linux/gpio.h> altogether. Get rid of <linux/of_gpio.h> This header and helpers appeared at one point when there was no proper driver infrastructure for doing simpler MMIO GPIO devices and there was no core support for parsing device tree GPIOs from the core library with the [devm_]gpiod_get() calls we have today that will implicitly go into the device tree back-end. Work items: - Get rid of struct of_mm_gpio_chip altogether: use the generic MMIO GPIO for all current users (see below). Delete struct of_mm_gpio_chip, to_of_mm_gpio_chip(), of_mm_gpiochip_add_data(), of_mm_gpiochip_add() of_mm_gpiochip_remove() from the kernel. - Change all consumer drivers that #include <linux/of_gpio.h> to #include <linux/gpio/consumer.h> and stop doing custom parsing of the GPIO lines from the device tree. This can be tricky and often ivolves changing boardfiles, etc. - Pull semantics for legacy device tree (OF) GPIO lookups into gpiolib-of.c: in some cases subsystems are doing custom flags and lookups for polarity inversion, open drain and what not. As we now handle this with generic OF bindings, pull all legacy handling into gpiolib so the library API becomes narrow and deep and handle all legacy bindings internally. (See e.g. commits 6953c57ab172, 6a537d48461d etc) - Delete <linux/of_gpio.h> when all the above is complete and everything uses <linux/gpio/consumer.h> or <linux/gpio/driver.h> instead. Collect drivers Collect GPIO drivers from arch/* and other places that should be placed in drivers/gpio/gpio-*. Augment platforms to create platform devices or similar and probe a proper driver in the gpiolib subsystem. In some cases it makes sense to create a GPIO chip from the local driver for a few GPIOs. Those should stay where they are. Generic MMIO GPIO The GPIO drivers can utilize the generic MMIO helper library in many cases, and the helper library should be as helpful as possible for MMIO drivers. (drivers/gpio/gpio-mmio.c) Work items: - Look over and identify any remaining easily converted drivers and dry-code conversions to MMIO GPIO for maintainers to test - Expand the MMIO GPIO or write a new library for port-mapped I/O helpers (x86 inb()/outb()) and convert port-mapped I/O drivers to use this with dry-coding and sending to maintainers to test GPIOLIB irqchip The GPIOLIB irqchip is a helper irqchip for "simple cases" that should try to cover any generic kind of irqchip cascaded from a GPIO. - Convert all the GPIOLIB_IRQCHIP users to pass an irqchip template, parent and flags before calling [devm_]gpiochip_add[_data](). Currently we set up the irqchip after setting up the gpiochip using gpiochip_irqchip_add() and gpiochip_set_[chained|nested]_irqchip(). This is too complex, so convert all users over to just set up the irqchip before registering the gpio_chip, typical example: /* Typical state container with dynamic irqchip */ struct my_gpio { struct gpio_chip gc; struct irq_chip irq; }; int irq; /* from platform etc */ struct my_gpio *g; struct gpio_irq_chip *girq /* Set up the irqchip dynamically */ g->irq.name = "my_gpio_irq"; g->irq.irq_ack = my_gpio_ack_irq; g->irq.irq_mask = my_gpio_mask_irq; g->irq.irq_unmask = my_gpio_unmask_irq; g->irq.irq_set_type = my_gpio_set_irq_type; /* Get a pointer to the gpio_irq_chip */ girq = &g->gc.irq; girq->chip = &g->irq; girq->parent_handler = ftgpio_gpio_irq_handler; girq->num_parents = 1; girq->parents = devm_kcalloc(dev, 1, sizeof(*girq->parents), GFP_KERNEL); if (!girq->parents) return -ENOMEM; girq->default_type = IRQ_TYPE_NONE; girq->handler = handle_bad_irq; girq->parents[0] = irq; When this is done, we will delete the old APIs for instatiating GPIOLIB_IRQCHIP and simplify the code. - Look over and identify any remaining easily converted drivers and dry-code conversions to gpiolib irqchip for maintainers to test - Support generic hierarchical GPIO interrupts: these are for the non-cascading case where there is one IRQ per GPIO line, there is currently no common infrastructure for this. Increase integration with pin control There are already ways to use pin control as back-end for GPIO and it may make sense to bring these subsystems closer. One reason for creating pin control as its own subsystem was that we could avoid any use of the global GPIO numbers. Once the above is complete, it may make sense to simply join the subsystems into one and make pin multiplexing, pin configuration, GPIO, etc selectable options in one and the same pin control and GPIO subsystem. |