Re: [PATCH 1/6] omap iommu: tlb and pagetable primitives

From: David Brownell
Date: Sat Jan 17 2009 - 15:48:59 EST


On Saturday 17 January 2009, Russell King - ARM Linux wrote:
> > +#ifdef DEBUG_VERBOSE
> > +             dev_dbg(obj->dev, "%s:\ta new pte:%p\n", __func__, iopte);
> > +#endif
>
> Not sure wrapping these in DEBUG_VERBOSE is necessary.  dev_dbg() is a
> no-op unless DEBUG is defined.

Use dev_vdbg() to get the "NOP unless VERBOSE is set" semantics.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/