[PATCH] ARM: integrator: fix section mismatch problem

Russell King - ARM Linux linux at arm.linux.org.uk
Wed Jun 4 03:57:26 PDT 2014


On Wed, Jun 04, 2014 at 12:57:02PM +0200, Arnd Bergmann wrote:
> On Wednesday 04 June 2014 12:53:22 Linus Walleij wrote:
> > This addresses a section mismatch problem in the IM-PD1
> > driver in the Integrator/AP.
> > 
> > The IM-PD1 contains a VIC interrupt controller and therefore
> > the driver calls vic_init_cascaded() which is marked __init as
> > irqchips are simply not hot-pluggable and specifically the VIC
> > is assumed to initiate only on boot.
> > 
> > However the module driver model of the Integrator LM bus
> > assumes that logic tile drivers can be probed at runtime. This
> > is not really the case for IM-PD1: these tiles are detected at
> > boot and they cannot be plugged into a running system. Before
> > this patch it is of course possible to modprobe them later.
> > 
> > By first forcing the IM-PD1 to bool we make sure this driver
> > gets compiled into the kernel, and we know it will be probed
> > only at boot time when the tiles are detected, so we can tag
> > its probe function __init_refok as we know it won't be called
> > after boot now, and the section mismatch problem goes away.
> > 
> > Cc: Arnd Bergmann <arnd at arndb.de>
> > Cc: Russell King <linux at arm.linux.org.uk>
> > Signed-off-by: Linus Walleij <linus.walleij at linaro.org>
> > ---
> > ARM SoC folks: please apply this directly if the solution seems
> > acceptable.
> 
> Looks good to me, I'm applying this on top of another fix
> I have just added to next/soc2.

That's a NAK on that until the issue I've raised has been checked.

-- 
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.



More information about the linux-arm-kernel mailing list