diff mbox series

RISC-V: defconfig: Remove ARCH_RENESAS

Message ID 20240917131936.18980-1-palmer@rivosinc.com (mailing list archive)
State New
Headers show
Series RISC-V: defconfig: Remove ARCH_RENESAS | expand

Checks

Context Check Description
conchuod/vmtest-for-next-PR success PR summary
conchuod/patch-1-test-1 success .github/scripts/patches/tests/build_rv32_defconfig.sh took 127.97s
conchuod/patch-1-test-2 success .github/scripts/patches/tests/build_rv64_clang_allmodconfig.sh took 1260.52s
conchuod/patch-1-test-3 success .github/scripts/patches/tests/build_rv64_gcc_allmodconfig.sh took 1490.26s
conchuod/patch-1-test-4 success .github/scripts/patches/tests/build_rv64_nommu_k210_defconfig.sh took 19.74s
conchuod/patch-1-test-5 success .github/scripts/patches/tests/build_rv64_nommu_virt_defconfig.sh took 21.98s
conchuod/patch-1-test-6 success .github/scripts/patches/tests/checkpatch.sh took 0.39s
conchuod/patch-1-test-7 success .github/scripts/patches/tests/dtb_warn_rv64.sh took 41.91s
conchuod/patch-1-test-8 success .github/scripts/patches/tests/header_inline.sh took 0.00s
conchuod/patch-1-test-9 success .github/scripts/patches/tests/kdoc.sh took 0.58s
conchuod/patch-1-test-10 success .github/scripts/patches/tests/module_param.sh took 0.01s
conchuod/patch-1-test-11 success .github/scripts/patches/tests/verify_fixes.sh took 0.00s
conchuod/patch-1-test-12 success .github/scripts/patches/tests/verify_signedoff.sh took 0.03s

Commit Message

Palmer Dabbelt Sept. 17, 2024, 1:19 p.m. UTC
The RZ/Five is gated behind CONFIG_NONPORTABLE because of the DMA pool
issue.  e36ddf322686 ("riscv: defconfig: Disable RZ/Five peripheral
support") removes the drivers, so let's remove the rest of the vendor
support as well -- users of these will already have to play aruond with
kernel configs, so this shouldn't be much of a burden.

Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
---
 arch/riscv/configs/defconfig | 1 -
 1 file changed, 1 deletion(-)

Comments

Sergey Shtylyov Sept. 17, 2024, 1:40 p.m. UTC | #1
On 9/17/24 16:19, Palmer Dabbelt wrote:

> The RZ/Five is gated behind CONFIG_NONPORTABLE because of the DMA pool
> issue.  e36ddf322686 ("riscv: defconfig: Disable RZ/Five peripheral

   Commit e36ddf322686?

> support") removes the drivers, so let's remove the rest of the vendor
> support as well -- users of these will already have to play aruond with

   Around. :-)

> kernel configs, so this shouldn't be much of a burden.
> 
> Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
[...]

MBR, Sergey
Geert Uytterhoeven Sept. 24, 2024, 3:03 p.m. UTC | #2
Hi Palmer,

On Tue, Sep 17, 2024 at 3:20 PM Palmer Dabbelt <palmer@rivosinc.com> wrote:
> The RZ/Five is gated behind CONFIG_NONPORTABLE because of the DMA pool
> issue.  e36ddf322686 ("riscv: defconfig: Disable RZ/Five peripheral
> support") removes the drivers, so let's remove the rest of the vendor
> support as well -- users of these will already have to play aruond with
> kernel configs, so this shouldn't be much of a burden.
>
> Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>

Thanks for your patch!

> --- a/arch/riscv/configs/defconfig
> +++ b/arch/riscv/configs/defconfig
> @@ -27,7 +27,6 @@ CONFIG_EXPERT=y
>  # CONFIG_SYSFS_SYSCALL is not set
>  CONFIG_PROFILING=y
>  CONFIG_ARCH_MICROCHIP=y
> -CONFIG_ARCH_RENESAS=y
>  CONFIG_ARCH_SIFIVE=y
>  CONFIG_ARCH_SOPHGO=y
>  CONFIG_SOC_STARFIVE=y

I had planned to send a v2 of the aforementioned commit, including the
above, but didn't get to that. Sorry...

With CONFIG_ARCH_RENESAS=y, you can also drop the following
line:

    # CONFIG_USB_XHCI_RCAR is not set

(USB_XHCI_RCAR depends on and defaults to ARCH_RENESAS)

Gr{oetje,eeting}s,

                        Geert
diff mbox series

Patch

diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig
index b51ef6cd1e39..7b896a0f78fc 100644
--- a/arch/riscv/configs/defconfig
+++ b/arch/riscv/configs/defconfig
@@ -27,7 +27,6 @@  CONFIG_EXPERT=y
 # CONFIG_SYSFS_SYSCALL is not set
 CONFIG_PROFILING=y
 CONFIG_ARCH_MICROCHIP=y
-CONFIG_ARCH_RENESAS=y
 CONFIG_ARCH_SIFIVE=y
 CONFIG_ARCH_SOPHGO=y
 CONFIG_SOC_STARFIVE=y