Messages in this thread | ![/](/images/icornerl.gif) | | From | Ard Biesheuvel <> | Date | Mon, 26 Jun 2023 08:42:56 +0200 | Subject | Re: [External] Re: [PATCH] firmware: added a firmware information passing method FFI |
| |
On Mon, 26 Jun 2023 at 04:35, 运辉崔 <cuiyunhui@bytedance.com> wrote: > > Hi Ron, Ard, > > On Sun, Jun 25, 2023 at 11:57 PM ron minnich <rminnich@gmail.com> 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@kernel.org> 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
| ![\](/images/icornerr.gif) |