Thread (4 messages) flat view 4 messages, 3 authors, 2011-06-17

Re: [PATCH] gx1fb: Fix section mismatch warnings

From: Sedat Dilek <hidden>
Date: 2011-06-17 07:15:15
Also in: lkml

On Thu, Jun 16, 2011 at 9:31 PM, Randy Dunlap [off-list ref] wrote:
On Thu, 6 Jan 2011 15:39:07 +0900 Paul Mundt wrote:
quoted
The problem seems to be because gx1fb_probe is annotated __init. In the
PCI case you want it to be __devinit, and you're also going to want to
annotate the remove function as __devexit and wrap it up with a
__devexit_p().
--
From: Randy Dunlap <redacted>

Fix a chain of section mismatches in geode driver, beginning with:

WARNING: drivers/video/geode/gx1fb.o(.data+0x70): Section mismatch in reference from the variable gx1fb_driver to the function .init.text:gx1fb_probe()
The variable gx1fb_driver references
the function __init gx1fb_probe()
If the reference is valid then annotate the
variable with __init* or __refdata (see linux/init.h) or name the variable:
*_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console

Making the changes that Paul pointed out resulted in a few more
changes being needed, so they are all included here.

Signed-off-by: Randy Dunlap <redacted>
---
Hi Randy,

thanks for taking care of this old issue!
It's around a week I did not compile a linux-next kernel.
I should again enable full section mismatch in my setup.

Regards,
- Sedat -
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help