On Sat, Jan 10, 2009 at 06:40:34PM +0100, Samuel Ortiz wrote: > On Sat, Jan 10, 2009 at 04:20:48PM +0000, Russell King - ARM Linux wrote: > > On Sat, Jan 10, 2009 at 04:56:12PM +0100, Samuel Ortiz wrote: > > > Hi Tony, > > > > > > On Wed, Jan 07, 2009 at 01:16:15PM +0200, Tony Lindgren wrote: > > > > diff --git a/drivers/mfd/twl4030-core.c b/drivers/mfd/twl4030-core.c > > > > index b59c385..18179ac 100644 > > > > --- a/drivers/mfd/twl4030-core.c > > > > +++ b/drivers/mfd/twl4030-core.c > > > > @@ -38,6 +38,7 @@ > > > > #include <linux/i2c.h> > > > > #include <linux/i2c/twl4030.h> > > > > > > > > +#include <mach/cpu.h> > > > That will break the build on !ARM for this driver. > > > > ... which in itself is an argument for making it use the clk API > > properly so that these kinds of CPU specific dependencies are kept > > out of the driver. > > I agree. > Until that gets done, if we really need to include a header file to fix this > build, I would rather go for a generic one, <linux/io.h> for example. I fear that such an approach will result in the current situation persisting. I've tried to tackle the OMAP clock implementation, and ended up giving up because other people are busy fiddling with it as well. I was rather hoping that these other changes were going to make it into the mainline kernel so I could resume my work on it, but alas that's not happened. And note that I _only_ work with the mainline kernel, so having work sitting in other peoples trees derived from Tony's tree is completely useless. Of course, I could say "sod it, waited long enough, I'm merging my omap-clks1 branch" for the next merge window. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
