]> git.baikalelectronics.ru Git - uboot.git/commit
fdtdec: drop needlessly convoluted CONFIG_PHANDLE_CHECK_SEQ
authorRasmus Villemoes <rasmus.villemoes@prevas.dk>
Thu, 19 May 2022 09:10:43 +0000 (11:10 +0200)
committerTom Rini <trini@konsulko.com>
Mon, 6 Jun 2022 22:01:21 +0000 (18:01 -0400)
commitd0a25807ffdf47f93952b296f1a82f58992eb544
treefb964eaab8c62edc153ceaf47fd11db121d3b960
parent71ae78cd1b9caba265bdc3792d2c990e85af057f
fdtdec: drop needlessly convoluted CONFIG_PHANDLE_CHECK_SEQ

Asking if the alias we found actually points at the device tree node
we passed in (in the guise of its offset from blob) can be done simply
by asking if the fdt_path_offset() of the alias' path is identical to
offset.

In fact, the current method suffers from the possibility of false
negatives: dtc does not necessarily emit a phandle property for a node
just because it is referenced in /aliases; it only emits a phandle
property for a node if it is referenced in <angle brackets>
somewhere. So if both the node we passed in and the alias node we're
considering don't have phandles, fdt_get_phandle() returns 0 for both.

Since the proper check is so simple, there's no reason to hide that
behind a config option (and if one really wanted that, it should be
called something else because there's no need to involve phandle in
the check).

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Acked-by: Aswath Govindraju <a-govindraju@ti.com>
configs/am65x_evm_a53_defconfig
configs/evb-ast2600_defconfig
configs/sama7g5ek_mmc1_defconfig
configs/sama7g5ek_mmc_defconfig
lib/Kconfig
lib/fdtdec.c