2014-02-19 6:19 GMT+08:00 Andrew Morton <a...@linux-foundation.org>:
> On Mon, 17 Feb 2014 16:13:34 +0800 Barry Song <21cn...@gmail.com> wrote:
>
>> From: Wei Chen <wei.c...@csr.com>
>>
>> rpmsg_init() always return ERROR code or random integer now, this patch fixes
>> it.
>>
>> ...
>>
>> --- a/drivers/rpmsg/virtio_rpmsg_bus.c
>> +++ b/drivers/rpmsg/virtio_rpmsg_bus.c
>> @@ -1096,9 +1096,10 @@ static int __init rpmsg_init(void)
>> if (ret) {
>> pr_err("failed to register virtio driver: %d\n", ret);
>> bus_unregister(&rpmsg_bus);
>> + return ret;
>> }
>>
>> - return ret;
>> + return 0;
>> }
>
> Look more closely. This patch has no effect...
yes, when i look again i find it does have no effect as successful
bus_register() has made ret 0. but the codes are much unreadable
then....
static int __init rpmsg_init(void)
{
int ret;
ret = bus_register(&rpmsg_bus);
if (ret) {
pr_err("failed to register rpmsg bus: %d\n", ret);
return ret;
}
ret = register_virtio_driver(&virtio_ipc_driver);
if (ret) {
pr_err("failed to register virtio driver: %d\n", ret);
bus_unregister(&rpmsg_bus);
}
return ret;
}
subsys_initcall(rpmsg_init);
-barry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
