Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Greg KH
On Wed, Oct 15, 2014 at 03:59:21PM -0700, Jeff Kirsher wrote: > On Wed, 2014-10-15 at 12:01 -0700, Jeff Kirsher wrote: > > From: Fabio Estevam > > > > The Beceem WIMAX was generating compile warnings on 64bit machines, > > which were: > > > > drivers/staging/bcm/CmHost.c: In function ‘StoreCmCon

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Dan Carpenter
COMPILE_TEST is just so that you can view build warnings. We would say: depends on USB && NET && (!64BIT || COMPILE_TEST) of depends on USB && NET && (X86_32 || (X86 && COMPILE_TEST)) If you don't have USB or NET, you're get build errors but they don't indicate bugs. If you build on X86_64 th

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Jeff Kirsher
On Thu, 2014-10-16 at 10:52 +0300, Dan Carpenter wrote: > On Wed, Oct 15, 2014 at 01:26:07PM -0700, Jeff Kirsher wrote: > > On Wed, 2014-10-15 at 22:55 +0300, Dan Carpenter wrote: > > > On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: > > > > From: Fabio Estevam > > > > > > > > The B

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Dan Carpenter
On Wed, Oct 15, 2014 at 01:26:07PM -0700, Jeff Kirsher wrote: > On Wed, 2014-10-15 at 22:55 +0300, Dan Carpenter wrote: > > On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: > > > From: Fabio Estevam > > > > > > The Beceem WIMAX was generating compile warnings on 64bit machines, > > >

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Jeff Kirsher
On Wed, 2014-10-15 at 12:01 -0700, Jeff Kirsher wrote: > From: Fabio Estevam > > The Beceem WIMAX was generating compile warnings on 64bit machines, > which were: > > drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: > drivers/staging/bcm/CmHost.c:1503:3: warning: cast t

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Jeff Kirsher
On Wed, 2014-10-15 at 22:55 +0300, Dan Carpenter wrote: > On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: > > From: Fabio Estevam > > > > The Beceem WIMAX was generating compile warnings on 64bit machines, > > which were: > > > > drivers/staging/bcm/CmHost.c: In function ‘StoreCmCo

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Dan Carpenter
On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: > From: Fabio Estevam > > The Beceem WIMAX was generating compile warnings on 64bit machines, > which were: > > drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: > drivers/staging/bcm/CmHost.c:1503:3: warning:

[PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Jeff Kirsher
From: Fabio Estevam The Beceem WIMAX was generating compile warnings on 64bit machines, which were: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1503:3: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]