pussuw commented on PR #6069: URL: https://github.com/apache/incubator-nuttx/pull/6069#issuecomment-1099182189
Thanks for the comments, I'll remove the FAR qualifiers in a bunch, as they are not needed for riscv. I will keep the kernel mappings mpfs specific due to the reason mentioned. The way I did it is very restrictive, all I/O regions mapped with vaddr=paddr, also kernel memory is mapped with vaddr=paddr. To support full mapping to any location is an enormous effort, especially the I/O region, as every single driver needs to be fixed. So let's keep the mapping file in mpfs for now. It can serve as an example or if other targets want to do more flexible mappings they can do so. The riscv_addrenv.c implementation only takes the L1 kernel mappings and when they are copied to the user mappings, everything will just work! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
