[RFC] Adding export-symbols to specification

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



Hello everyone,


The RFC aims to get feedback and any blockers/objections to adding export-symbols to base devicetree specification to allow for support of base board + runtime connector setups using devicetree overlays. The idea was actually proposed in the linux kernel mailing list by Herve Codina [0] with the devicetree schema and linux kernel implementation. Initial implementations for devicetree compiler [1] and fdtoverlay [2] have also been sent to the mailing lists.


# Introduction

There are a lot of setups, especially in embedded systems that consist of a base connector and addon boards that can be connected to this connector. Here are some examples:

- MikroBus

- GE SUNH

- BeagleCapes, etc


Some of these connectors have runtime detection capabilities (GE SUNH), while some do not (MikroBUS without 1-wire EEPROM). The goal is to decouple the connector on base device tree with the overlay for addon boards. This will allow having 1 overlay for each board that would work with connector devicetree on any board.


One of the issue was referencing resources available on the base board device tree from the addon overlay device tree. Using a nexus node [3] helps decoupling resources and avoid the knowledge of the full base board from the overlay. Indeed, with nexus node, the overlay need to know only about the nexus node itself.

For instance, suppose a connector where a GPIO is connected at PinA. On the base board this GPIO is connected to the GPIO 12 of the SoC GPIO controller.

The base board can describe this GPIO using a nexus node:

    soc_gpio: gpio-controller {
      #gpio-cells = <2>;
    };

    connector1: connector1 {
        /*
         * Nexus node for the GPIO available on the connector.
         * GPIO 0 (Pin A GPIO) is connected to GPIO 12 of the SoC gpio
         * controller
         */
        #gpio-cells = <2>;
        gpio-map = <0 0 &soc_gpio 12 0>;
        gpio-map-mask = <0xf 0x0>;
        gpio-map-pass-thru = <0x0 0xf>;
    };

The connector pin A GPIO can be referenced using:

  <&connector1 0 GPIO_ACTIVE_HIGH>


This implies that the overlay needs to know about exact label that references the connector. This label can be different on a different board and so applying the overlay could fail even if it is used to describe the exact same addon board. Further more, a given base board can have several connectors where the exact same addon board can be connected. In that case, the same overlay cannot be used on both connector. Indeed, the connector labels have to be different.


The export-symbols node solves this issue.


The idea of export-symbols is to have something similar to the global __symbols__ node but local to a specific node. Symbols listed in this export-symbols are local and visible only when an overlay is applied on a node having an export-symbols subnode.


Note: `export-symbols` properties differ from __symbols__ since they are phandles, not path references. This is much easier to work with in overlays as described in [7].


Using export-symbols, our example becomes:

    soc_gpio: gpio-controller {
      #gpio-cells = <2>;
    };

    connector1: connector1 {
        /*
         * Nexus node for the GPIO available on the connector.
         * GPIO 0 (Pin A GPIO) is connected to GPIO 12 of the SoC gpio
         * controller
         */
        #gpio-cells = <2>;
        gpio-map = <0 0 &soc_gpio 12 0>;
        gpio-map-mask = <0xf 0x0>;
        gpio-map-pass-thru = <0x0 0xf>;

        export-symbols {
          connector = <&connector1>;
        };
    };


With that export-symbols node, an overlay applied on connector1 node can have the symbol named 'connector' resolved to connector1. Indeed, the export-symbols node available at connector1 node is used when the overlay is applied. If the overlay has an unresolved 'connector' symbol, it will be resolved to connector1 thanks to export-symbols.


Our overlay using the nexus node can contains:


   node {
      foo-gpio = <&connector 0 GPIO_ACTIVE_HIGH>;
   };


It used the GPIO 0 from the connector it is applied on.

A board with two connectors can be described with:


    connector1: connector1 {
        ...
        export-symbols {
          connector = <&connector1>;
        };
    };

    connector2: connector2 {
        ...
        export-symbols {
          connector = <&connector2>;
        };
    };


In that case, the same overlay with unresolved 'connector' symbol can be applied on both connectors and the correct symbol resolution (connector1 or connector2) will be done.


# Alternatives


Some alternative approaches that were considered:



1. Using aliases.

Currently, it is not possible to update aliases in device tree overlays. I sent a patch a few weeks ago to add this support [4]. However, as was outlined by Rob, this can break existing drivers that used the unused indexes for devices not present in the aliases list.


2. Add support for phandles in `__symbols__`

This has been discussed in the following patch series [5]. However, since there is no way to distinguish between strings and phandles in devicetree (everything is bytestring), the type guessing is awkward. Also, the export-symbol solution is much more flexible than extending the old `__symbols__` node.


3. Add support for path reference resolution to overlays

An approach using `__symbols__` was proposed by Andrew Davis [6]. However, currently, it is difficult to support path reference resolution support to overlays [7]. This limitation makes it difficult to support connector chaining (MikroBUS -> Grove -> Addon board), which is possible in some connectors.


# Some other benefits to export-symbols

1. No need to enable generation of all symbols in base devicetree

Since the nodes used by connector are referenced by properties in `export-symbols`, the symbols table entries for these nodes will be generated, even if symbols generation is not enabled globally. This can help save space, specially in constrained devices.


2. Enables scoping symbol resolution

Does not pollute the global symbols, and can be useful outside addon board setups.


# Why add to specification?

I would like the ability to share the addon board overlays with ZephyrRTOS, which also has boards that support MikroBUS (like BeagleConnect Freedom [8]) and U-Boot. So it would make more sense if this node is part of the specification instead of linux specific.


# What's next?

I am also not sure if there is some sort of process that I have to follow to make changes to the device tree specification, so please inform me if I am missing something, or if there is some documentation regarding submitting proposals for new nodes.


[0]: https://lore.kernel.org/all/20241209151830.95723-1-herve.codina@xxxxxxxxxxx/

[1]: https://lore.kernel.org/all/20250110-export-symbols-v1-1-b6213fcd6c82@xxxxxxxxxxxxxxx/

[2]: https://lore.kernel.org/devicetree-compiler/86a7a08c-d81c-43d4-99fb-d0c4e9777601@xxxxxxxxxxxxxxx/T/#t

[3] https://github.com/devicetree-org/devicetree-specification/blob/v0.4/source/chapter2-devicetree-basics.rst#nexus-nodes-and-specifier-mapping

[4]: https://lore.kernel.org/all/20241110-of-alias-v2-0-16da9844a93e@xxxxxxxxxxxxxxx/T/#t

[5]: https://lore.kernel.org/devicetree-compiler/44bfc9b3-8282-4cc7-8d9a-7292cac663ef@xxxxxx/T/#mbbc181b0ef394b85b76b2024d7e209ebe70f7003

[6]: https://lore.kernel.org/lkml/20240702164403.29067-1-afd@xxxxxx/

[7]: https://lore.kernel.org/devicetree-compiler/6b2dba90-3c52-4933-88f3-b47f96dc7710@xxxxxxxxxxxxxxx/T/#m8259c8754f680b9da7b91f7b7dd89f10da91d8ed

[8]: https://www.beagleboard.org/boards/beagleconnect-freedom


Best Regards,

Ayush Singh





[Index of Archives]     [Device Tree]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Photos]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]

  Powered by Linux