Re: [PATCH 2/3] staging: vt6656: Remove ULONG_PTR/DWORD_PTR from driver.

[Greg KH <gregkh@linuxfoundation.org>]

On Wed, Nov 28, 2012 at 09:15:48PM +0000, Malcolm Priestley wrote:
> 
> Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
> ---
>  drivers/staging/vt6656/bssdb.c |   89 +++++++++++++++++++---------------------
>  drivers/staging/vt6656/ttype.h |    3 --
>  2 files changed, 42 insertions(+), 50 deletions(-)
> 
> diff --git a/drivers/staging/vt6656/bssdb.c b/drivers/staging/vt6656/bssdb.c
> index 539bc1d..d27469e 100644
> --- a/drivers/staging/vt6656/bssdb.c
> +++ b/drivers/staging/vt6656/bssdb.c
> @@ -355,13 +355,13 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext,
>  			 PBYTE pbyIEs,
>  			 void *pRxPacketContext)
>  {
> -
> -    PSDevice     pDevice = (PSDevice)hDeviceContext;
> -    PSMgmtObject    pMgmt = &(pDevice->sMgmtObj);
> -    PSRxMgmtPacket  pRxPacket = (PSRxMgmtPacket)pRxPacketContext;
> -    PKnownBSS       pBSSList = NULL;
> -    unsigned int            ii;
> -    BOOL            bParsingQuiet = FALSE;
> +	PSDevice pDevice = (PSDevice)hDeviceContext;
> +	PSMgmtObject pMgmt = &pDevice->sMgmtObj;
> +	PSRxMgmtPacket pRxPacket = (PSRxMgmtPacket)pRxPacketContext;
> +	PKnownBSS pBSSList = NULL;
> +	unsigned int ii;
> +	int bParsingQuiet = FALSE;
> +	u32 uLen;

This patch also adjusts formatting, making it hard to see that you
really changed the BOOL variable here, which is not what you said the
patch was doing :(

Please redo this patch, and _only_ make the changes you say you are
making.

thanks,

greg k-h