On Mon, Nov 21, 2022 at 8:17 PM Zong Li <zong...@sifive.com> wrote: > > On Mon, Nov 21, 2022 at 12:00 PM Sean Anderson <sean...@gmail.com> wrote: > > > > On 11/16/22 02:08, Zong Li wrote: > > > Use imply instead of select, then it can still be disabled by > > > board-specific defconfig, or be set to n manually. > > > > > > Signed-off-by: Zong Li <zong...@sifive.com> > > > --- > > > arch/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/arch/Kconfig b/arch/Kconfig > > > index ae39716697..102956d24c 100644 > > > --- a/arch/Kconfig > > > +++ b/arch/Kconfig > > > @@ -111,7 +111,7 @@ config RISCV > > > select SUPPORT_OF_CONTROL > > > select OF_CONTROL > > > select DM > > > - select SPL_SEPARATE_BSS if SPL > > > + imply SPL_SEPARATE_BSS if SPL > > > imply DM_SERIAL > > > imply DM_ETH > > > imply DM_EVENT > > > > Do you have an example of a board which does this? > > > > Hi Sean, > We'd like to disable 'SPL_SEPARATE_BSS' on our internal platforms that > don't exist in the mainline. It seems to me that using 'imply' might > be not only working on the mainline's board, but also making it > flexible to disable 'SPL_SEPARATE_BSS' by board-specific configuration > or disable it manually for debug purposes. Hope the idea is good to > you all. Thanks >
Hi all, Thanks for your reviewing, could I know whether this patch makes sense to you? > > --Sean