Search Linux Wireless

Re: [PATCH 1/2] staging: wilc1000: modify type casting warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 16, 2015 at 04:11:24PM +0900, Dean Lee wrote:
> modify message "warning: cast from pointer to integer of different size".
> add type cast 'uintptr_t'.
> 
> Signed-off-by: Dean Lee <dean.lee@xxxxxxxxx>
> ---
> Change Log:
>  add Signed-off-by line
> 
>  drivers/staging/wilc1000/host_interface.c         | 129 +++++++++++-----------
>  drivers/staging/wilc1000/linux_wlan.c             |   8 +-
>  drivers/staging/wilc1000/wilc_wfi_cfgoperations.c |  22 ++--
>  3 files changed, 80 insertions(+), 79 deletions(-)
> 
> diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c
> index 3f139ae..358283f 100644
> --- a/drivers/staging/wilc1000/host_interface.c
> +++ b/drivers/staging/wilc1000/host_interface.c
> @@ -1,5 +1,6 @@
>  #include "host_interface.h"
>  #include "coreconfigurator.h"
> +#include <linux/types.h>
>  
>  extern s32 TransportInit(void);
>  extern s32 TransportDeInit(void);
> @@ -620,7 +621,7 @@ static s32 Handle_SetChannel(void *drvHandler, tstrHostIFSetChan *pstrHostIFSetC
>  
>  	PRINT_D(HOSTINF_DBG, "Setting channel\n");
>  	/*Sending Cfg*/
> -	s32Error = SendConfigPkt(SET_CFG, &strWID, 1, true, (u32)pstrWFIDrv);
> +	s32Error = SendConfigPkt(SET_CFG, &strWID, 1, true, (u32)(uintptr_t)pstrWFIDrv);

This shows you are doing something really wrong here, why not just fix
the function type to be a real pointer to a real structure?  Or even
better, just remove the parameter as I don't think they are even used.

thanks,

greg k-h
--
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



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux