From: "Marek Behún" <kabel@kernel.org>
To: Marek Vasut <marex@denx.de>
Cc: "Pali Rohár" <pali@kernel.org>,
"Philip Oberfichtner" <pro@denx.de>,
u-boot@lists.denx.de,
"Christoph Niedermaier" <cniedermaier@dh-electronics.com>,
"Stefano Babic" <sbabic@denx.de>,
"Marcel Ziswiler" <marcel.ziswiler@toradex.com>,
"Peng Fan" <peng.fan@nxp.com>,
u-boot@dh-electronics.com
Subject: Re: [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig
Date: Tue, 9 Aug 2022 15:18:38 +0200 [thread overview]
Message-ID: <20220809151838.0d3ab2f8@thinkpad> (raw)
In-Reply-To: <20220809151635.01f9a943@thinkpad>
On Tue, 9 Aug 2022 15:16:35 +0200
Marek Behún <kabel@kernel.org> wrote:
> > I agree -- I think your proposal to make this non-user-configurable is
> > changing the behavior, see:
> >
> > Before -- you could comment the entry in include/configs/board.h out to
> > deselect an option.
> > After -- you can flip Kconfig switch to deselect an option.
>
> I don't think we should consider changing sources as something that
> end-user should be expected to do if they want to enable/disable
> something.
If we do, we can also have, if we use "select" instead of "default y on
MVEBU":
Before -- you could comment the entry in include/configs/board.h out to
deselect an option.
After -- you can comment the select in Kconfig :)
Marek
next prev parent reply other threads:[~2022-08-09 13:19 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-09 10:06 [PATCH v3 0/3] ARM: imx6: dh-imx6: Enable d-cache early in SPL Philip Oberfichtner
2022-08-09 10:07 ` [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig Philip Oberfichtner
2022-08-09 10:58 ` Pali Rohár
2022-08-09 11:16 ` Marek Vasut
2022-08-09 11:21 ` Pali Rohár
2022-08-09 11:27 ` Marek Vasut
2022-08-09 11:32 ` Pali Rohár
2022-08-09 11:38 ` Marek Vasut
2022-08-09 12:59 ` Pali Rohár
2022-08-09 13:16 ` Marek Behún
2022-08-09 13:18 ` Marek Behún [this message]
2022-08-09 13:46 ` Tom Rini
2022-08-09 16:03 ` Marek Vasut
2022-08-09 16:36 ` Tom Rini
2022-08-09 18:27 ` Marek Vasut
2022-08-11 10:17 ` Philip Oberfichtner
2022-08-16 14:42 ` Philip Oberfichtner
2022-08-09 14:32 ` Phil Sutter
2022-08-12 8:39 ` Soeren Moch
2022-08-09 10:07 ` [PATCH v3 2/3] ARM: cache: Allow SPL to build cache-pl310.c Philip Oberfichtner
2022-08-09 10:07 ` [PATCH v3 3/3] ARM: imx6: dh-imx6: Enable d-cache early in SPL Philip Oberfichtner
2022-08-09 11:17 ` Marek Vasut
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=20220809151838.0d3ab2f8@thinkpad \
--to=kabel@kernel.org \
--cc=cniedermaier@dh-electronics.com \
--cc=marcel.ziswiler@toradex.com \
--cc=marex@denx.de \
--cc=pali@kernel.org \
--cc=peng.fan@nxp.com \
--cc=pro@denx.de \
--cc=sbabic@denx.de \
--cc=u-boot@dh-electronics.com \
--cc=u-boot@lists.denx.de \
/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.