diff options
author | Mike Frysinger <vapier@gentoo.org> | 2009-07-17 13:13:21 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-07-17 13:13:21 -0400 |
commit | 0741241c6b80bfd58417e95de984d60c9e9ef2a0 (patch) | |
tree | 9fa70e5e918e0735c531988f64b92cb5655baf28 /drivers/connector/connector.c | |
parent | e36aa25a533962b08402530e8443ac804a454e27 (diff) |
connector: make callback argument type explicit
The connector documentation states that the argument to the callback
function is always a pointer to a struct cn_msg, but rather than encode it
in the API itself, it uses a void pointer everywhere. This doesn't make
much sense to encode the pointer in documentation as it prevents proper C
type checking from occurring and can easily allow people to use the wrong
pointer type. So convert the argument type to an explicit struct cn_msg
pointer.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/connector/connector.c')
-rw-r--r-- | drivers/connector/connector.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/connector/connector.c b/drivers/connector/connector.c index fd336c5a9057..3f45669f5d76 100644 --- a/drivers/connector/connector.c +++ b/drivers/connector/connector.c | |||
@@ -269,7 +269,8 @@ static void cn_notify(struct cb_id *id, u32 notify_event) | |||
269 | * | 269 | * |
270 | * May sleep. | 270 | * May sleep. |
271 | */ | 271 | */ |
272 | int cn_add_callback(struct cb_id *id, char *name, void (*callback)(void *)) | 272 | int cn_add_callback(struct cb_id *id, char *name, |
273 | void (*callback)(struct cn_msg *)) | ||
273 | { | 274 | { |
274 | int err; | 275 | int err; |
275 | struct cn_dev *dev = &cdev; | 276 | struct cn_dev *dev = &cdev; |
@@ -351,9 +352,8 @@ static int cn_ctl_msg_equals(struct cn_ctl_msg *m1, struct cn_ctl_msg *m2) | |||
351 | * | 352 | * |
352 | * Used for notification of a request's processing. | 353 | * Used for notification of a request's processing. |
353 | */ | 354 | */ |
354 | static void cn_callback(void *data) | 355 | static void cn_callback(struct cn_msg *msg) |
355 | { | 356 | { |
356 | struct cn_msg *msg = data; | ||
357 | struct cn_ctl_msg *ctl; | 357 | struct cn_ctl_msg *ctl; |
358 | struct cn_ctl_entry *ent; | 358 | struct cn_ctl_entry *ent; |
359 | u32 size; | 359 | u32 size; |