[-mm Patch] net/bluetooth/hidp/core.c: Make hidp_setup_input() return int

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: WANG Cong
Date: Saturday, September 22, 2007 - 8:01 pm

This patch does the following things:

- Make hidp_setup_input() return int to indicate errors.
- Check its return value to handle errors.

Signed-off-by: WANG Cong <xiyou.wangcong@gmail.com>

---
 net/bluetooth/hidp/core.c |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

Index: linux-2.6.23-rc6-mm1/net/bluetooth/hidp/core.c
===================================================================
--- linux-2.6.23-rc6-mm1.orig/net/bluetooth/hidp/core.c
+++ linux-2.6.23-rc6-mm1/net/bluetooth/hidp/core.c
@@ -625,7 +625,7 @@ static struct device *hidp_get_device(st
 	return conn ? &conn->dev : NULL;
 }
 
-static inline void hidp_setup_input(struct hidp_session *session, struct hidp_connadd_req *req)
+static inline int hidp_setup_input(struct hidp_session *session, struct hidp_connadd_req *req)
 {
 	struct input_dev *input = session->input;
 	int i;
@@ -669,7 +669,7 @@ static inline void hidp_setup_input(stru
 
 	input->event = hidp_input_event;
 
-	input_register_device(input);
+	return input_register_device(input);
 }
 
 static int hidp_open(struct hid_device *hid)
@@ -823,7 +823,8 @@ int hidp_add_connection(struct hidp_conn
 	session->idle_to = req->idle_to;
 
 	if (session->input)
-		hidp_setup_input(session, req);
+		if ((err = (hidp_setup_input(session, req))))
+			goto failed;
 
 	if (session->hid)
 		hidp_setup_hid(session, req);
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
2.6.23-rc6-mm1, Andrew Morton, (Tue Sep 18, 1:18 am)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Tue Sep 18, 1:24 am)
Re: 2.6.23-rc6-mm1, Kamalesh Babulal, (Tue Sep 18, 2:13 am)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Tue Sep 18, 2:27 am)
Re: 2.6.23-rc6-mm1, Andy Whitcroft, (Tue Sep 18, 2:34 am)
Re: 2.6.23-rc6-mm1, Satyam Sharma, (Tue Sep 18, 2:34 am)
Re: 2.6.23-rc6-mm1, Benjamin Herrenschmidt, (Tue Sep 18, 3:02 am)
Re: 2.6.23-rc6-mm1, Kamalesh Babulal, (Tue Sep 18, 5:07 am)
Re: 2.6.23-rc6-mm1, Valdis.Kletnieks, (Tue Sep 18, 8:07 am)
Re: 2.6.23-rc6-mm1, Miles Lane, (Tue Sep 18, 8:27 am)
Re: 2.6.23-rc6-mm1, Miles Lane, (Tue Sep 18, 8:39 am)
modpost errors ( Re: 2.6.23-rc6-mm1), Gabriel C, (Tue Sep 18, 8:43 am)
Re: 2.6.23-rc6-mm1, Sam Ravnborg, (Tue Sep 18, 8:50 am)
Re: 2.6.23-rc6-mm1, Sam Ravnborg, (Tue Sep 18, 8:52 am)
Re: modpost errors ( Re: 2.6.23-rc6-mm1), Sam Ravnborg, (Tue Sep 18, 8:56 am)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Tue Sep 18, 9:53 am)
Re: 2.6.23-rc6-mm1 (watchdog), Randy Dunlap, (Tue Sep 18, 10:18 am)
Re: 2.6.23-rc6-mm1, Miles Lane, (Tue Sep 18, 10:20 am)
Re: 2.6.23-rc6-mm1 (watchdog), Randy Dunlap, (Tue Sep 18, 10:41 am)
Re: 2.6.23-rc6-mm1, Mel Gorman, (Tue Sep 18, 10:44 am)
Re: 2.6.23-rc6-mm1, Mel Gorman, (Tue Sep 18, 11:05 am)
Re: 2.6.23-rc6-mm1, Greg KH, (Tue Sep 18, 12:16 pm)
Re: 2.6.23-rc6-mm1, Greg KH, (Tue Sep 18, 12:17 pm)
Re: 2.6.23-rc6-mm1, Sam Ravnborg, (Tue Sep 18, 12:17 pm)
2.6.23-rc6-mm1 - Mostly working, with a kbuild oddity, Valdis.Kletnieks, (Tue Sep 18, 12:32 pm)
Re: 2.6.23-rc6-mm1, Miles Lane, (Tue Sep 18, 12:42 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, Rafael J. Wysocki, (Tue Sep 18, 1:21 pm)
Re: 2.6.23-rc6-mm1, Sam Ravnborg, (Tue Sep 18, 1:26 pm)
Re: 2.6.23-rc6-mm1 sparc build error, Mathieu Desnoyers, (Tue Sep 18, 1:54 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, Rafael J. Wysocki, (Tue Sep 18, 1:54 pm)
Re: 2.6.23-rc6-mm1 sparc build error, Andrew Morton, (Tue Sep 18, 2:05 pm)
Re: 2.6.23-rc6-mm1, Benjamin Herrenschmidt, (Tue Sep 18, 2:35 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Tue Sep 18, 2:37 pm)
Re: 2.6.23-rc6-mm1: atomic counter underflow, Mariusz Kozlowski, (Tue Sep 18, 2:45 pm)
Re: 2.6.23-rc6-mm1 - Mostly working, with a kbuild oddity, Valdis.Kletnieks, (Tue Sep 18, 3:03 pm)
Re: [v4l-dvb-maintainer] modpost errors ( Re: 2.6.23-rc6-mm1), Mauro Carvalho Chehab, (Tue Sep 18, 3:06 pm)
Re: 2.6.23-rc6-mm1, Gabriel C, (Tue Sep 18, 3:38 pm)
Re: 2.6.23-rc6-mm1, Gabriel C, (Tue Sep 18, 3:48 pm)
Re: 2.6.23-rc6-mm1 - Mostly working, with a kbuild oddity, Sam Ravnborg, (Wed Sep 19, 12:34 am)
Re: 2.6.23-rc6-mm1: atomic counter underflow, Cornelia Huck, (Wed Sep 19, 1:27 am)
2.6.23-rc6-mm1 -- powerpc link failure, Andy Whitcroft, (Wed Sep 19, 2:28 am)
X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Jiri Slaby, (Wed Sep 19, 4:43 am)
Re: 2.6.23-rc6-mm1 - Mostly working, with a kbuild oddity, Valdis.Kletnieks, (Wed Sep 19, 8:53 am)
2.6.23-rc6-mm1: kgdb support on ppc64 utterly broken, Mel Gorman, (Wed Sep 19, 9:29 am)
Re: 2.6.23-rc6-mm1 -- powerpc link failure, Segher Boessenkool, (Wed Sep 19, 9:36 am)
2.6.23-rc6-mm1 -- mkfs stuck in 'D', Andy Whitcroft, (Wed Sep 19, 9:43 am)
Re: 2.6.23-rc6-mm1: atomic counter underflow, Mariusz Kozlowski, (Wed Sep 19, 9:43 am)
Re: 2.6.23-rc6-mm1 -- powerpc link failure, Andy Whitcroft, (Wed Sep 19, 9:52 am)
Re: 2.6.23-rc6-mm1 - Mostly working, with a kbuild oddity, Sam Ravnborg, (Wed Sep 19, 10:39 am)
Re: 2.6.23-rc6-mm1 -- powerpc link failure, Sam Ravnborg, (Wed Sep 19, 10:44 am)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Wed Sep 19, 10:44 am)
Re: 2.6.23-rc6-mm1: atomic counter underflow, Cornelia Huck, (Wed Sep 19, 11:02 am)
Re: 2.6.23-rc6-mm1, Sam Ravnborg, (Wed Sep 19, 11:48 am)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Andrew Morton, (Wed Sep 19, 12:10 pm)
Re: 2.6.23-rc6-mm1, Sam Ravnborg, (Wed Sep 19, 12:33 pm)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Hugh Dickins, (Wed Sep 19, 1:03 pm)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Wed Sep 19, 1:32 pm)
Re: 2.6.23-rc6-mm1, Gabriel C, (Wed Sep 19, 1:36 pm)
Re: 2.6.23-rc6-mm1, Sam Ravnborg, (Wed Sep 19, 1:43 pm)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Peter Zijlstra, (Wed Sep 19, 1:44 pm)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Andrew Morton, (Wed Sep 19, 2:42 pm)
Re: 2.6.23-rc6-mm1, Tilman Schmidt, (Wed Sep 19, 4:02 pm)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Wed Sep 19, 4:24 pm)
Re: 2.6.23-rc6-mm1, Chuck Ebbert, (Wed Sep 19, 4:28 pm)
Re: 2.6.23-rc6-mm1, David Brownell, (Wed Sep 19, 4:44 pm)
Re: 2.6.23-rc6-mm1, Tilman Schmidt, (Wed Sep 19, 4:55 pm)
Re: 2.6.23-rc6-mm1, Joseph Fannin, (Wed Sep 19, 4:58 pm)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Wed Sep 19, 5:06 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Wed Sep 19, 5:06 pm)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Wed Sep 19, 5:09 pm)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Andrew Morton, (Wed Sep 19, 7:24 pm)
Re: 2.6.23-rc6-mm1, David Brownell, (Wed Sep 19, 9:43 pm)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Wed Sep 19, 11:11 pm)
Re: 2.6.23-rc6-mm1, Kay Sievers, (Thu Sep 20, 12:54 am)
Re: 2.6.23-rc6-mm1 - make access to tasks nsproxy ligther ..., Cedric Le Goater, (Thu Sep 20, 1:41 am)
Re: 2.6.23-rc6-mm1, Alessandro Zummo, (Thu Sep 20, 1:51 am)
Re: 2.6.23-rc6-mm1 - make access to tasks nsproxy ligther ..., Cedric Le Goater, (Thu Sep 20, 2:12 am)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Hugh Dickins, (Thu Sep 20, 4:31 am)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Peter Zijlstra, (Thu Sep 20, 5:15 am)
Re: 2.6.23-rc6-mm1 sparc build error, Guennadi Liakhovetski, (Thu Sep 20, 5:53 am)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Thu Sep 20, 6:17 am)
Build failure on ppc64 drivers/block/ps3disk.c, Mel Gorman, (Thu Sep 20, 6:25 am)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 6:29 am)
Re: Build failure on ppc64 drivers/block/ps3disk.c, Jens Axboe, (Thu Sep 20, 6:32 am)
Re: 2.6.23-rc6-mm1: atomic counter underflow, Cornelia Huck, (Thu Sep 20, 6:35 am)
2.6.23-rc6-mm1: Build failures on ppc64_defconfig, Satyam Sharma, (Thu Sep 20, 6:37 am)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 7:12 am)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 7:47 am)
Re: 2.6.23-rc6-mm1, David Brownell, (Thu Sep 20, 9:15 am)
Re: 2.6.23-rc6-mm1: atomic counter underflow, Pierre Ossman, (Thu Sep 20, 9:30 am)
Re: 2.6.23-rc6-mm1 - make access to tasks nsproxy ligther ..., Cedric Le Goater, (Thu Sep 20, 10:08 am)
Re: kobject: Temporarily save k_name on cleanup for debug ..., Mariusz Kozlowski, (Thu Sep 20, 10:26 am)
Re: 2.6.23-rc6-mm1: atomic counter underflow, Mariusz Kozlowski, (Thu Sep 20, 10:27 am)
Re: 2.6.23-rc6-mm1, David Brownell, (Thu Sep 20, 10:36 am)
Re: 2.6.23-rc6-mm1, Tilman Schmidt, (Thu Sep 20, 12:10 pm)
Re: 2.6.23-rc6-mm1, Tilman Schmidt, (Thu Sep 20, 12:20 pm)
Re: 2.6.23-rc6-mm1, Andrew Morton, (Thu Sep 20, 1:25 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 1:39 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 1:42 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 2:45 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 2:54 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 3:30 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Thu Sep 20, 3:35 pm)
Re: 2.6.23-rc6-mm1, Tilman Schmidt, (Thu Sep 20, 5:53 pm)
Re: [PATCH] Remove broken netfilter binary sysctls from br ..., Eric W. Biederman, (Thu Sep 20, 9:21 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Fri Sep 21, 2:24 am)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Fri Sep 21, 2:25 am)
Re: 2.6.23-rc6-mm1 sparc build error, Mathieu Desnoyers, (Fri Sep 21, 6:51 am)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Fri Sep 21, 7:20 am)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Fri Sep 21, 10:16 am)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Fri Sep 21, 10:18 am)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Fri Sep 21, 10:54 am)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Fri Sep 21, 12:20 pm)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Fri Sep 21, 12:33 pm)
Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound ..., Rafael J. Wysocki, (Fri Sep 21, 12:37 pm)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Fengguang Wu, (Fri Sep 21, 6:55 pm)
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig, Satyam Sharma, (Fri Sep 21, 11:50 pm)
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig, Satyam Sharma, (Fri Sep 21, 11:51 pm)
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig, Satyam Sharma, (Fri Sep 21, 11:54 pm)
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig, Satyam Sharma, (Sat Sep 22, 12:25 am)
[PATCH -mm] iseries_veth: Kill unused variable, Satyam Sharma, (Sat Sep 22, 12:55 am)
Re: 2.6.23-rc6-mm1, Satyam Sharma, (Sat Sep 22, 1:41 am)
Re: 2.6.23-rc6-mm1, Satyam Sharma, (Sat Sep 22, 2:21 am)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Peter Zijlstra, (Sat Sep 22, 6:16 am)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Fengguang Wu, (Sat Sep 22, 6:20 pm)
[-mm Patch] net/bluetooth/hidp/core.c: Make hidp_setup_inp ..., WANG Cong, (Sat Sep 22, 8:01 pm)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Peter Zijlstra, (Sun Sep 23, 6:02 am)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Fengguang Wu, (Sun Sep 23, 8:01 pm)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Sun Sep 23, 8:25 pm)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Peter Zijlstra, (Mon Sep 24, 12:35 am)
Re: X-freeze after clflush changes [Was: 2.6.23-rc6-mm1], Valdis.Kletnieks, (Mon Sep 24, 12:37 am)
Re: 2.6.23-rc6-mm1 -- mkfs stuck in 'D', Fengguang Wu, (Mon Sep 24, 1:12 am)
Re: 2.6.23-rc6-mm1, Mel Gorman, (Mon Sep 24, 4:14 am)
Re: [PATCH] Remove broken netfilter binary sysctls from br ..., Stephen Hemminger, (Mon Sep 24, 1:14 pm)
Re: 2.6.23-rc6-mm1, Greg KH, (Mon Sep 24, 10:35 pm)
Re: 2.6.23-rc6-mm1 -- powerpc link failure, Andy Whitcroft, (Tue Sep 25, 6:40 am)
Re: [PATCH] Remove broken netfilter binary sysctls from br ..., Eric W. Biederman, (Tue Sep 25, 7:03 am)
Re: [PATCH] Remove broken netfilter binary sysctls from br ..., Stephen Hemminger, (Tue Sep 25, 9:12 am)
Re: [PATCH] Remove broken netfilter binary sysctls from br ..., Eric W. Biederman, (Tue Sep 25, 9:38 am)