Hi Uwe, On Tue, Jan 26, 2021 at 8:21 AM Uwe Kleine-König <uwe@xxxxxxxxxxxxxxxxx> wrote: > And then I learned with hints from Rob and Geert that symbols are not > really necessary for overlays, you just cannot use named labels. But > using > > target-path = "/soc/i2c@23473245"; > > or > > target = <&{/soc/i2c@23473245}>; > > instead of > > target = <&i2c1>; > > works fine. (And if you need to add a phandle the &{/path/to/node} > construct should work, too (but I didn't test).) Using labels is a tad > nicer, but the problem I wanted to address with my patch now has a known > different solution. Please don't use "target" and "target-path". Since the introduction of sugar syntax support in v4.15[1], you can just use "&label", like in a normal DTS file. Paths do need the special "&{/path/to/node}" syntax instead of "/path/to/node", though. As usual, you can find lots of examples of DT overlays in my repo[2]. [1] commit 4201d057ea91c3d6 ("scripts/dtc: Update to upstream version v1.4.5-3-gb1a60033c110") [2] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/renesas-overlays Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds