Skip to content

Commit

Permalink
Staging: hv: Remove typedef DRIVER_OBJECT and PDRIVER_OBJECT
Browse files Browse the repository at this point in the history
typedef DRIVER_OBJECT and PDRIVER_OBJECT are removed and their usages
are replace by the use of struct hv_driver and struct hv_driver *
respectively.

Here is the semantic patch generated to perform this transformation:
(http://coccinelle.lip6.fr/)

//<smpl>
@rm_DRIVER_OBJECT@
@@
-typedef struct _DRIVER_OBJECT
+struct hv_driver
{...}
-DRIVER_OBJECT
;

@rm_PDRIVER_OBJECT@
@@
-typedef struct _DRIVER_OBJECT *PDRIVER_OBJECT;
+struct hv_driver;

@fixtypedef_DRIVER_OBJECT@
typedef DRIVER_OBJECT;
@@
-DRIVER_OBJECT
+struct hv_driver

@fixstruct__DRIVER_OBJECT@
@@
struct
-_DRIVER_OBJECT
+hv_driver

@fixtypedef_PDRIVER_OBJECT@
typedef PDRIVER_OBJECT;
@@
-PDRIVER_OBJECT
+struct hv_driver*
//</smpl>

Signed-off-by: Nicolas Palix <npalix@diku.dk>
Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Nicolas Palix authored and Greg Kroah-Hartman committed Sep 15, 2009
1 parent b7d7ae6 commit 775ef25
Show file tree
Hide file tree
Showing 10 changed files with 37 additions and 43 deletions.
2 changes: 1 addition & 1 deletion drivers/staging/hv/BlkVsc.c
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ BlkVscOnDeviceAdd(

int
BlkVscInitialize(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
)
{
STORVSC_DRIVER_OBJECT* storDriver = (STORVSC_DRIVER_OBJECT*)Driver;
Expand Down
6 changes: 3 additions & 3 deletions drivers/staging/hv/NetVsc.c
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ NetVscOnDeviceRemove(

static void
NetVscOnCleanup(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
);

static void
Expand Down Expand Up @@ -231,7 +231,7 @@ static inline struct NETVSC_DEVICE *ReleaseInboundNetDevice(struct hv_device *De
--*/
int
NetVscInitialize(
DRIVER_OBJECT *drv
struct hv_driver *drv
)
{
NETVSC_DRIVER_OBJECT* driver = (NETVSC_DRIVER_OBJECT*)drv;
Expand Down Expand Up @@ -983,7 +983,7 @@ NetVscOnDeviceRemove(
--*/
void
NetVscOnCleanup(
DRIVER_OBJECT *drv
struct hv_driver *drv
)
{
DPRINT_ENTER(NETVSC);
Expand Down
4 changes: 2 additions & 2 deletions drivers/staging/hv/RndisFilter.c
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ RndisFilterOnDeviceRemove(

static void
RndisFilterOnCleanup(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
);

static int
Expand Down Expand Up @@ -1032,7 +1032,7 @@ RndisFilterOnDeviceRemove(

static void
RndisFilterOnCleanup(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
)
{
DPRINT_ENTER(NETVSC);
Expand Down
6 changes: 3 additions & 3 deletions drivers/staging/hv/StorVsc.c
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ StorVscOnHostReset(

static void
StorVscOnCleanup(
DRIVER_OBJECT *Device
struct hv_driver *Device
);

static void
Expand Down Expand Up @@ -263,7 +263,7 @@ static inline STORVSC_DEVICE* FinalReleaseStorDevice(struct hv_device *Device)
--*/
int
StorVscInitialize(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
)
{
STORVSC_DRIVER_OBJECT* storDriver = (STORVSC_DRIVER_OBJECT*)Driver;
Expand Down Expand Up @@ -807,7 +807,7 @@ StorVscOnIORequest(
--*/
void
StorVscOnCleanup(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
)
{
DPRINT_ENTER(STORVSC);
Expand Down
20 changes: 10 additions & 10 deletions drivers/staging/hv/Vmbus.c
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ static const GUID gVmbusDeviceId={
.Data = {0xfc, 0x60, 0x37, 0xac, 0xdf, 0x9a, 0xaa, 0x40, 0x94, 0x27, 0xa7, 0x0e, 0xd6, 0xde, 0x95, 0xc5}
};

static DRIVER_OBJECT* gDriver; /* vmbus driver object */
static struct hv_driver *gDriver; /* vmbus driver object */
static struct hv_device* gDevice; /* vmbus root device */


Expand Down Expand Up @@ -82,22 +82,22 @@ VmbusOnDeviceRemove(

static void
VmbusOnCleanup(
DRIVER_OBJECT* drv
struct hv_driver *drv
);

static int
VmbusOnISR(
DRIVER_OBJECT* drv
struct hv_driver *drv
);

static void
VmbusOnMsgDPC(
DRIVER_OBJECT* drv
struct hv_driver *drv
);

static void
VmbusOnEventDPC(
DRIVER_OBJECT* drv
struct hv_driver *drv
);

/*++;
Expand All @@ -111,7 +111,7 @@ VmbusOnEventDPC(
--*/
int
VmbusInitialize(
DRIVER_OBJECT* drv
struct hv_driver *drv
)
{
VMBUS_DRIVER_OBJECT* driver = (VMBUS_DRIVER_OBJECT*)drv;
Expand Down Expand Up @@ -377,7 +377,7 @@ int VmbusOnDeviceRemove(
--*/
void
VmbusOnCleanup(
DRIVER_OBJECT* drv
struct hv_driver *drv
)
{
/* VMBUS_DRIVER_OBJECT* driver = (VMBUS_DRIVER_OBJECT*)drv; */
Expand All @@ -401,7 +401,7 @@ VmbusOnCleanup(
--*/
void
VmbusOnMsgDPC(
DRIVER_OBJECT* drv
struct hv_driver *drv
)
{
void *page_addr = gHvContext.synICMessagePage[0];
Expand Down Expand Up @@ -460,7 +460,7 @@ VmbusOnMsgDPC(
--*/
void
VmbusOnEventDPC(
DRIVER_OBJECT* drv
struct hv_driver* drv
)
{
/* TODO: Process any events */
Expand All @@ -479,7 +479,7 @@ VmbusOnEventDPC(
--*/
int
VmbusOnISR(
DRIVER_OBJECT* drv
struct hv_driver *drv
)
{
/* VMBUS_DRIVER_OBJECT* driver = (VMBUS_DRIVER_OBJECT*)drv; */
Expand Down
4 changes: 2 additions & 2 deletions drivers/staging/hv/include/NetVscApi.h
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ typedef struct _NETVSC_PACKET {

/* Represents the net vsc driver */
typedef struct _NETVSC_DRIVER_OBJECT {
DRIVER_OBJECT Base; /* Must be the first field */
struct hv_driver Base; /* Must be the first field */

u32 RingBufferSize;
u32 RequestExtSize;
Expand Down Expand Up @@ -147,7 +147,7 @@ typedef struct _NETVSC_DEVICE_INFO {

int
NetVscInitialize(
DRIVER_OBJECT* drv
struct hv_driver *drv
);

#endif /* _NETVSC_API_H_ */
6 changes: 3 additions & 3 deletions drivers/staging/hv/include/StorVscApi.h
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ typedef struct _STORVSC_REQUEST {

/* Represents the block vsc driver */
typedef struct _STORVSC_DRIVER_OBJECT {
DRIVER_OBJECT Base; /* Must be the first field */
struct hv_driver Base; /* Must be the first field */

/* Set by caller (in bytes) */
u32 RingBufferSize;
Expand Down Expand Up @@ -129,11 +129,11 @@ typedef struct _STORVSC_DEVICE_INFO {

int
StorVscInitialize(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
);

int
BlkVscInitialize(
DRIVER_OBJECT *Driver
struct hv_driver *Driver
);
#endif /* _STORVSC_API_H_ */
26 changes: 10 additions & 16 deletions drivers/staging/hv/include/VmbusApi.h
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,6 @@
#define MAX_PAGE_BUFFER_COUNT 16
#define MAX_MULTIPAGE_BUFFER_COUNT 32 /* 128K */



/* Fwd declarations */

typedef struct _DRIVER_OBJECT *PDRIVER_OBJECT;


/* Data types */


Expand Down Expand Up @@ -68,19 +61,20 @@ typedef struct _MULTIPAGE_BUFFER {

#pragma pack(pop)

struct hv_driver;
struct hv_device;

/* All drivers */
typedef int (*PFN_ON_DEVICEADD)(struct hv_device *Device, void* AdditionalInfo);
typedef int (*PFN_ON_DEVICEREMOVE)(struct hv_device *Device);
typedef char** (*PFN_ON_GETDEVICEIDS)(void);
typedef void (*PFN_ON_CLEANUP)(PDRIVER_OBJECT Driver);
typedef void (*PFN_ON_CLEANUP)(struct hv_driver *Driver);

/* Vmbus extensions */
/* typedef int (*PFN_ON_MATCH)(struct hv_device *dev, PDRIVER_OBJECT drv); */
/* typedef int (*PFN_ON_MATCH)(struct hv_device *dev, struct hv_driver *drv); */
/* typedef int (*PFN_ON_PROBE)(struct hv_device *dev); */
typedef int (*PFN_ON_ISR)(PDRIVER_OBJECT drv);
typedef void (*PFN_ON_DPC)(PDRIVER_OBJECT drv);
typedef int (*PFN_ON_ISR)(struct hv_driver *drv);
typedef void (*PFN_ON_DPC)(struct hv_driver *drv);
typedef void (*PFN_GET_CHANNEL_OFFERS)(void);

typedef struct hv_device *(*PFN_ON_CHILDDEVICE_CREATE)(GUID DeviceType, GUID DeviceInstance, void *Context);
Expand Down Expand Up @@ -205,7 +199,7 @@ typedef struct _VMBUS_CHANNEL_INTERFACE {
typedef void (*VMBUS_GET_CHANNEL_INTERFACE)(VMBUS_CHANNEL_INTERFACE *Interface);

/* Base driver object */
typedef struct _DRIVER_OBJECT {
struct hv_driver {
const char* name;
GUID deviceType; /* the device type supported by this driver */

Expand All @@ -215,12 +209,12 @@ typedef struct _DRIVER_OBJECT {
PFN_ON_CLEANUP OnCleanup;

VMBUS_CHANNEL_INTERFACE VmbusChannelInterface;
} DRIVER_OBJECT;
};


/* Base device object */
struct hv_device {
DRIVER_OBJECT* Driver; /* the driver for this device */
struct hv_driver *Driver; /* the driver for this device */
char name[64];
GUID deviceType; /* the device type id of this device */
GUID deviceInstance; /* the device instance id of this device */
Expand All @@ -231,7 +225,7 @@ struct hv_device {

/* Vmbus driver object */
typedef struct _VMBUS_DRIVER_OBJECT {
DRIVER_OBJECT Base; /* !! Must be the 1st field !! */
struct hv_driver Base; /* !! Must be the 1st field !! */

/* Set by the caller */
PFN_ON_CHILDDEVICE_CREATE OnChildDeviceCreate;
Expand All @@ -257,7 +251,7 @@ typedef struct _VMBUS_DRIVER_OBJECT {

int
VmbusInitialize(
DRIVER_OBJECT* drv
struct hv_driver *drv
);

#endif /* _VMBUS_API_H_ */
4 changes: 2 additions & 2 deletions drivers/staging/hv/include/vmbus.h
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@
/* Data types */


typedef int (*PFN_DRIVERINITIALIZE)(DRIVER_OBJECT *drv);
typedef int (*PFN_DRIVEREXIT)(DRIVER_OBJECT *drv);
typedef int (*PFN_DRIVERINITIALIZE)(struct hv_driver *drv);
typedef int (*PFN_DRIVEREXIT)(struct hv_driver *drv);

struct driver_context {
GUID class_id;
Expand Down
2 changes: 1 addition & 1 deletion drivers/staging/hv/vmbus_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -765,7 +765,7 @@ static int vmbus_match(struct device *device, struct device_driver *driver)
if (memcmp(&device_ctx->class_id, &driver_ctx->class_id, sizeof(GUID)) == 0)
{
/* !! NOTE: The driver_ctx is not a vmbus_drv_ctx. We typecast it here to access the */
/* DRIVER_OBJECT field */
/* struct hv_driver field */
struct vmbus_driver_context *vmbus_drv_ctx = (struct vmbus_driver_context*)driver_ctx;
device_ctx->device_obj.Driver = &vmbus_drv_ctx->drv_obj.Base;
DPRINT_INFO(VMBUS_DRV, "device object (%p) set to driver object (%p)", &device_ctx->device_obj, device_ctx->device_obj.Driver);
Expand Down

0 comments on commit 775ef25

Please sign in to comment.