Re: [Linaro-mm-sig] [RFC/PATCH v2 2/2] drivers: dma-contiguous: add initialization from device tree

/**

• dma_contiguous_reserve() - reserve area(s) for contiguous memory handling
• @limit: End address of the reserved memory (optional, 0 for any).

@@ -149,6 +200,10 @@ void __init dma_contiguous_reserve(phys_addr_t limit)

    pr_debug("%s(limit %08lx)\n", __func__, (unsigned long)limit);

+#ifdef CONFIG_OF

•   of_scan_flat_dt(cma_fdt_scan, NULL);
  

+#endif

What is your expectation with the contention between default region setup via the kernel config (CONFIG_CMA_SIZE_SEL_*) and via the DT? Could the call to 'of_scan_flat_dt()' be done before the setup of the kernel config early regions, followed by a return code check 'fail-over' scheme, or were you intending for the default region setups to be mutually-exclusive?

Thanks, Marc