On Tue, Aug 06 2013, Rob Herring wrote:

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.

quoted

+               dma_release_declared_memory(dev);
+}

-- 
Best regards,                                         _     _
.o. | Liege of Serenely Enlightened Majesty of      o' \,=./ `o
..o | Computer Science,  Michał “mina86” Nazarewicz    (o o)
ooo +----<email/xmpp: mpn@google.com>--------------ooO--(_)--Ooo--