* [PATCH] Staging: vt6656: Remove the ULONG_PTR and DWORD_PTR defines from ttype.h @ 2010-07-12 22:49 Roel Van Nyen 2010-08-05 16:23 ` Roel Van Nyen 0 siblings, 1 reply; 5+ messages in thread From: Roel Van Nyen @ 2010-07-12 22:49 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: Forest Bond, linux-kernel, Roel Van Nyen Remove the ULONG and DWORD PTR defines from ttype.h Signed-of-by: Roel Van Nyen <roel.vannyen@gmail.com> --- drivers/staging/vt6656/bssdb.c | 8 ++++---- drivers/staging/vt6656/ttype.h | 3 --- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/staging/vt6656/bssdb.c b/drivers/staging/vt6656/bssdb.c index 830dd23..0b8584b 100644 --- a/drivers/staging/vt6656/bssdb.c +++ b/drivers/staging/vt6656/bssdb.c @@ -469,7 +469,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, unsigned int uLen = pRSNWPA->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { pBSSList->wWPALen = uLen; memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); WPA_ParseRSN(pBSSList, pRSNWPA); @@ -482,7 +482,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, unsigned int uLen = pRSN->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { pBSSList->wRSNLen = uLen; memcpy(pBSSList->byRSNIE, pRSN, uLen); WPA2vParseRSN(pBSSList, pRSN); @@ -674,7 +674,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, if (pRSNWPA != NULL) { unsigned int uLen = pRSNWPA->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { pBSSList->wWPALen = uLen; memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); WPA_ParseRSN(pBSSList, pRSNWPA); @@ -686,7 +686,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, if (pRSN != NULL) { unsigned int uLen = pRSN->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { pBSSList->wRSNLen = uLen; memcpy(pBSSList->byRSNIE, pRSN, uLen); WPA2vParseRSN(pBSSList, pRSN); diff --git a/drivers/staging/vt6656/ttype.h b/drivers/staging/vt6656/ttype.h index 3425773..42091c4 100644 --- a/drivers/staging/vt6656/ttype.h +++ b/drivers/staging/vt6656/ttype.h @@ -83,9 +83,6 @@ typedef UQuadWord QWORD; // 64-bit /****** Common pointer types ***********************************************/ -typedef unsigned long ULONG_PTR; // 32-bit -typedef unsigned long DWORD_PTR; // 32-bit - // boolean pointer typedef unsigned int * PUINT; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] Staging: vt6656: Remove the ULONG_PTR and DWORD_PTR defines from ttype.h 2010-07-12 22:49 [PATCH] Staging: vt6656: Remove the ULONG_PTR and DWORD_PTR defines from ttype.h Roel Van Nyen @ 2010-08-05 16:23 ` Roel Van Nyen 2010-08-19 10:18 ` Roel Van Nyen 2010-08-31 18:33 ` Greg KH 0 siblings, 2 replies; 5+ messages in thread From: Roel Van Nyen @ 2010-08-05 16:23 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: Forest Bond, linux-kernel On Tue, Jul 13, 2010 at 12:49:57AM +0200, Roel Van Nyen wrote: > Remove the ULONG and DWORD PTR defines from ttype.h > > Signed-of-by: Roel Van Nyen <roel.vannyen@gmail.com> > --- > drivers/staging/vt6656/bssdb.c | 8 ++++---- > drivers/staging/vt6656/ttype.h | 3 --- > 2 files changed, 4 insertions(+), 7 deletions(-) > > diff --git a/drivers/staging/vt6656/bssdb.c b/drivers/staging/vt6656/bssdb.c > index 830dd23..0b8584b 100644 > --- a/drivers/staging/vt6656/bssdb.c > +++ b/drivers/staging/vt6656/bssdb.c > @@ -469,7 +469,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, > unsigned int uLen = pRSNWPA->len + 2; > > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { > pBSSList->wWPALen = uLen; > memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); > WPA_ParseRSN(pBSSList, pRSNWPA); > @@ -482,7 +482,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, > unsigned int uLen = pRSN->len + 2; > > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { > pBSSList->wRSNLen = uLen; > memcpy(pBSSList->byRSNIE, pRSN, uLen); > WPA2vParseRSN(pBSSList, pRSN); > @@ -674,7 +674,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, > if (pRSNWPA != NULL) { > unsigned int uLen = pRSNWPA->len + 2; > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { > pBSSList->wWPALen = uLen; > memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); > WPA_ParseRSN(pBSSList, pRSNWPA); > @@ -686,7 +686,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, > if (pRSN != NULL) { > unsigned int uLen = pRSN->len + 2; > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { > pBSSList->wRSNLen = uLen; > memcpy(pBSSList->byRSNIE, pRSN, uLen); > WPA2vParseRSN(pBSSList, pRSN); > diff --git a/drivers/staging/vt6656/ttype.h b/drivers/staging/vt6656/ttype.h > index 3425773..42091c4 100644 > --- a/drivers/staging/vt6656/ttype.h > +++ b/drivers/staging/vt6656/ttype.h > @@ -83,9 +83,6 @@ typedef UQuadWord QWORD; // 64-bit > > /****** Common pointer types ***********************************************/ > > -typedef unsigned long ULONG_PTR; // 32-bit > -typedef unsigned long DWORD_PTR; // 32-bit > - > // boolean pointer > typedef unsigned int * PUINT; > > -- > 1.7.0.4 > Hi Greg, I have re-done my patch and this time without any checkpatch.pl errors. Remove DWORD_PTR and ULONG_PTR from ttype.h Signed-off-by: Roel Van Nyen <roel.vannyen@gmail.com> --- drivers/staging/vt6656/bssdb.c | 8 ++++---- drivers/staging/vt6656/ttype.h | 4 ---- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/staging/vt6656/bssdb.c b/drivers/staging/vt6656/bssdb.c index a9f68bd..024dff3 100644 --- a/drivers/staging/vt6656/bssdb.c +++ b/drivers/staging/vt6656/bssdb.c @@ -466,7 +466,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, unsigned int uLen = pRSNWPA->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { pBSSList->wWPALen = uLen; memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); WPA_ParseRSN(pBSSList, pRSNWPA); @@ -479,7 +479,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, unsigned int uLen = pRSN->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { pBSSList->wRSNLen = uLen; memcpy(pBSSList->byRSNIE, pRSN, uLen); WPA2vParseRSN(pBSSList, pRSN); @@ -671,7 +671,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, if (pRSNWPA != NULL) { unsigned int uLen = pRSNWPA->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { pBSSList->wWPALen = uLen; memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); WPA_ParseRSN(pBSSList, pRSNWPA); @@ -683,7 +683,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, if (pRSN != NULL) { unsigned int uLen = pRSN->len + 2; if (uLen <= (uIELength - - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { pBSSList->wRSNLen = uLen; memcpy(pBSSList->byRSNIE, pRSN, uLen); WPA2vParseRSN(pBSSList, pRSN); diff --git a/drivers/staging/vt6656/ttype.h b/drivers/staging/vt6656/ttype.h index 8e9450e..e8ed3ff 100644 --- a/drivers/staging/vt6656/ttype.h +++ b/drivers/staging/vt6656/ttype.h @@ -59,10 +59,6 @@ typedef union tagUQuadWord { typedef UQuadWord QWORD; // 64-bit /****** Common pointer types ***********************************************/ - -typedef unsigned long ULONG_PTR; // 32-bit -typedef unsigned long DWORD_PTR; // 32-bit - // boolean pointer typedef BYTE * PBYTE; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] Staging: vt6656: Remove the ULONG_PTR and DWORD_PTR defines from ttype.h 2010-08-05 16:23 ` Roel Van Nyen @ 2010-08-19 10:18 ` Roel Van Nyen 2010-08-19 15:13 ` Greg KH 2010-08-31 18:33 ` Greg KH 1 sibling, 1 reply; 5+ messages in thread From: Roel Van Nyen @ 2010-08-19 10:18 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: Forest Bond, linux-kernel Hi greg, Did you see my previous mail with this patch or did it get lost in the noise ? Thanks, Roel 2010/8/5 Roel Van Nyen <roel.vannyen@gmail.com>: > On Tue, Jul 13, 2010 at 12:49:57AM +0200, Roel Van Nyen wrote: >> Remove the ULONG and DWORD PTR defines from ttype.h >> >> Signed-of-by: Roel Van Nyen <roel.vannyen@gmail.com> >> --- >> drivers/staging/vt6656/bssdb.c | 8 ++++---- >> drivers/staging/vt6656/ttype.h | 3 --- >> 2 files changed, 4 insertions(+), 7 deletions(-) >> >> diff --git a/drivers/staging/vt6656/bssdb.c b/drivers/staging/vt6656/bssdb.c >> index 830dd23..0b8584b 100644 >> --- a/drivers/staging/vt6656/bssdb.c >> +++ b/drivers/staging/vt6656/bssdb.c >> @@ -469,7 +469,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, >> unsigned int uLen = pRSNWPA->len + 2; >> >> if (uLen <= (uIELength - >> - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { >> + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { >> pBSSList->wWPALen = uLen; >> memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); >> WPA_ParseRSN(pBSSList, pRSNWPA); >> @@ -482,7 +482,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, >> unsigned int uLen = pRSN->len + 2; >> >> if (uLen <= (uIELength - >> - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { >> + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { >> pBSSList->wRSNLen = uLen; >> memcpy(pBSSList->byRSNIE, pRSN, uLen); >> WPA2vParseRSN(pBSSList, pRSN); >> @@ -674,7 +674,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, >> if (pRSNWPA != NULL) { >> unsigned int uLen = pRSNWPA->len + 2; >> if (uLen <= (uIELength - >> - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { >> + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { >> pBSSList->wWPALen = uLen; >> memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); >> WPA_ParseRSN(pBSSList, pRSNWPA); >> @@ -686,7 +686,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, >> if (pRSN != NULL) { >> unsigned int uLen = pRSN->len + 2; >> if (uLen <= (uIELength - >> - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { >> + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { >> pBSSList->wRSNLen = uLen; >> memcpy(pBSSList->byRSNIE, pRSN, uLen); >> WPA2vParseRSN(pBSSList, pRSN); >> diff --git a/drivers/staging/vt6656/ttype.h b/drivers/staging/vt6656/ttype.h >> index 3425773..42091c4 100644 >> --- a/drivers/staging/vt6656/ttype.h >> +++ b/drivers/staging/vt6656/ttype.h >> @@ -83,9 +83,6 @@ typedef UQuadWord QWORD; // 64-bit >> >> /****** Common pointer types ***********************************************/ >> >> -typedef unsigned long ULONG_PTR; // 32-bit >> -typedef unsigned long DWORD_PTR; // 32-bit >> - >> // boolean pointer >> typedef unsigned int * PUINT; >> >> -- >> 1.7.0.4 >> > > Hi Greg, > > I have re-done my patch and this time without any checkpatch.pl errors. > > Remove DWORD_PTR and ULONG_PTR from ttype.h > > Signed-off-by: Roel Van Nyen <roel.vannyen@gmail.com> > --- > drivers/staging/vt6656/bssdb.c | 8 ++++---- > drivers/staging/vt6656/ttype.h | 4 ---- > 2 files changed, 4 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/vt6656/bssdb.c b/drivers/staging/vt6656/bssdb.c > index a9f68bd..024dff3 100644 > --- a/drivers/staging/vt6656/bssdb.c > +++ b/drivers/staging/vt6656/bssdb.c > @@ -466,7 +466,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, > unsigned int uLen = pRSNWPA->len + 2; > > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { > pBSSList->wWPALen = uLen; > memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); > WPA_ParseRSN(pBSSList, pRSNWPA); > @@ -479,7 +479,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, > unsigned int uLen = pRSN->len + 2; > > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { > pBSSList->wRSNLen = uLen; > memcpy(pBSSList->byRSNIE, pRSN, uLen); > WPA2vParseRSN(pBSSList, pRSN); > @@ -671,7 +671,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, > if (pRSNWPA != NULL) { > unsigned int uLen = pRSNWPA->len + 2; > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { > pBSSList->wWPALen = uLen; > memcpy(pBSSList->byWPAIE, pRSNWPA, uLen); > WPA_ParseRSN(pBSSList, pRSNWPA); > @@ -683,7 +683,7 @@ BOOL BSSbUpdateToBSSList(void *hDeviceContext, > if (pRSN != NULL) { > unsigned int uLen = pRSN->len + 2; > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSN - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSN - pbyIEs))) { > pBSSList->wRSNLen = uLen; > memcpy(pBSSList->byRSNIE, pRSN, uLen); > WPA2vParseRSN(pBSSList, pRSN); > diff --git a/drivers/staging/vt6656/ttype.h b/drivers/staging/vt6656/ttype.h > index 8e9450e..e8ed3ff 100644 > --- a/drivers/staging/vt6656/ttype.h > +++ b/drivers/staging/vt6656/ttype.h > @@ -59,10 +59,6 @@ typedef union tagUQuadWord { > typedef UQuadWord QWORD; // 64-bit > > /****** Common pointer types ***********************************************/ > - > -typedef unsigned long ULONG_PTR; // 32-bit > -typedef unsigned long DWORD_PTR; // 32-bit > - > // boolean pointer > > typedef BYTE * PBYTE; > -- > 1.7.0.4 > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Staging: vt6656: Remove the ULONG_PTR and DWORD_PTR defines from ttype.h 2010-08-19 10:18 ` Roel Van Nyen @ 2010-08-19 15:13 ` Greg KH 0 siblings, 0 replies; 5+ messages in thread From: Greg KH @ 2010-08-19 15:13 UTC (permalink / raw) To: Roel Van Nyen; +Cc: Forest Bond, linux-kernel On Thu, Aug 19, 2010 at 12:18:50PM +0200, Roel Van Nyen wrote: > Hi greg, > > Did you see my previous mail with this patch or did it get lost in the noise ? I have it, but it was during the merge window so I will queue it up in a few days for the next major merge window. thanks for your patience. greg k-h ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Staging: vt6656: Remove the ULONG_PTR and DWORD_PTR defines from ttype.h 2010-08-05 16:23 ` Roel Van Nyen 2010-08-19 10:18 ` Roel Van Nyen @ 2010-08-31 18:33 ` Greg KH 1 sibling, 0 replies; 5+ messages in thread From: Greg KH @ 2010-08-31 18:33 UTC (permalink / raw) To: Roel Van Nyen; +Cc: Greg Kroah-Hartman, Forest Bond, linux-kernel On Thu, Aug 05, 2010 at 06:23:21PM +0200, Roel Van Nyen wrote: > I have re-done my patch and this time without any checkpatch.pl errors. Well, some you can ignore: > > Remove DWORD_PTR and ULONG_PTR from ttype.h > > Signed-off-by: Roel Van Nyen <roel.vannyen@gmail.com> > --- > drivers/staging/vt6656/bssdb.c | 8 ++++---- > drivers/staging/vt6656/ttype.h | 4 ---- > 2 files changed, 4 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/vt6656/bssdb.c b/drivers/staging/vt6656/bssdb.c > index a9f68bd..024dff3 100644 > --- a/drivers/staging/vt6656/bssdb.c > +++ b/drivers/staging/vt6656/bssdb.c > @@ -466,7 +466,7 @@ BOOL BSSbInsertToBSSList(void *hDeviceContext, > unsigned int uLen = pRSNWPA->len + 2; > > if (uLen <= (uIELength - > - (unsigned int) (ULONG_PTR) ((PBYTE) pRSNWPA - pbyIEs))) { > + (unsigned int) (unsigned long) ((PBYTE) pRSNWPA - pbyIEs))) { That's just horrible, move it to the right like it was, just don't add the extra spaces where they are not needed. And think, why would you want to cast from an unsigned long and then to an unsigned int, something is really wrong with that code, it should be fixed properly. Care to try again? thanks, greg k-h ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-08-31 18:36 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-07-12 22:49 [PATCH] Staging: vt6656: Remove the ULONG_PTR and DWORD_PTR defines from ttype.h Roel Van Nyen 2010-08-05 16:23 ` Roel Van Nyen 2010-08-19 10:18 ` Roel Van Nyen 2010-08-19 15:13 ` Greg KH 2010-08-31 18:33 ` Greg KH
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox