Re: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS
> From: Zong Li > Sent: Tuesday, November 29, 2022 10:02 AM > To: Sean Anderson > Cc: s...@chromium.org; michal.si...@amd.com; sean.ander...@seco.com; Leo > Yu-Chi Liang(梁育齊) ; Rick Jian-Zhi Chen(陳建志) > ; u-boot@lists.denx.de > Subject: Re: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS > > On Mon, Nov 21, 2022 at 8:17 PM Zong Li wrote: > > > > On Mon, Nov 21, 2022 at 12:00 PM Sean Anderson 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 > > > > --- > > > > 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? It's reasonable for me. Thanks, Rick > > > > --Sean
Re: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS
On Mon, Nov 21, 2022 at 8:17 PM Zong Li wrote: > > On Mon, Nov 21, 2022 at 12:00 PM Sean Anderson 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 > > > --- > > > 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
Re: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS
On Mon, Nov 21, 2022 at 12:00 PM Sean Anderson 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 > > --- > > 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 > --Sean
Re: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS
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 --- 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? --Sean
Re: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS
On Wed, Nov 16, 2022 at 3:09 PM 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 > --- > arch/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Bin Meng
Re: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS
> From: Zong Li > Sent: Wednesday, November 16, 2022 3:09 PM > To: s...@chromium.org; michal.si...@amd.com; sean.ander...@seco.com; Leo > Yu-Chi Liang(梁育齊) ; Rick Jian-Zhi Chen(陳建志) > ; u-boot@lists.denx.de > Cc: Zong Li > Subject: [PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS > > 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 > --- > arch/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Rick Chen
[PATCH] riscv: use imply instead of select for SPL_SEPARATE_BSS
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 --- 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 -- 2.17.1