2013-03-16 14:10:03

by Alexandru Gheorghiu

[permalink] [raw]
Subject: [PATCH] net: bluetooth: Use PTR_RET function

Used PTR_RET function instead of IS_ERR and PTR_ERR.
Patch found using coccinelle.

Signed-off-by: Alexandru Gheorghiu <[email protected]>
---
net/bluetooth/hci_sysfs.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/net/bluetooth/hci_sysfs.c b/net/bluetooth/hci_sysfs.c
index 23b4e24..ff38561 100644
--- a/net/bluetooth/hci_sysfs.c
+++ b/net/bluetooth/hci_sysfs.c
@@ -590,10 +590,8 @@ int __init bt_sysfs_init(void)
bt_debugfs = debugfs_create_dir("bluetooth", NULL);

bt_class = class_create(THIS_MODULE, "bluetooth");
- if (IS_ERR(bt_class))
- return PTR_ERR(bt_class);

- return 0;
+ return PTR_RET(bt_class);
}

void bt_sysfs_cleanup(void)
--
1.7.9.5


2013-03-18 15:03:10

by Alexandru Gheorghiu

[permalink] [raw]
Subject: Re: [PATCH] net: bluetooth: Use PTR_RET function

Sorry, I made the change without checking. I shall resend it.

Thank you,
Alexandru Gheorghiu

On Mon, Mar 18, 2013 at 4:57 PM, Gustavo Padovan <[email protected]>wrote:

> Hi Alexandru,
>
> * Alexandru Gheorghiu <[email protected]> [2013-03-16 16:10:03
> +0200]:
>
> > Used PTR_RET function instead of IS_ERR and PTR_ERR.
> > Patch found using coccinelle.
> >
> > Signed-off-by: Alexandru Gheorghiu <[email protected]>
> > ---
> > net/bluetooth/hci_sysfs.c | 4 +---
> > 1 file changed, 1 insertion(+), 3 deletions(-)
>
> Patch has been applied to bluetooth-next. Thanks.
>
> Gustavo
>

2013-03-18 14:57:11

by Gustavo Padovan

[permalink] [raw]
Subject: Re: [PATCH] net: bluetooth: Use PTR_RET function

Hi Alexandru,

* Alexandru Gheorghiu <[email protected]> [2013-03-16 16:10:03 +0200]:

> Used PTR_RET function instead of IS_ERR and PTR_ERR.
> Patch found using coccinelle.
>
> Signed-off-by: Alexandru Gheorghiu <[email protected]>
> ---
> net/bluetooth/hci_sysfs.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)

Patch has been applied to bluetooth-next. Thanks.

Gustavo