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? Thanks, Yunhui
