MIME-Version: 1.0
References: <20230309233945.1199358-1-echanude@redhat.com> <20230313124923.iwaknvrnitqal3wa@ripper>
In-Reply-To: <20230313124923.iwaknvrnitqal3wa@ripper>
From:   Rob Herring <robh+dt@kernel.org>
Date:   Tue, 14 Mar 2023 10:14:00 -0500
Message-ID: <CAL_JsqLCM59bZakpq3sp0oR06Kv7aCN5LSA7tNwO-89Tx7SaQQ@mail.gmail.com>
Subject: Re: [PATCH] arm64: dts: qcom: sa8775p: add symbols to dtb
To:     Bjorn Andersson <andersson@kernel.org>,
        Eric Chanudet <echanude@redhat.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Andy Gross <agross@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Mon, Mar 13, 2023 at 7:46=E2=80=AFAM Bjorn Andersson <andersson@kernel.o=
rg> wrote:
>
> On Thu, Mar 09, 2023 at 06:39:48PM -0500, Eric Chanudet wrote:
> > ABL uses the __symbols__ section to process the DTB before passing it
> > forward. Without it, the bootstrap is interrupted.

No clue what ABL is... Put this in high level terms, "I want to be
able to apply overlays", not some unknown acronym and low-level detail
of how overlays work.

> >
> > Signed-off-by: Eric Chanudet <echanude@redhat.com>
>
> @Rob, @Krzysztof, it seems useful to be able to use the upstream
> generated DTBs with overlays.
>
> Do you suggest that we enable this on a per-board basis when it's being
> requested, across all devices, or tell the users that they have to
> re-generate the dtbs themselves?

It's up to you whether you want the bloat in the dtbs or not. Labels
become an ABI when symbols are enabled.

If it was my platform(s), I'd require the overlays to be upstream too,
but I'm just paranoid about the modifications people want to make.

Rob