On Fri, 2016-02-19 at 23:02 -0800, Guenter Roeck wrote: > On Wed, Feb 17, 2016 at 08:46:13AM +0800, Yong Wu wrote: > > This patch adds support for mediatek m4u (MultiMedia Memory Management > > Unit). [...] > > +#include <linux/clk.h> > > +#include <linux/component.h> > > +#include <linux/dma-iommu.h> > > +#include <linux/interrupt.h> > > +#include <linux/io.h> > > +#include <linux/iommu.h> > > +#include <linux/iopoll.h> > > +#include <linux/list.h> > > +#include <linux/module.h> > > +#include <linux/of_address.h> > > +#include <linux/of_iommu.h> > > +#include <linux/of_irq.h> > > +#include <linux/of_platform.h> > > +#include <linux/platform_device.h> > > +#include <soc/mediatek/smi.h> > > +#include <dt-bindings/memory/mt8173-larb-port.h> > > + > > Some missing includes: > > linux/bug.h > for WARN_ON() > linux/device.h > for dev_xxx() error messages, dev_get_drvdata(), etc. > linux/err.h > for error functions (IS_ERR, error definitions) > linux/slab.h > for kzalloc() etc. > linux/spinlock.h > for spinlock functions > asm/barrier.h > for wmb() > > Guenter Hi Guenter, Thanks for this suggestion. I'm sorry that I didn't get any build warning or error here. Anyway I will add them and send a quick v10. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html