From: Simon Glass <sjg@chromium.org>
To: Sean Anderson <seanga2@gmail.com>
Cc: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>,
Tom Rini <trini@konsulko.com>,
Ilias Apalodimas <ilias.apalodimas@linaro.org>,
Bin Meng <bmeng.cn@gmail.com>,
Jerry Van Baren <vanbaren@cideas.com>,
Leo Yu-Chi Liang <ycliang@andestech.com>,
Priyanka Jain <priyanka.jain@nxp.com>,
Rick Chen <rick@andestech.com>,
Vladimir Oltean <vladimir.oltean@nxp.com>,
U-Boot Mailing List <u-boot@lists.denx.de>
Subject: Re: [PATCH 15/16] fdt: Make OF_BOARD a bool option
Date: Sun, 24 Oct 2021 13:53:14 -0600 [thread overview]
Message-ID: <CAPnjgZ0H_pMu=ukmx=MT9JHi54yZCNDkLYBtnOxU4NxURZxF-Q@mail.gmail.com> (raw)
In-Reply-To: <17991b96-ed1e-4b63-2ad0-784a1f864fdc@gmail.com>
Hi Sean,
On Wed, 13 Oct 2021 at 11:30, Sean Anderson <seanga2@gmail.com> wrote:
>
> On 10/13/21 12:58 PM, Simon Glass wrote:
> > 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:
>
> FWIW I have always thought that patman CCs too many people by default.
> It is only partially based on MAINTAINERS; the rest of the CCs are from
> people who have touched the file recently. This results in many people
> being CC'd who have made only minor or unrelated changes to the code
> (especially from treewide changes). For this reason, I curate my CCs
> manually to hopefully CC only those people who are actually interested
> in my patches.
Patman uses only the commit tags and get_maintainers, so the fault may
live with the latter?
Regards,
Simon
next prev parent reply other threads:[~2021-10-24 19:53 UTC|newest]
Thread overview: 164+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-13 1:01 [PATCH 00/16] fdt: Make OF_BOARD a boolean option Simon Glass
2021-10-13 1:01 ` Simon Glass
2021-10-13 1:01 ` [PATCH 01/16] arm: qemu: Mention -nographic in the docs Simon Glass
2021-10-13 1:01 ` Simon Glass
2021-10-13 1:01 ` [PATCH 02/16] arm: qemu: Explain how to extract the generate devicetree Simon Glass
2021-10-13 1:01 ` Simon Glass
2021-10-13 1:19 ` François Ozog
2021-10-13 1:19 ` François Ozog
2021-10-13 16:58 ` Simon Glass
2021-10-13 16:58 ` Simon Glass
2021-10-13 17:36 ` Tom Rini
2021-10-13 17:36 ` Tom Rini
2021-10-13 1:01 ` [PATCH 03/16] riscv: " Simon Glass
2021-10-13 1:01 ` Simon Glass
2021-10-13 1:01 ` [PATCH 04/16] arm: qemu: Add a devicetree file for qemu_arm Simon Glass
2021-10-13 1:01 ` Simon Glass
2021-10-13 1:01 ` [PATCH 05/16] arm: qemu: Add a devicetree file for qemu_arm64 Simon Glass
2021-10-13 1:01 ` Simon Glass
2021-10-13 1:15 ` François Ozog
2021-10-13 1:15 ` François Ozog
2021-10-27 14:44 ` Alex Bennée
2021-10-27 14:44 ` Alex Bennée
2021-10-27 14:56 ` Tom Rini
2021-10-27 14:56 ` Tom Rini
2021-10-27 18:34 ` Simon Glass
2021-10-27 18:34 ` Simon Glass
2021-10-27 18:39 ` Tom Rini
2021-10-27 18:39 ` Tom Rini
2021-10-27 19:45 ` Alex Bennée
2021-10-27 19:45 ` Alex Bennée
2021-10-13 1:01 ` [PATCH 06/16] riscv: qemu: Add devicetree files for qemu_riscv32/64 Simon Glass
2021-10-13 1:01 ` Simon Glass
2021-10-13 4:21 ` Heinrich Schuchardt
2021-10-13 4:21 ` Heinrich Schuchardt
2021-10-13 1:01 ` [PATCH 07/16] arm: rpi: Add a devicetree file for rpi_4 Simon Glass
2021-10-13 1:24 ` François Ozog
2021-10-13 1:01 ` [PATCH 08/16] arm: vexpress: Add a devicetree file for juno Simon Glass
2021-10-13 1:01 ` [PATCH 09/16] arm: xenguest_arm64: Add a fake devicetree file Simon Glass
2021-10-13 1:01 ` [PATCH 10/16] arm: octeontx: " Simon Glass
2021-10-13 1:27 ` François Ozog
2021-10-13 1:01 ` [PATCH 11/16] arm: xilinx_versal_virt: Add a " Simon Glass
2021-10-13 6:13 ` Michal Simek
2021-10-13 16:58 ` Simon Glass
2021-10-13 1:01 ` [PATCH 12/16] arm: bcm7xxx: " Simon Glass
2021-10-13 1:01 ` [PATCH 13/16] arm: qemu-ppce500: " Simon Glass
2021-10-13 1:01 ` [PATCH 14/16] arm: highbank: Add a fake " Simon Glass
2021-10-13 1:01 ` [PATCH 15/16] fdt: Make OF_BOARD a bool option Simon Glass
2021-10-13 4:22 ` Heinrich Schuchardt
2021-10-13 16:58 ` Simon Glass
2021-10-13 17:30 ` Sean Anderson
2021-10-24 19:53 ` Simon Glass [this message]
2021-10-13 1:01 ` [PATCH 16/16] Drop CONFIG_BINMAN_STANDALONE_FDT Simon Glass
2021-10-13 1:29 ` [PATCH 00/16] fdt: Make OF_BOARD a boolean option Bin Meng
2021-10-13 1:29 ` Bin Meng
2021-10-13 1:34 ` Tom Rini
2021-10-13 1:34 ` Tom Rini
2021-10-13 8:02 ` François Ozog
2021-10-13 8:02 ` François Ozog
2021-10-13 14:47 ` Simon Glass
2021-10-13 14:47 ` Simon Glass
2021-10-13 17:34 ` François Ozog
2021-10-13 17:34 ` François Ozog
2021-10-13 18:06 ` Simon Glass
2021-10-13 18:06 ` Simon Glass
2021-10-14 14:56 ` Tom Rini
2021-10-14 14:56 ` Tom Rini
2021-10-14 15:17 ` Simon Glass
2021-10-14 15:17 ` Simon Glass
2021-10-14 15:28 ` Tom Rini
2021-10-14 15:28 ` Tom Rini
2021-10-14 17:58 ` François Ozog
2021-10-14 17:58 ` François Ozog
2021-10-15 18:03 ` Simon Glass
2021-10-15 18:03 ` Simon Glass
2021-10-26 6:46 ` Ilias Apalodimas
2021-10-26 6:46 ` Ilias Apalodimas
2021-10-27 12:59 ` Tom Rini
2021-10-27 12:59 ` Tom Rini
2021-10-27 13:30 ` François Ozog
2021-10-27 13:30 ` François Ozog
2021-10-27 13:38 ` Tom Rini
2021-10-27 13:38 ` Tom Rini
2021-10-27 13:47 ` Ilias Apalodimas
2021-10-27 13:47 ` Ilias Apalodimas
2021-10-27 14:26 ` Tom Rini
2021-10-27 14:26 ` Tom Rini
2021-10-27 13:48 ` François Ozog
2021-10-27 13:48 ` François Ozog
2021-10-27 14:30 ` Tom Rini
2021-10-27 14:30 ` Tom Rini
2021-10-28 2:50 ` Simon Glass
2021-10-28 2:50 ` Simon Glass
2021-10-28 8:21 ` François Ozog
2021-10-28 8:21 ` François Ozog
2021-10-28 14:30 ` Simon Glass
2021-10-28 14:30 ` Simon Glass
2021-10-28 14:50 ` François Ozog
2021-10-28 14:50 ` François Ozog
2021-10-28 15:44 ` Simon Glass
2021-10-28 15:44 ` Simon Glass
2021-10-28 16:25 ` François Ozog
2021-10-28 16:25 ` François Ozog
2021-11-02 14:59 ` Simon Glass
2021-11-02 14:59 ` Simon Glass
2021-11-01 11:04 ` Ilias Apalodimas
2021-11-01 11:04 ` Ilias Apalodimas
2021-11-02 10:06 ` Michael Walle
2021-11-02 10:06 ` Michael Walle
2021-11-02 12:34 ` François Ozog
2021-11-02 12:34 ` François Ozog
2021-11-02 14:59 ` Simon Glass
2021-11-02 14:59 ` Simon Glass
2021-10-27 12:48 ` Tom Rini
2021-10-27 12:48 ` Tom Rini
2021-10-27 13:15 ` François Ozog
2021-10-27 13:15 ` François Ozog
2021-10-27 13:23 ` Heinrich Schuchardt
2021-10-27 13:23 ` Heinrich Schuchardt
2021-10-27 14:55 ` Tom Rini
2021-10-27 14:55 ` Tom Rini
2021-10-27 15:02 ` Heinrich Schuchardt
2021-10-27 15:02 ` Heinrich Schuchardt
2021-10-27 18:04 ` Tom Rini
2021-10-27 18:04 ` Tom Rini
2021-10-27 14:54 ` Tom Rini
2021-10-27 14:54 ` Tom Rini
2021-10-27 15:10 ` Mark Kettenis
2021-10-27 15:10 ` Mark Kettenis
2021-10-27 15:24 ` Simon Glass
2021-10-27 15:24 ` Simon Glass
2021-10-27 18:06 ` Tom Rini
2021-10-27 18:06 ` Tom Rini
2021-10-27 18:11 ` François Ozog
2021-10-27 18:11 ` François Ozog
2021-10-27 21:52 ` Mark Kettenis
2021-10-27 21:52 ` Mark Kettenis
2021-10-27 16:02 ` François Ozog
2021-10-27 16:02 ` François Ozog
2021-10-27 19:06 ` Tom Rini
2021-10-27 19:06 ` Tom Rini
2021-10-27 22:00 ` François Ozog
2021-10-27 22:00 ` François Ozog
2021-10-28 14:41 ` Tom Rini
2021-10-28 14:41 ` Tom Rini
2021-10-14 16:24 ` Andre Przywara
2021-10-14 16:24 ` Andre Przywara
2021-10-14 17:48 ` François Ozog
2021-10-14 17:48 ` François Ozog
2021-10-14 18:12 ` François Ozog
2021-10-14 18:12 ` François Ozog
2021-10-14 21:00 ` Simon Glass
2021-10-14 21:00 ` Simon Glass
2021-10-13 12:39 ` Philippe Mathieu-Daudé
2021-10-13 12:39 ` Philippe Mathieu-Daudé
2021-10-13 13:06 ` François Ozog
2021-10-13 13:06 ` François Ozog
2021-10-13 4:26 ` Heinrich Schuchardt
2021-10-13 4:26 ` Heinrich Schuchardt
2021-10-13 13:06 ` François Ozog
2021-10-13 13:06 ` François Ozog
2021-10-13 9:50 ` Andre Przywara
2021-10-13 9:50 ` Andre Przywara
2021-10-13 13:05 ` François Ozog
2021-10-13 13:05 ` François Ozog
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='CAPnjgZ0H_pMu=ukmx=MT9JHi54yZCNDkLYBtnOxU4NxURZxF-Q@mail.gmail.com' \
--to=sjg@chromium.org \
--cc=bmeng.cn@gmail.com \
--cc=heinrich.schuchardt@canonical.com \
--cc=ilias.apalodimas@linaro.org \
--cc=priyanka.jain@nxp.com \
--cc=rick@andestech.com \
--cc=seanga2@gmail.com \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
--cc=vanbaren@cideas.com \
--cc=vladimir.oltean@nxp.com \
--cc=ycliang@andestech.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.