Messages in this thread | | | Subject | Re: [PATCH][Trivial] staging: vt6656: Remove some redundant casts and parentheses + other style cleanups | From | Joe Perches <> | Date | Tue, 17 Apr 2012 14:23:32 -0600 |
| |
On Mon, 2012-04-16 at 23:20 +0200, Jesper Juhl wrote: > This is just a small/trivial cleanup patch. It makes the following > changes to drivers/staging/vt6656/ioctl.c: > > 1. Remove some redundant casts.
drivers/staging/vt6656/wcmd.h:BOOL bScheduleCommand(void *hDeviceContext,
so you could remove the casts of the first argument too.
> diff --git a/drivers/staging/vt6656/ioctl.c b/drivers/staging/vt6656/ioctl.c [] > @@ -101,7 +101,7 @@ int private_ioctl(PSDevice pDevice, struct ifreq *rq) > WLAN_CMD_BSSID_SCAN, > abyScanSSID); > else > - bScheduleCommand((void *) pDevice, WLAN_CMD_BSSID_SCAN, NULL); > + bScheduleCommand((void *)pDevice, WLAN_CMD_BSSID_SCAN, NULL);
bScheduleCommand(pDevice, WLAN_CMD_BSSID_SCAN, NULL); [] > @@ -190,10 +191,9 @@ int private_ioctl(PSDevice pDevice, struct ifreq *rq) > netif_stop_queue(pDevice->dev); > spin_lock_irq(&pDevice->lock); > pMgmt->eCurrState = WMAC_STATE_IDLE; > - bScheduleCommand((void *) pDevice, > - WLAN_CMD_BSSID_SCAN, > + bScheduleCommand((void *)pDevice, WLAN_CMD_BSSID_SCAN, > pMgmt->abyDesireSSID); bScheduleCommand(pDevice, WLAN_CMD_BSSID_SCAN, pMgmt->abyDesireSSID); etc
| |