7 Aug
2013
7 Aug
'13
12:38 p.m.
On Tue, Aug 06 2013, Sylwester Nawrocki wrote:
On 08/06/2013 05:07 PM, Michal Nazarewicz wrote:
On Tue, Aug 06 2013, Rob Herring wrote:
+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.
I think Rob's suggestion was to replace two 'if' statements with a single one, which makes sense.
Ah, yes, silly me...
--
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--