| Message ID | 20230324121240.3594777-1-conor.dooley@microchip.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | d34a6b715a23ccd9c9d0bc7a475bea59dc3e28b2 |
| Headers | show |
| Series | [v1] RISC-V: convert new selectors of RISCV_ALTERNATIVE to dependencies | expand |
| Context | Check | Description |
|---|---|---|
| conchuod/cover_letter | success | Single patches do not need cover letters |
| conchuod/tree_selection | success | Guessed tree name to be for-next |
| conchuod/fixes_present | success | Fixes tag not required for -next series |
| conchuod/maintainers_pattern | success | MAINTAINERS pattern errors before the patch: 1 and now 1 |
| conchuod/verify_signedoff | success | Signed-off-by tag matches author and committer |
| conchuod/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/build_rv64_clang_allmodconfig | success | Errors and warnings before: 18 this patch: 18 |
| conchuod/module_param | success | Was 0 now: 0 |
| conchuod/build_rv64_gcc_allmodconfig | success | Errors and warnings before: 18 this patch: 18 |
| conchuod/build_rv32_defconfig | success | Build OK |
| conchuod/dtb_warn_rv64 | success | Errors and warnings before: 3 this patch: 3 |
| conchuod/header_inline | success | No static functions without inline keyword in header files |
| conchuod/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 19 lines checked |
| conchuod/source_inline | success | Was 0 now: 0 |
| conchuod/build_rv64_nommu_k210_defconfig | success | Build OK |
| conchuod/verify_fixes | success | No Fixes tag |
| conchuod/build_rv64_nommu_virt_defconfig | success | Build OK |
On Fri, Mar 24, 2023 at 12:12:41PM +0000, Conor Dooley wrote: > for-next contains two additional extensions that select > RISCV_ALTERNATIVE. RISCV_ALTERNATIVE no longer needs to be selected by > individual config options as it is now selected for !XIP_KERNEL builds > by the top level RISCV option. > These extensions rely on the alternative framework, so convert the > "select"s to "depends on"s instead. > > Signed-off-by: Conor Dooley <conor.dooley@microchip.com> > --- > Applies on top of my series doing the aforementioned conversion: > https://lore.kernel.org/all/20230324100538.3514663-3-conor.dooley@microchip.com/ > > This patch is sent on its own, because I expect the others to be applied > to fixes, as it fixes a v6.3 regression. > For this patch, I applied my other series to v6.3-rc1 and merged that > into for-next, and created this patch on top of the result. > --- > arch/riscv/Kconfig | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig > index d2acd69d6e3e..a48d1ee677ec 100644 > --- a/arch/riscv/Kconfig > +++ b/arch/riscv/Kconfig > @@ -402,8 +402,8 @@ config RISCV_ISA_C > config RISCV_ISA_SVNAPOT > bool "SVNAPOT extension support" > depends on 64BIT && MMU > + depends on RISCV_ALTERNATIVE > default y > - select RISCV_ALTERNATIVE > help > Allow kernel to detect the SVNAPOT ISA-extension dynamically at boot > time and enable its usage. > @@ -478,8 +478,8 @@ config RISCV_ISA_ZICBOM > > config RISCV_ISA_ZICBOZ > bool "Zicboz extension support for faster zeroing of memory" > - depends on !XIP_KERNEL && MMU > - select RISCV_ALTERNATIVE > + depends on MMU > + depends on RISCV_ALTERNATIVE > default y > help > Enable the use of the ZICBOZ extension (cbo.zero instruction) > -- > 2.39.2 > Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
On Fri, 24 Mar 2023 12:12:41 +0000, Conor Dooley wrote: > for-next contains two additional extensions that select > RISCV_ALTERNATIVE. RISCV_ALTERNATIVE no longer needs to be selected by > individual config options as it is now selected for !XIP_KERNEL builds > by the top level RISCV option. > These extensions rely on the alternative framework, so convert the > "select"s to "depends on"s instead. > > [...] Applied, thanks! [1/1] RISC-V: convert new selectors of RISCV_ALTERNATIVE to dependencies https://git.kernel.org/palmer/c/d34a6b715a23 Best regards,
Hello: This patch was applied to riscv/linux.git (for-next) by Palmer Dabbelt <palmer@rivosinc.com>: On Fri, 24 Mar 2023 12:12:41 +0000 you wrote: > for-next contains two additional extensions that select > RISCV_ALTERNATIVE. RISCV_ALTERNATIVE no longer needs to be selected by > individual config options as it is now selected for !XIP_KERNEL builds > by the top level RISCV option. > These extensions rely on the alternative framework, so convert the > "select"s to "depends on"s instead. > > [...] Here is the summary with links: - [v1] RISC-V: convert new selectors of RISCV_ALTERNATIVE to dependencies https://git.kernel.org/riscv/c/d34a6b715a23 You are awesome, thank you!
diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig index d2acd69d6e3e..a48d1ee677ec 100644 --- a/arch/riscv/Kconfig +++ b/arch/riscv/Kconfig @@ -402,8 +402,8 @@ config RISCV_ISA_C config RISCV_ISA_SVNAPOT bool "SVNAPOT extension support" depends on 64BIT && MMU + depends on RISCV_ALTERNATIVE default y - select RISCV_ALTERNATIVE help Allow kernel to detect the SVNAPOT ISA-extension dynamically at boot time and enable its usage. @@ -478,8 +478,8 @@ config RISCV_ISA_ZICBOM config RISCV_ISA_ZICBOZ bool "Zicboz extension support for faster zeroing of memory" - depends on !XIP_KERNEL && MMU - select RISCV_ALTERNATIVE + depends on MMU + depends on RISCV_ALTERNATIVE default y help Enable the use of the ZICBOZ extension (cbo.zero instruction)
for-next contains two additional extensions that select RISCV_ALTERNATIVE. RISCV_ALTERNATIVE no longer needs to be selected by individual config options as it is now selected for !XIP_KERNEL builds by the top level RISCV option. These extensions rely on the alternative framework, so convert the "select"s to "depends on"s instead. Signed-off-by: Conor Dooley <conor.dooley@microchip.com> --- Applies on top of my series doing the aforementioned conversion: https://lore.kernel.org/all/20230324100538.3514663-3-conor.dooley@microchip.com/ This patch is sent on its own, because I expect the others to be applied to fixes, as it fixes a v6.3 regression. For this patch, I applied my other series to v6.3-rc1 and merged that into for-next, and created this patch on top of the result. --- arch/riscv/Kconfig | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)