On 11/25, Arnd Bergmann wrote: > diff --git a/drivers/clk/mmp/clk-mmp2.c b/drivers/clk/mmp/clk-mmp2.c > index 09d2832fbd78..38931dbd1eff 100644 > --- a/drivers/clk/mmp/clk-mmp2.c > +++ b/drivers/clk/mmp/clk-mmp2.c > @@ -9,6 +9,7 @@ > * warranty of any kind, whether express or implied. > */ > > +#include <linux/clk.h>
Why are we adding this include? I don't see any clk consumer API usage being added. > #include <linux/module.h> > #include <linux/kernel.h> > #include <linux/spinlock.h> > diff --git a/drivers/clk/mmp/clk-pxa168.c b/drivers/clk/mmp/clk-pxa168.c > index 93e967c0f972..0dd83fb950c9 100644 > --- a/drivers/clk/mmp/clk-pxa168.c > +++ b/drivers/clk/mmp/clk-pxa168.c > @@ -9,6 +9,7 @@ > * warranty of any kind, whether express or implied. > */ > > +#include <linux/clk.h> Ditto. > diff --git a/drivers/clk/mmp/clk-pxa910.c b/drivers/clk/mmp/clk-pxa910.c > index 993abcdb32cc..e1d2ce22cdf1 100644 > --- a/drivers/clk/mmp/clk-pxa910.c > +++ b/drivers/clk/mmp/clk-pxa910.c > @@ -9,6 +9,7 @@ > * warranty of any kind, whether express or implied. > */ > > +#include <linux/clk.h> Ditto. > #include <linux/module.h> > #include <linux/kernel.h> > #include <linux/spinlock.h> -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

