On 2025/1/14 07:16, Rob Herring wrote: >> From: Zijun Hu <quic_zijuhu@xxxxxxxxxxx> >> >> For child node of /reserved-memory, its property 'reg' may contain >> multiple regions, but fdt_scan_reserved_mem_reg_nodes() only takes >> into account the first region, and miss remaining regions. >> >> Give warning message when missing remaining regions. > Can't we just fix it to support more than 1 entry? actually, i ever considered supporting more entries. and find out there are no simple approach to achieve this aim. it may need to change 'struct reserved_mem' to support multiple regions that may also involve 4 RESERVEDMEM_OF_DECLARE instances. RESERVEDMEM_OF_DECLARE(tegra210_emc_table, "nvidia,tegra210-emc-table", tegra210_emc_table_init); RESERVEDMEM_OF_DECLARE(dma, "shared-dma-pool", rmem_dma_setup); RESERVEDMEM_OF_DECLARE(cma, "shared-dma-pool", rmem_cma_setup); RESERVEDMEM_OF_DECLARE(dma, "restricted-dma-pool", rmem_swiotlb_setup); so i finally chose this warning way due to simplification and low risk.