[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1e713129-db54-64c3-59e3-fc0dba0e0f19@gmail.com>
Date: Tue, 26 Jan 2021 12:03:56 -0600
From: Frank Rowand <frowand.list@...il.com>
To: Geert Uytterhoeven <geert@...ux-m68k.org>,
Uwe Kleine-König <uwe@...ine-koenig.org>,
Frank Rowand <frowand.list@...il.com>
Cc: DTML <devicetree@...r.kernel.org>,
Michal Marek <michal.lkml@...kovi.net>,
Arnd Bergmann <arnd@...db.de>,
Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
Masahiro Yamada <masahiroy@...nel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Maxime Ripard <mripard@...nel.org>, cyril@...amax.com,
linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] cmd_dtc: Enable generation of device tree symbols
+frank
On 1/26/21 2:43 AM, Geert Uytterhoeven wrote:
> Hi Uwe,
>
> On Tue, Jan 26, 2021 at 8:21 AM Uwe Kleine-König <uwe@...ine-koenig.org> 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@...73245";
>>
>> or
>>
>> target = <&{/soc/i2c@...73245}>;
>>
>> 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
>
Powered by blists - more mailing lists