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 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel