devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: Masahiro Yamada <masahiroy@kernel.org>,
	DTML <devicetree@vger.kernel.org>
Cc: Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>
Subject: Re: [RFC][PATCH] kbuild: Generate symbols for DTO application in DTBs
Date: Sun, 10 Apr 2022 02:07:02 +0200	[thread overview]
Message-ID: <88e8194a-1657-c35d-7794-dd377289a77c@denx.de> (raw)
In-Reply-To: <CAK7LNATp4_Hjte3tj_tqVLzJu30fidAowkrBgXkjC-68FwFzkQ@mail.gmail.com>

On 4/7/22 14:30, Masahiro Yamada wrote:
> Hi Marek.
> (+CC Rob, DT ML)

Hi,

> On Thu, Apr 7, 2022 at 8:05 PM Marek Vasut <marex@denx.de> wrote:
>>
>> Emit symbols section in DTBs to permit symbol resolution when applying DTOs.
> 
> I CCed DT folks, but if I remember correctly, adding -@ globally
> was NACKed because it would increase blob size for platforms that
> do not need overlay.
> 
> 
>> Signed-off-by: Marek Vasut <marex@denx.de>
>> Cc: Masahiro Yamada <masahiroy@kernel.org>
>> ---
>> NOTE: I am sending this as RFC, because I suspect there is a better way how
>>        to pass extra flags to DTC during kernel build ?
>>        Maybe from shell environment somehow ?
> 
> 
> For local use, yes, you can add -@ from the command line.
> 
> Try this:
> 
>      DTC_FLAGS=-@  make ARCH=arm64 dtbs
> 
> 
> This is undocumented tip, but it seems to work
> for the current code.

This is real nice (and very much what I need), thank you.

>>        Or maybe b7e70391a5451 ("arm64: tegra: Enable device-tree overlay support")
>>        is the way to go about this ?
> 
> 
> Since commit 15d16d6dadf6947ac7f9a686c615995c5a426ce2,
> adding -@ is automatic for platforms that support overlay.
> 
> 
> If  <platform>-dtbs exists, -@ is automatically added.
> 
> See arch/arm64/boot/dts/xilinx/Makefile
> as an example code.

Is it now allowed to start upstreaming DT overlays then ?
That would be nice too.

      reply	other threads:[~2022-04-10  0:07 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20220407110522.122393-1-marex@denx.de>
2022-04-07 12:30 ` [RFC][PATCH] kbuild: Generate symbols for DTO application in DTBs Masahiro Yamada
2022-04-10  0:07   ` Marek Vasut [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=88e8194a-1657-c35d-7794-dd377289a77c@denx.de \
    --to=marex@denx.de \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=masahiroy@kernel.org \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).