On Sun, Oct 04 2026, Marek Vasut <[email protected]> wrote:
> Since DTC 7c78c8542d73 ("Added empty node name check") , U-Boot binman
> use of placeholder node names such as @fdt-SEQ and @config-SEQ triggers
> DTC ERROR (node_name_not_empty). Inhibit the error, as these placeholder
> nodes are updated by binman and replaced by proper nodes, the encoding
> in DTB is temporary. Since -Eno-node_name_not_empty is only available in
> DTC 1.8.0 and newer, introduce "dtc-option" make function which is very
> similar to "cc-option" and other such Kbuild functions, and add the DTC
> option conditionally.
Is this the best way to handle this? This also removes the error for
empty node names that might appear in the "real" part of the dtb, no?
I'm not really familiar with the binman magic, but is there any reason
those placeholder nodes cannot be binman@fdt-SEQ and binman@config-SEQ,
or anything like that? Sure, the binman code would need updating and
there would be some -u-boot.dtsi files to update, but I'm currently
counting less than 70 instances, about half in tests, so it would not be
a lot of churn, and changing it now would mean we avoid having to carry
around that dtc flag forever.
Rasmus
