[PATCH 4/6] ARM: OMAP: fix missing __devexit_p() annotations
Tony Lindgren
tony at atomide.com
Thu Mar 15 12:59:33 EDT 2012
* Jean Pihet <jean.pihet at newoldbits.com> [120315 04:09]:
> Russell,
>
> On Thu, Mar 15, 2012 at 11:30 AM, Russell King
> <rmk+kernel at arm.linux.org.uk> wrote:
> > Missing __devexit_p() annotations in driver structures for remove functions
> > marked with __devexit is waiting for build errors to happen, such as:
> >
> > `omap_system_dma_remove' referenced in section `.data' of arch/arm/plat-omap/bui
> > lt-in.o: defined in discarded section `.devexit.text' of arch/arm/plat-omap/buil
> > t-in.o
> >
> > Add the necessary annotation, and as a result of audit, fix others which
> > are also missing in arch/arm/*omap*.
> >
> > Signed-off-by: Russell King <rmk+kernel at arm.linux.org.uk>
> > Acked-by: Tony Lindgren <tony at atomide.com>
> > ---
> > arch/arm/mach-omap2/smartreflex.c | 2 +-
> > arch/arm/plat-omap/dma.c | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/mach-omap2/smartreflex.c b/arch/arm/mach-omap2/smartreflex.c
> > index 7e755bb..47c77a1 100644
> > --- a/arch/arm/mach-omap2/smartreflex.c
> > +++ b/arch/arm/mach-omap2/smartreflex.c
> > @@ -1012,7 +1012,7 @@ static int __devexit omap_sr_remove(struct platform_device *pdev)
> > }
> >
> > static struct platform_driver smartreflex_driver = {
> > - .remove = omap_sr_remove,
> > + .remove = __devexit_p(omap_sr_remove),
> > .driver = {
> > .name = "smartreflex",
> > },
> Tony already has this change in his master branch, commit
> 149f1d5c8d2fd6879cdf93ca70f1cb745e698a1e ('ARM: OMAP2+: Fix devexit
> for smartreflex when CONFIG_HOTPLUG is not set').
Also same commit in arm-soc/for-next.
Regards,
Tony
More information about the linux-arm-kernel
mailing list