From: Tom Rini <trini@konsulko.com>
To: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Cc: sjg@chromium.org,
"Patrick Delaunay" <patrick.delaunay@foss.st.com>,
"Patrice Chotard" <patrice.chotard@foss.st.com>,
"Liviu Dudau" <liviu.dudau@foss.arm.com>,
"Thomas Fitzsimmons" <fitzsim@fitzsim.org>,
"Matthias Brugger" <mbrugger@suse.com>,
"Michal Simek" <monstr@monstr.eu>, "Pali Rohár" <pali@kernel.org>,
"Marek Behún" <marek.behun@nic.cz>,
"Bin Meng" <bmeng.cn@gmail.com>,
"Heinrich Schuchardt" <xypron.glpk@gmx.de>,
"Stefan Roese" <sr@denx.de>, "Tim Harvey" <tharvey@gateworks.com>,
"Harald Seiler" <hws@denx.de>,
"Sughosh Ganu" <sughosh.ganu@linaro.org>,
"Priyanka Jain" <priyanka.jain@nxp.com>,
"Vladimir Oltean" <vladimir.oltean@nxp.com>,
"Andre Przywara" <andre.przywara@arm.com>,
"Ramon Fried" <rfried.dev@gmail.com>,
"Nicolas Saenz Julienne" <nsaenzjulienne@suse.de>,
"Peter Robinson" <pbrobinson@gmail.com>,
"Ivan T. Ivanov" <iivanov@suse.de>,
"Marek Szyprowski" <m.szyprowski@samsung.com>,
"Leo Yu-Chi Liang" <ycliang@andestech.com>,
"Zong Li" <zong.li@sifive.com>,
"Sean Anderson" <seanga2@gmail.com>,
"Rick Chen" <rick@andestech.com>,
"Green Wan" <green.wan@sifive.com>,
"T Karthik Reddy" <t.karthik.reddy@xilinx.com>,
"Ashok Reddy Soma" <ashok.reddy.soma@xilinx.com>,
"Alper Nebi Yasak" <alpernebiyasak@gmail.com>,
"AKASHI Takahiro" <takahiro.akashi@linaro.org>,
"Wasim Khan" <wasim.khan@nxp.com>,
"Alexandru Gagniuc" <mr.nuke.me@gmail.com>,
"Nandor Han" <nandor.han@vaisala.com>,
"Steffen Jaeckel" <jaeckel-floss@eyet-services.de>,
"Heiko Schocher" <hs@denx.de>,
"Asherah Connor" <ashe@kivikakk.ee>,
u-boot@lists.denx.de, uboot-stm32@st-md-mailman.stormreply.com
Subject: Re: [PATCH v3] sandbox: Remove OF_HOSTFILE
Date: Mon, 18 Oct 2021 16:50:33 -0400 [thread overview]
Message-ID: <20211018205033.GA3078606@bill-the-cat> (raw)
In-Reply-To: <20211015084243.285423-1-ilias.apalodimas@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 1334 bytes --]
On Fri, Oct 15, 2021 at 11:42:40AM +0300, Ilias Apalodimas wrote:
> OF_HOSTFILE is used on sandbox configs only. Although it's pretty
> unique and rarely causes confusion, we are better of having simpler
> config options for the DTB.
>
> So let's replace that with the existing OF_BOARD. U-Boot would then
> have only three config options for the DTB origin.
> - OF_SEPARATE, build separately from U-Boot
> - OF_BOARD, board specific way of providing the DTB
> - OF_EMBED embedded in the u-boot binary(should not be used in production)
>
> Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
> ---
> Note that this must be applied on top of
> https://lore.kernel.org/u-boot/20211011210016.135929-1-ilias.apalodimas@linaro.org/
> changes since v2:
> - Rebased on top of the updated OF_BOARD patchset
> Changes since v1:
> - Added internal error value on board_fdt_blob_setup(). Arguably
> we can just check against NULL and simplify this even more if we
> don't care about the errno
> - OF_BOARD is now default for sandbox builds
With the previous series applied, this breaks xilinx_zynq_virt:
https://source.denx.de/u-boot/u-boot/-/jobs/337428
I had a little trouble in general getting the board to run locally, even
without the patches, so I didn't debug further yet.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
next prev parent reply other threads:[~2021-10-18 20:51 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-15 8:42 [PATCH v3] sandbox: Remove OF_HOSTFILE Ilias Apalodimas
2021-10-18 20:50 ` Tom Rini [this message]
2021-10-19 0:29 ` Tom Rini
2021-10-19 7:56 ` Ilias Apalodimas
2021-10-19 11:16 ` Ilias Apalodimas
2021-10-19 12:54 ` Tom Rini
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=20211018205033.GA3078606@bill-the-cat \
--to=trini@konsulko.com \
--cc=alpernebiyasak@gmail.com \
--cc=andre.przywara@arm.com \
--cc=ashe@kivikakk.ee \
--cc=ashok.reddy.soma@xilinx.com \
--cc=bmeng.cn@gmail.com \
--cc=fitzsim@fitzsim.org \
--cc=green.wan@sifive.com \
--cc=hs@denx.de \
--cc=hws@denx.de \
--cc=iivanov@suse.de \
--cc=ilias.apalodimas@linaro.org \
--cc=jaeckel-floss@eyet-services.de \
--cc=liviu.dudau@foss.arm.com \
--cc=m.szyprowski@samsung.com \
--cc=marek.behun@nic.cz \
--cc=mbrugger@suse.com \
--cc=monstr@monstr.eu \
--cc=mr.nuke.me@gmail.com \
--cc=nandor.han@vaisala.com \
--cc=nsaenzjulienne@suse.de \
--cc=pali@kernel.org \
--cc=patrice.chotard@foss.st.com \
--cc=patrick.delaunay@foss.st.com \
--cc=pbrobinson@gmail.com \
--cc=priyanka.jain@nxp.com \
--cc=rfried.dev@gmail.com \
--cc=rick@andestech.com \
--cc=seanga2@gmail.com \
--cc=sjg@chromium.org \
--cc=sr@denx.de \
--cc=sughosh.ganu@linaro.org \
--cc=t.karthik.reddy@xilinx.com \
--cc=takahiro.akashi@linaro.org \
--cc=tharvey@gateworks.com \
--cc=u-boot@lists.denx.de \
--cc=uboot-stm32@st-md-mailman.stormreply.com \
--cc=vladimir.oltean@nxp.com \
--cc=wasim.khan@nxp.com \
--cc=xypron.glpk@gmx.de \
--cc=ycliang@andestech.com \
--cc=zong.li@sifive.com \
/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