[PATCH] crypto: talitos - Fix modpost warning

[PATCH] crypto: talitos - Fix modpost warning

[Kumar Gala]

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 <galak@kernel.crashing.org>
---
 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

Re: [PATCH] crypto: talitos - Fix modpost warning

[Herbert Xu]

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 <galak@kernel.crashing.org>

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~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Re: [PATCH] crypto: talitos - Fix modpost warning

[Kumar Gala]

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 <galak@kernel.crashing.org>
>
> 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

Re: [PATCH] crypto: talitos - Fix modpost warning

[Kumar Gala]

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 <galak@kernel.crashing.org>
>>
>> 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