mirror of
https://github.com/zephyrproject-rtos/zephyr
synced 2025-09-09 17:11:58 +00:00
b97ed9e4b4
Generating generic information for 'type: phandle-array' properties in edtlib was difficult due to defining phandle-array as just a list of phandles and numbers. To make sense of a phandle-array property like 'pwms', you have to know that #pwm-cells is expected to appear on each referenced controller, and that the binding for the controller has a #cells. Because of this, handling of various 'type: phandle-array' properties was previously hardcoded in edtlib and exposed through properties like Node.pwms, instead of through the generic Node.props (though with a lot of shared code). In practice, it turns out that all 'type: phandle-array' properties in Zephyr work exactly the same way: They all have names that end in -s, the 's' is removed to derive the name of related properties, and they all look up #cells in the binding for the controller, which gives names to the data values. Strengthen the definition of 'type: phandle-array' to mean a property that works exactly like the existing phandle-array properties (which also means requiring that the name ends in -s). This removes a ton of hardcoding from edtlib and allows new 'type: phandle-array' properties to be added without making any code changes. If we ever need a property type that's a list of phandles and numbers but that doesn't follow this scheme, then we could add a separate type for it. We should check if the standard scheme is fine first though. The only property type for which no information is generated is now 'compound'. There's some inconsistency in how we generate identifiers for clocks compared to other 'type: phandle-array' properties, so keep special-casing them for now in gen_defines.py (see the comment in write_clocks()). This change also enabled a bunch of other simplifications, like reusing the ControllerAndData class for interrupts. Piggyback generalization of *-map properties so that they work for any phandle-array properties. It's now possible to have things like 'io-channel-map', if you need to. Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no> |
||
|---|---|---|
| .. | ||
| arc | ||
| arm | ||
| audio | ||
| base | ||
| bluetooth | ||
| can | ||
| clock | ||
| cpu | ||
| crypto | ||
| display | ||
| espi | ||
| ethernet | ||
| flash_controller | ||
| gpio | ||
| hwinfo | ||
| i2c | ||
| i2s | ||
| ieee802154 | ||
| iio/adc | ||
| interrupt-controller | ||
| ipm | ||
| led | ||
| led_strip | ||
| memory-controllers | ||
| mhu | ||
| misc | ||
| mmc | ||
| mmu_mpu | ||
| modem | ||
| mtd | ||
| phy | ||
| pinctrl | ||
| power | ||
| ps2 | ||
| pwm | ||
| riscv | ||
| rng | ||
| rtc | ||
| sensor | ||
| serial | ||
| spi | ||
| sram | ||
| timer | ||
| usb | ||
| watchdog | ||
| wifi | ||
| vendor-prefixes.txt | ||