On Mon, 26 Jun 2023 at 04:35, 运辉崔 <cuiyunhui@xxxxxxxxxxxxx> wrote: > > Hi Ron, Ard, > > On Sun, Jun 25, 2023 at 11:57 PM ron minnich <rminnich@xxxxxxxxx> wrote: > > > > Hey Ard, thanks for the discussion, sounds like we are able to move forward now! > > > > > On Sun, Jun 25, 2023, 6:13 AM Ard Biesheuvel <ardb@xxxxxxxxxx> wrote: > >> > >> If this is only used on RISC-V, and implemented under arch/riscv, I > >> have no objections. > > Thank you for your suggestions that made us reach an agreement, let's > continue to review this patch. > > The current logic is to implement the common interface under > drivers/firmware/, if we need this function, we can call > fdt_fwtbl_init() to complete it in arch/xxx/kernel/setup.c. > > For enabling on RISC-V, we can complete it in a subsequent patch to > setup_arch-->fdt_fwtbl_init() in arch/riscv/kernel/setup.c. > > What do you think? > I think all of this belongs under arch/riscv
