Thread (1 message) 1 message, 1 author, 2011-12-19

[PATCH V2 1/4] i2c: tegra: Add __devinit/exit to probe/remove

From: Stephen Warren <hidden>
Date: 2011-12-19 16:59:44
Also in: linux-devicetree, linux-i2c, linux-tegra

Shubhrajyoti Datta wrote at Sunday, December 18, 2011 11:54 PM:
On Sun, Dec 18, 2011 at 11:59 AM, Stephen Warren [off-list ref] wrote:
quoted
This fixes some section mismatch build warnings.
What was the error was a devinit function called from these?
As I Understand It, probe/remove are devinit/exit functions, and hence
should be marked as such.

The errors were from building with CONFIG_DEBUG_SECTION_MISMATCH=y.

I guess the section mismatch errors won't actually show up in the code
that's in the kernel right now; they only showed up because I had a local
patch that modified probe() to reference tegra_i2c_of_match[] which is
__devinitconst. I since reworked that patch so that it doesn't introduce
that reference, for other reasons. Still, I think this is a good cleanup
change even if nothing is actually causing these mismatch errors to fire
right now.

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