On Tue, Jun 30, 2015 at 05:34:35PM +0900, Dean Lee wrote:
> argument type change for support 64 bit system.
>
> Signed-off-by: Dean Lee <dean.lee@xxxxxxxxx>
> -u32 gu8FlushedJoinReqDrvHandler;
> +void *gpFlushedJoinReqDrvHandler;
There are a lot of variables renamed. It makes reviewing this patch
hard. Also the new names are bad style so just keep the original names
for now and fix the names in a later patch.
> - if ((pstrHostIfSetOperationMode->u32Mode) == (u32)NULL) {
> + if (pstrHostIfSetOperationMode == NULL) {
This was not in the patch description. Do it in a later patch.
> @@ -6838,17 +6838,14 @@ void NetworkInfoReceived(u8 *pu8Buffer, u32 u32Length)
> {
> s32 s32Error = WILC_SUCCESS;
> tstrHostIFmsg strHostIFmsg;
> - u32 drvHandler;
> - tstrWILC_WFIDrv *pstrWFIDrv = NULL;
> -
> - drvHandler = ((pu8Buffer[u32Length - 4]) | (pu8Buffer[u32Length - 3] << 8) | (pu8Buffer[u32Length - 2] << 16) | (pu8Buffer[u32Length - 1] << 24));
> - pstrWFIDrv = (tstrWILC_WFIDrv *)drvHandler;
> -
> + uintptr_t drvData;
> + void *drvHandler = NULL;
More renaming. Do this in a different patch.
> static int linux_wlan_init_test_config(struct net_device *dev, linux_wlan_t *p_nic)
> {
> -
> + void *drvHandler = NULL;
Bad indenting.
> unsigned char c_val[64];
> #ifndef STATIC_MACADDRESS
> unsigned char mac_add[] = {0x00, 0x80, 0xC2, 0x5E, 0xa2, 0xff};
> @@ -1102,62 +1102,62 @@ static int linux_wlan_init_test_config(struct net_device *dev, linux_wlan_t *p_n
> goto _fail_;
> }
>
> - *(int *)c_val = (u32)pstrWFIDrv;
> + *(int *)c_val = (uintptr_t)pstrWFIDrv;
>
> - if (!g_linux_wlan->oup.wlan_cfg_set(1, WID_SET_DRV_HANDLER, c_val, 4, 0, 0))
> + if (!g_linux_wlan->oup.wlan_cfg_set(1, WID_SET_DRV_HANDLER, c_val, 4, 0, drvHandler))
> goto _fail_;
We are changing from 0 to drvHandler. That is not described in the
patch description. Do it in a different patch.
regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
