Hi Gustavo,
On Tue, Jun 21, 2011 at 7:05 PM, Gustavo F. Padovan
<padovan@xxxxxxxxxxxxxx> wrote:
>> static int __init nfc_init(void)
>> {
>> + int rc;
>> +
>> printk(KERN_INFO "NFC Core ver %s\n", VERSION);
>>
>> - return class_register(&nfc_class);
>> + rc = class_register(&nfc_class);
>> + if (rc)
>> + goto err;
>
> Just return rc here and get rid of the label.
>
ok
>> +/**
>> + * nfc_genl_exit() - Deinitialize netlink interface
>> + *
>> + * This exit function unregisters the nfc netlink family.
>> + */
>> +void nfc_genl_exit(void)
>
> You may want __exit here.
The nfc_genl_exit() is called in '__init nfc_init(void)' if any error
occurs. In that case we will have an __exit code inside an __init
code, resulting in a section mismatch.
Aloisio
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
