On Tue, Aug 6, 2013 at 10:07 AM, Michal Nazarewicz [off-list ref] wrote:

On Tue, Aug 06 2013, Rob Herring wrote:

quoted

quoted

+static void reserved_mem_release_device_from_dt(struct device *dev)
+{
+       struct reserved_mem *region = get_dma_memory_region(dev);
+       if (!region)
+               return;
+       if (!region->cma)

if (region && !region->cma)

!region case is handled two lines above, so no need to check if region
is not null.

Right. Sorry if I was not clear. Combine the 3 lines into 1 line.

Rob