On Fri, Mar 31, 2023, at 12:55, Conor Dooley wrote: > On Fri, Mar 31, 2023 at 11:37:30AM +0100, Lad, Prabhakar wrote: >> > > Does that actually work? I don't think it does. > If you try to enable RISCV_ISA_ZICBOM then you won't get > RISC_DMA_NONCOHERENT turned on. Run menuconfig and disable support for > Renesas, SiFive and T-Head SoCs & you can replicate. Right, the circular dependency has to be broken in some form. > I think one of RISCV_ISA_ZICBOM and RISCV_DMA_NONCOHERENT should just be > dropped, although I don't know which one to pick! > Making RISCV_DMA_NONCOHERENT user selectable probably makes the most > sense. That sounds good to me. Arnd