Received-SPF: pass (google.com: domain of linux-kernel+bounces-82676-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240225151209.343160-1-alexander.stein@mailbox.org> <CAK7LNARxfhYm9WyDH-S0PT7Mb5Tj5tmbqGLs9NzNxzsi1_4OQw@mail.gmail.com>
In-Reply-To: <CAK7LNARxfhYm9WyDH-S0PT7Mb5Tj5tmbqGLs9NzNxzsi1_4OQw@mail.gmail.com>
From: Rob Herring <robh+dt@kernel.org>
Date: Mon, 26 Feb 2024 22:13:16 -0600
Message-ID: <CAL_JsqJp5aHJiibQ6=dcBPKQ8k=sK2Nbgter7Y7hczJP1AO0yg@mail.gmail.com>
Subject: Re: [PATCH 1/1] kbuild: Add DT schema check also when applying DT overlays
To: Masahiro Yamada <masahiroy@kernel.org>
Cc: Alexander Stein <alexander.stein@mailbox.org>, DTML <devicetree@vger.kernel.org>, 
	Nathan Chancellor <nathan@kernel.org>, Nicolas Schier <nicolas@fjasle.eu>, linux-kbuild@vger.kernel.org, 
	linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Mon, Feb 26, 2024 at 1:03=E2=80=AFAM Masahiro Yamada <masahiroy@kernel.o=
rg> wrote:
>
> + Rob Herring, DT List
>
>
> On Mon, Feb 26, 2024 at 12:12=E2=80=AFAM Alexander Stein
> <alexander.stein@mailbox.org> wrote:
> >
> > Add a similar mechanism as for regular DT files to call the DT checker
> > after applying DT overlays to base DT files.

I suspect this needs more work than just enabling checking. Otherwise,
I would have done just that. Depending on the split with an overlay,
we could see strange results.

The requirement in the kernel is overlays be applied to a base tree at
build time (though that is not checked ATM. When we check that result,
we're checking the overlay anyways. Obviously, that's a bit harder to
trace the source of errors, but it is otherwise redundant to check
overlays in addition.

Rob