On Wed, Sep 24, 2008 at 4:13 AM, Felipe Balbi <me@xxxxxxxxxxxxxxx> wrote: > On Tue, Sep 23, 2008 at 08:53:09PM +0530, Arun KS wrote: >> This patch is a temporary fix for a kernel panic causing at omap1_clk_init >> >> Signed-off-by: Arun KS <arunks@xxxxxxxxxxxxxxxxxxxx> >> --- >> arch/arm/plat-omap/include/mach/clock.h | 4 ++++ >> 1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/plat-omap/include/mach/clock.h >> b/arch/arm/plat-omap/include/mach/clock.h >> index f26ad8c..b9c471e 100644 >> --- a/arch/arm/plat-omap/include/mach/clock.h >> +++ b/arch/arm/plat-omap/include/mach/clock.h >> @@ -67,7 +67,11 @@ struct clk { >> struct clk *parent; >> unsigned long rate; >> __u32 flags; >> +#if defined(CONFIG_ARCH_OMAP1) >> + void __iomem *enable_reg; >> +#elif >> u16 enable_reg; >> +#endif > > Did you try removing those (void __iomem *) casts from > arch/arm/mach-omap1/clock.h ?? > > with vim you can fix it by: > > :%s/(void\ __iomem\ \*)//g > This is not working, still panic. enable_reg variable is u16 now(which is offset). clock.c file of omap2 is modified to support this, but clock.c of omap1 is not. > -- > balbi > -- > 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 > -- 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