diff mbox series

RISC-V: defconfig: Remove ARCH_RENESAS

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

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