Re: [PATCH 15/16] fdt: Make OF_BOARD a bool option

[Simon Glass <sjg@chromium.org>]

Hi Heinrich,

On Tue, 12 Oct 2021 at 22:22, Heinrich Schuchardt
<heinrich.schuchardt@canonical.com> wrote:
>
>
>
> On 10/13/21 03:01, Simon Glass wrote:
> > This should not be a separate option from OF_SEPARATE. It is a run-time
> > option to override the devicetree, even if present.
> >
> > Move the option out of the choice.
> >
> > Disable BINMAN_FDT for a few boards which don't actually use it.
>
> You only sent patch 6/16 and 15/16 to me. No clue why. Please, send
> complete patch sets instead of selected patches which cannot be reviewed
> without the context.

I actually only explicitly cc'd Ilias (who has been working on similar
things) and Tom. Everyone else came in via maintainer files. Can you
see the rest in the mailing list, assuming you are subscribed? I can
add you to the next version.

For the record, here is what patman produced:

$ patman -n
Cleaned 16 patches
...
checkpatch.pl found 0 error(s), 14 warning(s), 0 checks(s)
Not sending emails due to errors/warnings
Dry run, so not doing much. But I would do this:

Send a total of 16 patches with a cover letter.
   0001-arm-qemu-Mention-nographic-in-the-docs.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  qemu-devel@nongnu.org
   0002-arm-qemu-Explain-how-to-extract-the-generate-devicet.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  qemu-devel@nongnu.org
   0003-riscv-qemu-Explain-how-to-extract-the-generate-devic.patch
      Cc:  Atish Patra <atish.patra@wdc.com>
      Cc:  Bin Meng <bin.meng@windriver.com>
      Cc:  Heinrich Schuchardt <xypron.glpk@gmx.de>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  qemu-devel@nongnu.org
   0004-arm-qemu-Add-a-devicetree-file-for-qemu_arm.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
      Cc:  Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
      Cc:  qemu-devel@nongnu.org
   0005-arm-qemu-Add-a-devicetree-file-for-qemu_arm64.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
      Cc:  Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
      Cc:  qemu-devel@nongnu.org
   0006-riscv-qemu-Add-devicetree-files-for-qemu_riscv32-64.patch
      Cc:  Bin Meng <bmeng.cn@gmail.com>
      Cc:  David Abdurachmanov <david.abdurachmanov@sifive.com>
      Cc:  Dimitri John Ledkov <dimitri.ledkov@canonical.com>
      Cc:  Green Wan <green.wan@sifive.com>
      Cc:  Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
      Cc:  Jonathan Balkind <jbalkind@ucsb.edu>
      Cc:  Leo <ycliang@andestech.com>
      Cc:  Padmarao Begari <padmarao.begari@microchip.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Tianrui Wei <tianrui-wei@outlook.com>
      Cc:  Zong Li <zong.li@sifive.com>
      Cc:  qemu-devel@nongnu.org
   0007-arm-rpi-Add-a-devicetree-file-for-rpi_4.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Matthias Brugger <mbrugger@suse.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Stephen Warren <swarren@nvidia.com>
      Cc:  Stephen Warren <swarren@wwwdotorg.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
   0008-arm-vexpress-Add-a-devicetree-file-for-juno.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Bin Meng <bin.meng@windriver.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Linus Walleij <linus.walleij@linaro.org>
      Cc:  Liviu Dudau <liviu.dudau@foss.arm.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
   0009-arm-xenguest_arm64-Add-a-fake-devicetree-file.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Anastasiia Lukianenko <anastasiia_lukianenko@epam.com>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Bin Meng <bin.meng@windriver.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
   0010-arm-octeontx-Add-a-fake-devicetree-file.patch
      Cc:  Aaron Williams <awilliams@marvell.com>
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Stefan Roese <sr@denx.de>
      Cc:  Tero Kristo <kristo@kernel.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
   0011-arm-xilinx_versal_virt-Add-a-devicetree-file.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Michal Simek <michal.simek@xilinx.com>
      Cc:  Michal Simek <monstr@monstr.eu>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  T Karthik Reddy <t.karthik.reddy@xilinx.com>
      Cc:  Tim Harvey <tharvey@gateworks.com>
   0012-arm-bcm7xxx-Add-a-devicetree-file.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Thomas Fitzsimmons <fitzsim@fitzsim.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
   0013-arm-qemu-ppce500-Add-a-devicetree-file.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Alexander Graf <agraf@csgraf.de>
      Cc:  Bin Meng <bmeng.cn@gmail.com>
      Cc:  Niel Fourie <lusus@denx.de>
      Cc:  Priyanka Jain <priyanka.jain@nxp.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Sinan Akman <sinan@writeme.com>
      Cc:  Stefan Roese <sr@denx.de>
      Cc:  Valentin Longchamp <valentin.longchamp@hitachi-powergrids.com>
      Cc:  Vladimir Oltean <vladimir.oltean@nxp.com>
      Cc:  Wolfgang Denk <wd@denx.de>
   0014-arm-highbank-Add-a-fake-devicetree-file.patch
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Ramon Fried <rfried.dev@gmail.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Tim Harvey <tharvey@gateworks.com>
   0015-fdt-Make-OF_BOARD-a-bool-option.patch
      Cc:  Bin Meng <bmeng.cn@gmail.com>
      Cc:  Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
      Cc:  Jerry Van Baren <vanbaren@cideas.com>
      Cc:  Leo Yu-Chi Liang <ycliang@andestech.com>
      Cc:  Priyanka Jain <priyanka.jain@nxp.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Vladimir Oltean <vladimir.oltean@nxp.com>
   0016-Drop-CONFIG_BINMAN_STANDALONE_FDT.patch
      Cc:  Bin Meng <bmeng.cn@gmail.com>
      Cc:  Marek Behún <marek.behun@nic.cz>
      Cc:  Pali Rohár <pali@kernel.org>
      Cc:  Simon Glass <sjg@chromium.org>
To:   U-Boot Mailing List <u-boot@lists.denx.de>
To:   u-boot@lists.denx.de
Cc:   Ilias Apalodimas <ilias.apalodimas@linaro.org>
Cc:   Tom Rini <trini@konsulko.com>
Version:  None
Prefix:   None
Cover: 36 lines
      Cc:  Aaron Williams <awilliams@marvell.com>
      Cc:  Albert Aribaud <albert.u.boot@aribaud.net>
      Cc:  Alexander Graf <agraf@csgraf.de>
      Cc:  Anastasiia Lukianenko <anastasiia_lukianenko@epam.com>
      Cc:  Andre Przywara <andre.przywara@arm.com>
      Cc:  Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>
      Cc:  Atish Patra <atish.patra@wdc.com>
      Cc:  Bin Meng <bin.meng@windriver.com>
      Cc:  Bin Meng <bmeng.cn@gmail.com>
      Cc:  Christian Hewitt <christianshewitt@gmail.com>
      Cc:  David Abdurachmanov <david.abdurachmanov@sifive.com>
      Cc:  Dimitri John Ledkov <dimitri.ledkov@canonical.com>
      Cc:  Fabio Estevam <festevam@gmail.com>
      Cc:  Green Wan <green.wan@sifive.com>
      Cc:  Heiko Schocher <hs@denx.de>
      Cc:  Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
      Cc:  Heinrich Schuchardt <xypron.glpk@gmx.de>
      Cc:  Jagan Teki <jagan@amarulasolutions.com>
      Cc:  Jerry Van Baren <vanbaren@cideas.com>
      Cc:  Jonathan Balkind <jbalkind@ucsb.edu>
      Cc:  Kever Yang <kever.yang@rock-chips.com>
      Cc:  Leo <ycliang@andestech.com>
      Cc:  Leo Yu-Chi Liang <ycliang@andestech.com>
      Cc:  Linus Walleij <linus.walleij@linaro.org>
      Cc:  Liviu Dudau <liviu.dudau@foss.arm.com>
      Cc:  Marek Behún <marek.behun@nic.cz>
      Cc:  Matthias Brugger <mbrugger@suse.com>
      Cc:  Michal Simek <michal.simek@xilinx.com>
      Cc:  Michal Simek <monstr@monstr.eu>
      Cc:  Neil Armstrong <narmstrong@baylibre.com>
      Cc:  Niel Fourie <lusus@denx.de>
      Cc:  Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
      Cc:  Padmarao Begari <padmarao.begari@microchip.com>
      Cc:  Pali Rohár <pali@kernel.org>
      Cc:  Peter Robinson <pbrobinson@gmail.com>
      Cc:  Priyanka Jain <priyanka.jain@nxp.com>
      Cc:  Ramon Fried <rfried.dev@gmail.com>
      Cc:  Rick Chen <rick@andestech.com>
      Cc:  Sean Anderson <seanga2@gmail.com>
      Cc:  Simon Glass <sjg@chromium.org>
      Cc:  Sinan Akman <sinan@writeme.com>
      Cc:  Stefan Roese <sr@denx.de>
      Cc:  Stephen Warren <swarren@nvidia.com>
      Cc:  Stephen Warren <swarren@wwwdotorg.org>
      Cc:  T Karthik Reddy <t.karthik.reddy@xilinx.com>
      Cc:  Tero Kristo <kristo@kernel.org>
      Cc:  Thomas Fitzsimmons <fitzsim@fitzsim.org>
      Cc:  Tianrui Wei <tianrui-wei@outlook.com>
      Cc:  Tim Harvey <tharvey@gateworks.com>
      Cc:  Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
      Cc:  Valentin Longchamp <valentin.longchamp@hitachi-powergrids.com>
      Cc:  Vladimir Oltean <vladimir.oltean@nxp.com>
      Cc:  Wolfgang Denk <wd@denx.de>
      Cc:  Zong Li <zong.li@sifive.com>
      Cc:  qemu-devel@nongnu.org

Regards,
Simon