Removed __devexit from talitos_remove() since its also called from
talitos_probe().
WARNING: vmlinux.o(.text+0x28a008): Section mismatch in reference from the function talitos_probe() to the function .devexit.text:talitos_remove()
The function talitos_probe() references a function in an exit section.
Often the function talitos_remove() has valid usage outside the exit section
and the fix is to remove the __devexit annotation of talitos_remove.
Signed-off-by: Kumar Gala <[email protected]>
---
drivers/crypto/talitos.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/crypto/talitos.c b/drivers/crypto/talitos.c
index b6ad3ac..2479348 100644
--- a/drivers/crypto/talitos.c
+++ b/drivers/crypto/talitos.c
@@ -1357,7 +1357,7 @@ static int hw_supports(struct device *dev, __be32 desc_hdr_template)
return ret;
}
-static int __devexit talitos_remove(struct of_device *ofdev)
+static int talitos_remove(struct of_device *ofdev)
{
struct device *dev = &ofdev->dev;
struct talitos_private *priv = dev_get_drvdata(dev);
--
1.5.6.5
On Tue, Dec 02, 2008 at 10:29:16PM -0600, Kumar Gala wrote:
> Removed __devexit from talitos_remove() since its also called from
> talitos_probe().
>
> WARNING: vmlinux.o(.text+0x28a008): Section mismatch in reference from the function talitos_probe() to the function .devexit.text:talitos_remove()
> The function talitos_probe() references a function in an exit section.
> Often the function talitos_remove() has valid usage outside the exit section
> and the fix is to remove the __devexit annotation of talitos_remove.
>
> Signed-off-by: Kumar Gala <[email protected]>
OK, but what about the __devexit_p reference on talitos_remove?
We should remove that too.
Thanks,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
On Dec 3, 2008, at 6:06 AM, Herbert Xu wrote:
> On Tue, Dec 02, 2008 at 10:29:16PM -0600, Kumar Gala wrote:
>> Removed __devexit from talitos_remove() since its also called from
>> talitos_probe().
>>
>> WARNING: vmlinux.o(.text+0x28a008): Section mismatch in reference
>> from the function talitos_probe() to the
>> function .devexit.text:talitos_remove()
>> The function talitos_probe() references a function in an exit
>> section.
>> Often the function talitos_remove() has valid usage outside the
>> exit section
>> and the fix is to remove the __devexit annotation of talitos_remove.
>>
>> Signed-off-by: Kumar Gala <[email protected]>
>
> OK, but what about the __devexit_p reference on talitos_remove?
> We should remove that too.
missed that one. will fix up and post a new patch.
- k
On Dec 3, 2008, at 8:52 AM, Kumar Gala wrote:
>
> On Dec 3, 2008, at 6:06 AM, Herbert Xu wrote:
>
>> On Tue, Dec 02, 2008 at 10:29:16PM -0600, Kumar Gala wrote:
>>> Removed __devexit from talitos_remove() since its also called from
>>> talitos_probe().
>>>
>>> WARNING: vmlinux.o(.text+0x28a008): Section mismatch in reference
>>> from the function talitos_probe() to the
>>> function .devexit.text:talitos_remove()
>>> The function talitos_probe() references a function in an exit
>>> section.
>>> Often the function talitos_remove() has valid usage outside the
>>> exit section
>>> and the fix is to remove the __devexit annotation of talitos_remove.
>>>
>>> Signed-off-by: Kumar Gala <[email protected]>
>>
>> OK, but what about the __devexit_p reference on talitos_remove?
>> We should remove that too.
>
> missed that one. will fix up and post a new patch.
Nevermind. Al bet me to this fix :)
- k