Skip to content

Commit

Permalink
Staging: hv: remove function pointer typedefs from NetVscApi.h
Browse files Browse the repository at this point in the history
function pointer typedefs are allowed in the kernel, but only if they
make sense, which they really do not here, as they are not passed around
with any kind of frequency.  So just spell them all out, it makes the
code smaller and easier to understand overall.

Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Greg Kroah-Hartman committed Sep 15, 2009
1 parent 7498261 commit 211ccd6
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 25 deletions.
32 changes: 8 additions & 24 deletions drivers/staging/hv/NetVscApi.h
Original file line number Diff line number Diff line change
Expand Up @@ -35,19 +35,6 @@
/* Fwd declaration */
struct hv_netvsc_packet;

/* Data types */
typedef int (*PFN_ON_OPEN)(struct hv_device *Device);
typedef int (*PFN_ON_CLOSE)(struct hv_device *Device);

typedef void (*PFN_QUERY_LINKSTATUS)(struct hv_device *Device);
typedef int (*PFN_ON_SEND)(struct hv_device *dev,
struct hv_netvsc_packet *packet);
typedef void (*PFN_ON_SENDRECVCOMPLETION)(void *Context);

typedef int (*PFN_ON_RECVCALLBACK)(struct hv_device *dev,
struct hv_netvsc_packet *packet);
typedef void (*PFN_ON_LINKSTATUS_CHANGED)(struct hv_device *dev, u32 Status);

/* Represent the xfer page packet which contains 1 or more netvsc packet */
struct xferpage_packet {
LIST_ENTRY ListEntry;
Expand Down Expand Up @@ -80,12 +67,12 @@ struct hv_netvsc_packet {
struct{
u64 ReceiveCompletionTid;
void *ReceiveCompletionContext;
PFN_ON_SENDRECVCOMPLETION OnReceiveCompletion;
void (*OnReceiveCompletion)(void *context);
} Recv;
struct{
u64 SendCompletionTid;
void *SendCompletionContext;
PFN_ON_SENDRECVCOMPLETION OnSendCompletion;
void (*OnSendCompletion)(void *context);
} Send;
} Completion;

Expand Down Expand Up @@ -114,17 +101,14 @@ struct netvsc_driver {
* This is set by the caller to allow us to callback when we
* receive a packet from the "wire"
*/
PFN_ON_RECVCALLBACK OnReceiveCallback;

PFN_ON_LINKSTATUS_CHANGED OnLinkStatusChanged;
int (*OnReceiveCallback)(struct hv_device *dev,
struct hv_netvsc_packet *packet);
void (*OnLinkStatusChanged)(struct hv_device *dev, u32 Status);

/* Specific to this driver */
PFN_ON_OPEN OnOpen;
PFN_ON_CLOSE OnClose;
PFN_ON_SEND OnSend;
/* PFN_ON_RECVCOMPLETION OnReceiveCompletion; */

/* PFN_QUERY_LINKSTATUS QueryLinkStatus; */
int (*OnOpen)(struct hv_device *dev);
int (*OnClose)(struct hv_device *dev);
int (*OnSend)(struct hv_device *dev, struct hv_netvsc_packet *packet);

void *Context;
};
Expand Down
2 changes: 1 addition & 1 deletion drivers/staging/hv/RndisFilter.c
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ struct rndis_request {

struct rndis_filter_packet {
void *CompletionContext;
PFN_ON_SENDRECVCOMPLETION OnCompletion;
void (*OnCompletion)(void *context);
struct rndis_message Message;
};

Expand Down

0 comments on commit 211ccd6

Please sign in to comment.